Using import-map-overrides for making an 'in-browser' IDE.

You say multiple times in the documentation that its risky to use import-map-overrides in production (and to use it only during development). I'm putting together a browser-based IDE, and it seems like import-map-overrides is the only way to let users of the IDE link to external dependencies.
I understand that doing this opens up arbitrary code execution and self-xss attacks. But those are par for the course for an IDE anyway. (Wouldn't be much of an IDE if you couldn't run code in it after all. ;) )
I'm curious if this is the security (and probably performance) implications are the only reasons you say to not do this, or are there other reasons too?  (And if it really is a bad idea to use import-map-overrides, is there any other way to create a dynamic import-map type of thing (or a lexically scoped import-map)).
Thanks.

Security is the only reason to not use import-map-overrides in production. I have used it in production and it works great. Performance overhead is not really a concern, as import-map-overrides just inserts a <script> into the DOM after reading local storage, and neither of those are extremely slow (especially since it doesn't read all values of local storage, which could be long and slow to read, but only the import-map-overrides: local storage values).
I don't see any reason not to use import-map-overrides in production - I wish you best of luck in doing so!


conemu.exe missing?

Are we meant to download it ourselves and place it in tools/conemu?

Did you try to build from source as described here, or download release package?

Ah, sorry. I did a git clone and forgot that put me in the "building from source" group... I'll grab the zip file.

@milh0use yeah, the releases page with the zip files is not immediately obvious. I'm currently uploading 3.0-rc3, which has some more improvements (especially if you are working with docker) and bugfixes. You might want to grab that version once it's up.

...and its up now: https://github.com/tknerr/bills-kitchen/releases/tag/3.0-rc3


translationData[langasset] is not a function

Howdy,
with any fresh qwik install and following your quick install guide i end up with the following error;
20:01:48 [vite] Internal server error: translationData[langasset] is not a function
  File: /IdeaProjects/dare-qwik-4/src/speak-functions.ts:13:5
  11 |   */
  12 |  const loadTranslation$: LoadTranslationFn = server$(async (lang: string, asset: string) =>
  13 |    await translationData[`/i18n/${lang}/${asset}.json`]()
     |       ^
  14 |  );

If i remove the asset: string and have it like
await translationData[/i18n/${lang}/welcome.json]()
I use welcome because of the component;
export default component$(() => {
  return (
    <Speak assets={['welcome']}>
      <Welcome />
    </Speak>
  );
});

it will work just fine
all the code is from the quick install, either i'm missing something or there is a bug...
Can somebody verify if there's a bug or i'm just overlooking something ^^

If i debug the following;
const translationData = import.meta.glob('/i18n/**/*.json');
console.log(translationData)

it will return the follwoing object which is inline with the stackblitz example
{
  '/i18n/de-DE/dashboard.json': [Function: /i18n/de-DE/dashboard.json],
  '/i18n/de-DE/welcome.json': [Function: /i18n/de-DE/welcome.json],
  '/i18n/en-GB/dashboard.json': [Function: /i18n/en-GB/dashboard.json],
  '/i18n/en-GB/welcome.json': [Function: /i18n/en-GB/welcome.json],
  '/i18n/es-ES/dashboard.json': [Function: /i18n/es-ES/dashboard.json],
  '/i18n/es-ES/welcome.json': [Function: /i18n/es-ES/welcome.json],
  '/i18n/fr-FR/dashboard.json': [Function: /i18n/fr-FR/dashboard.json],
  '/i18n/fr-FR/welcome.json': [Function: /i18n/fr-FR/welcome.json],
  '/i18n/nl-NL/dashboard.json': [Function: /i18n/nl-NL/dashboard.json],
  '/i18n/nl-NL/welcome.json': [Function: /i18n/nl-NL/welcome.json]
}

Maybe it will help with getting to the bottom of the issue

The error indicates that you are trying to pass an asset that is not found among those imported from import.meta.glob('/i18n/**/*.json').
Are you sure all your assets exist? Please check assets values in configuration and in Speak components.
You could also catch the error:
const loadTranslation$: LoadTranslationFn = server$((lang: string, asset: string) => {
  const langAsset = `/i18n/${lang}/${asset}.json`;
  if (langAsset in translationData) {
    return translationData[langAsset]();
  } else {
    console.warn(`loadTranslation$: ${langAsset} not found`);
    return null;
  }
});

 loadTranslation$ is a customizable function.

Awesome,
The error catching was the solution it give the proper insight of the issue and indeed it was a asset that didn't exist.
Thank for the help.


1048-longest-string-chain problem

CPP solution to 1048-longest-string-chain problem (#666)
Runtime: 309 ms
Memory: 49.4 MB

Added code for Leetcode: 188. Best Time to Buy and Sell Stock IV (#472)
Dynamic Programming solution
Runtime: 20 ms
Memory: 10.9 MB


Catification

Description
We would like to see an addition to the Sir Lancebot commands: .catify. This would insert random ᓚᘏᗢ's into a message sent by the invoker of the command.
Reasoning
This would be useful since it enhances the ᓚᘏᗢ cult.
Proposed Implementation

Additional Details
Would you like to implement this yourself?

[ ] I'd like to implement this feature myself
[x ] Anyone can implement this feature

#691 is the same thing/

Yeah I kinda beat you to it

I'm going to close this as it's a duplicate of #691. We can move discussion there.

got it.


Laravel 5.3 & 5.4 Support

Please update your package to include support for Laravel 5.3 and 5.4.

Hello! Is there anything I can help with to make this compatible with 5.4?
Thank you.

Bump.

up!!


(unify) Update code style.

Fix code style
Using Go 1.20 or 1.21, run:
find . -name '*.go' -exec gofmt -s -w {} \;

This should update the formatting so it is compliant.
Enable copyright checking in the action
In .github.workflows/ci.yml remove the line style-skip: true.

Addressed by https://github.com/xmidt-org/retry/pull/15


Documents/examples about setting entity physcial properties(for domain randomization in RL training)

Thank you for creating this excellent simulator! I am currently migrating my reinforcement learning code from IsaacGym to Genesis. While working on this transition, I've noticed that I need some clarification regarding the APIs for setting physical properties of entities.
Specifically, I'm looking for information on how to set:
Link properties:
Mass
Inertial properties
Center of mass
Joint properties:
Friction
Motor rotor parameters (armature/inertia)
RigidShape material properties:
Friction coefficients
Restitution
I've found some relevant APIs in the documentation:
set_mass(link)
set_friction(link)
set_friction(geometry)
set_friction_ratio(entity)
However, I'm wondering if there are additional APIs for setting other physical properties that I might have missed. Would you be able to:
Point me to any documentation covering these features?
Share examples demonstrating how to set these various properties?
Provide a comprehensive list of all available property-setting APIs?
Or is it possible to customize these APIs easily if they are not currently available?
Thank you for your time and assistance.

Thanks for bringing this up. Theoretically, we should be able to set most of the properties, but we need to write an interface to extract those tensor via the kernel, e.g., from dofs, from link, from geom, etc. We only implemented those we were using. Would you mind make a list of physical properties that you would like to set so that we can incorporate the interface later?
Another note is that modifying some physical properties may make simulation unstable.

In RigidSolver, you can find
set_links_mass_shift
set_links_COM_shift
for domain randomization of mass and com.

It seems that there is no interface for setting link speed and force

Thanks for bringing this up. Theoretically, we should be able to set most of the properties, but we need to write an interface to extract those tensor via the kernel, e.g., from dofs, from link, from geom, etc. We only implemented those we were using. Would you mind make a list of physical properties that you would like to set so that we can incorporate the interface later?
Another note is that modifying some physical properties may make simulation unstable.

Thank you very much for the quick respose. From my experience, the most common used physical properties in DR are:

from dofs: armature; friction.
from link: mass, com, inertia
from geom: friction, restitution
others: add external force on links/base.
I've found that most of these properties can already be set using the existing APIs, which is great. For the few remaining properties that aren't yet implemented, thank you very much.

Thanks for bringing this up. Theoretically, we should be able to set most of the properties, but we need to write an interface to extract those tensor via the kernel, e.g., from dofs, from link, from geom, etc. We only implemented those we were using. Would you mind make a list of physical properties that you would like to set so that we can incorporate the interface later?
Another note is that modifying some physical properties may make simulation unstable.

Bumping in the discussion for this topic.
AFAIK, the common practice for the entitity-related randomized parameters such as frictions, restitution, etc. are randomized per entity at the time the environment is created. Thus, there is no need to reset the values at every episode reset.
Then would it still affect the simulation stability/accuracy?

Thanks for bringing this up. Theoretically, we should be able to set most of the properties, but we need to write an interface to extract those tensor via the kernel, e.g., from dofs, from link, from geom, etc. We only implemented those we were using. Would you mind make a list of physical properties that you would like to set so that we can incorporate the interface later?
Another note is that modifying some physical properties may make simulation unstable.

Bumping in the discussion for this topic.
AFAIK, the common practice for the entitity-related randomized parameters such as frictions, restitution, etc. are randomized per entity at the time the environment is created. Thus, there is no need to reset the values at every episode reset.
Then would it still affect the simulation stability/accuracy?

This doesn’t seem like a common practice but rather a limitation, so I wouldn’t recommend keeping it this way.

It seems that there is no interface for setting link speed and force

Hi! I've added a function to apply external force and torque on links. Please check examples/rigid/apply_external_force_torque.py fir the detail

Thank you very much for the quick respose. From my experience, the most common used physical properties in DR are:

from dofs: armature; friction.
from link: mass, com, inertia
from geom: friction, restitution
others: add external force on links/base.
I've found that most of these properties can already be set using the existing APIs, which is great. For the few remaining properties that aren't yet implemented, thank you very much.

Currently some of the attributes are not stored in batch and thus we cannot set different params across environments, including dofs_info, links_info, geoms_info. @YilingQiao do we want to set back to include the batch dimension for these info stuffs? Or perhaps include an argument like include_batch_dim_in_info in rigid solver option to toggle this? Or doing the shift thingy in the xxx_state field?


Custom indicator data

Hi, How can I send custom indicator data to zmq?

Hi @2knarf ,
It depends on what you're trying to achieve by doing that.
For example,

You could retrieve data for say the RSI indicator applied to an asset e.g. EURUSD,  via ZeroMQ from MetaTrader.

If you were calculating an indicator outside of MetaTrader and using it to perform trading actions, you wouldn't need to send that data to MetaTrader, you could simply send orders to execute based on such an external indicator.

Hope the above helps!

Hi. Thanks for the reply. I want to achieve #1. Can you give me some guidelines for This?

Hi @2knarf ,
To achieve point 1, you'd need to copy one of the DWX_() methods in the MQL EA and edit it to include your desired functionality.
We don't unfortunately have the resources at the present time to provide specific programming support in MQL and Python, but the code is fairly self-explanatory so hopefully should be fairly straightforward for you to implement.
All the best!


add go.work.sum to .gitignore

To my understanding this file doesn't have to be tracked though I'm not sure as there really isn't that much information about it.

go.work and go.work.sum aren't actually necessary as they refer to the local environment
See
https://github.com/github/gitignore/pull/3884
https://github.com/github/gitignore/pull/4081

Makes sense to me :+1: But shouldn't we also delete the files from the repo then and add a hint to the readme on how to enable workspaces when making changes to the worker?


Feature645:  Batch processing ( insert, update, and delete)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.B.Mady seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@balanka you still need to sign CLA. Click here https://cla-assistant.io/zio/zio-sql?pullRequest=663 scroll down and click on Sign in with Github to agree

Due to some trouble related to the signing of the CLA I need to close and reopen this PR


programsRouter should support retrieval and updating of ParticipantActivities completion status

Expected behaviour
There should be two programs router functions that return a participant activity's completed? status and allow for the value of that attribute to be modified.
Details and resources
This ticket is associated with #454 and associated FE ticket in order to fulfill the product spec requirement for a user to be able to mark an activity as complete.
The two programsRouter functions should be something along the lines of:

programsRouter.get('/activityStatus/:programId/:activity_id') - returns the completed? status of a given participantActivity. this function will allow a user to get their own status (admin access to be addressed later on)
programsRouter.post('/activityStatus/:program_id/:activity_id') - modifies the completed? value of an existing participantActivity (POST instead of PUT since we're creating new records instead of updating them)

in the POST body, there should be info about what the completed? value is being changed to (eg, {"completed?": true})
associated service function should only allow activities of type: assignment to have this attribute updated

The functions should take the following parameters: principal_id (corresponding to the user whose activity status is being tracked), program_id, and activity_id
__tests__/programsRouter.ts should also be updated to test these new router functions.
Checklist

[x] This issue is linked to the appropriate project.
[x] This issue has appropriate labels.

Questions to discuss:

[ ] are ParticipantActivities records created automatically for each user, or should updating the completion status create a new record, or something other flow?
[ ] does it matter whether this issue gets addressed before or after #454?

11/29 progress update: pseudocoded to plan code changes. will catch up on latest discussion in C4 channel tomorrow in order to further flesh out pseudocode and start translating it into code changes.

11/30 progress update: built out the getter function. tomorrow will continue with adding in necessary checks/error handling and following up with setter function

associated service function should only allow activities of type: assignment to have this attribute updated

per 12/1 discussion w Davey, this check should only be taken care of on the webapp side (see #428)

Hi there! I've merged in the latest changes from main to bring this branch up to date, and I tried to answer as many questions as I could. I love the communication and coordination with the other tickets, and you've got a strong start so far!
I've also added a stub for the route to get a list of all program activities and their completed statuses, which should help with the feature y'all were proposing at the end of last week.
Sorry about any confusion in the miscommunication about which functions would be throwing errors versus returning differently-shaped values.

As per our discussion this morning, I've removed the stubs I added before to restrict scope for the backend tickets. Valuable feedback, thanks!

12/6 progress update:

addressed questions about service function return values with Davey - see comment here. this informed our decision to keep L54 as is - !activityCompletionStatus - because the variable's value is a hash, not a boolean
built out setter function test
waiting on a PR for #454 tomorrow in order to finalize this ticket as well

12/7 progress update:

updated router functions and tests to reflect today's updates in service file


32 bootstrap not customized

What this PR does / why we need it:
Remove Bootstrap customisation to simplify the development of new features and to not be forced to add important! rules to CSS properties
Special notes for your reviewer:
Bootstrap button with custom classes overriding Bootstrap styles to look exactly as the JSF button:

Bootstrap button with SASS variables overrided:

The problem:

The thing is, since the JSF button has a different size than the Bootstrap button, the transformation property on the dropdown doesn't work well, so I need to edit the transformation to correctly show the dropdown menu.
This transformation property is added by Bootstrap js with the onlcick event, so is added as an inline styles, since inline styles have the greatest priority in CSS, no matter where I add a class to override this property. So I need to add an !important which is a bad practice.
Proposed solution:
Don't change the Bootstrap button size. Then no need to override styles that are added by Bootstrap javascript
In general, I think it would be easier for the development to don't override bootstrap classes with CSS classes on top, but rather overriding only the Bootstrap SASS variables. The problem with this is that the SPA frontend would look slightly different than the JSF frontend

ok, done! I removed the tertiary button and the navbar will look slightly different but now the code base it's simpler, we can always add css if once we start developing pages we think it doesn't look good. But fore the moment simpler is better
Should we merge this branch and the https://github.com/IQSS/dataverse-frontend/pull/42 branch into https://github.com/IQSS/dataverse-frontend/pull/41 ? @pdurbin

@MellyGray yes, please!


networking/swarm mode: Make gossip timeout configurable

Using docker master
Additional environment details (AWS, VirtualBox, physical, etc.):
Digital Ocean VMs:
3 Managers: 2 GB Memory / 40 GB Disk / Ubuntu 16.04 x64
3 Agents: 1 GB Memory / 30 GB Disk / Ubuntu 16.04 x64
Steps to reproduce the issue:

Just start a swarm cluster and schedule a few tasks

Describe the results you received:
Often, memberlist (gossip) will mark nodes as unhealthy even though they are healthy. libnetwork uses the default timeout value for the gossip which can be aggressive for some setups.
Error:
WARN[0442] 2016/07/12 17:19:52 [ERR] memberlist: Failed TCP fallback ping: read tcp X.X.X.X:54690->X.X.X.X:7946: i/o timeout

INFO[0448] 2016/07/12 17:19:58 [INFO] memberlist: Marking node06 as failed, suspect timeout reached

Describe the results you expected:
No error. Or at least very few false positives (which could happen in practice).
Additional information you deem important (e.g. issue happens only occasionally):
Making the timeout configurable should allow a user to configure this timeout according to their environment and get rid of those false positives.

@abronan Something weird is happening in this case. A TCP fallback ping failed with i/o timeout which is indicating some kind of underlying infrastructure issue or network congestion. We should never see TCP fallback ping failing unless there is severe problem in the infrastructure or the daemon is stuck in some weird state that it is not able to respond to the ping. We should debug it more to find the root cause of that.

@mrjana I don't think the error on TCP is relevant, sometimes nodes are marked as failed even without those TCP errors (I'm on Digital Ocean btw). I think it's definitely network congestion as this can happen without doing any specific action but is definitely triggered when orchestrating tasks at a large scale (I have to debug this more but I suspect the orchestrator to be a resource hog as it is also blocking raft heartbeats after a docker service scale but I'll have to pprof to identify the true cause).
Looking at the code on libnetwork, I see that the ProbeTimeout is at 500ms which is reasonable most of the time but when orchestrating a huge amount of tasks, this might crumble. And I'm not sure if the AwarenessMaxMultiplier is even used in this case.
Probe timeout should still be configurable and tunable I think as gossip largely depends on the Infrastructure and the default values may not be adapted. I'm catching up on libnetwork and looking into ways to tune the ProbeTimeout and ProbeInterval. If you also think it makes sense of course.

@tiborvass should we assign a priority to this one?

This issue hasn't happened in a long time - closing for now, will re-open if the problem occurs again


Desktop Voice Assistant

Description
This is a desktop voice assistant that will help you automate a lot of task on your device. Before you start the assistant its advised to read the readme file and the other text files to make the required changes.
Fixes #709
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes
[ ] No

Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Documentation Update

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] My documentation follows Template for README.md
[x] My changes generate no new warnings
[ ] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

I don't know why the lint code fails because I did run the code in my device.
Can the mentors please suggest?

I don't know it works fine in my device, can someone tell the underlying problem if possible

@sayantani11

@kaustubhgupta I ran it in jupyter lab, it was working fine.....Lemme see

@kaustubhgupta have a look, if any changes are required.....do inform

@Kushal997-das

@Kushal997-das @santushtisharma10 @antrikshmisri


Even stronger type for eventName in useEventListener.ts

Hi, so string might be too lose of a type for this, since we could put some random string as an event.
useEventListener("random", ()=>{}) didn't catch that "random" isn't an event listener type, so i updated it.

Hi @Nishit-Dua, thank you for the contribution!
WindowEventMap is very better than string :100:


DE-6413: Prep React SDK for release 0.6.0.

Here is my first batch of improvements, some more will follow. (here is the breakdown https://castlabs.atlassian.net/browse/DE-6413)
It could be good enough for release 0.6.0, but it needs a little bit more work before it's ready for 1.0.0.

Fixed and improved package.json
Fixed and improved rollup configs (especially the one that builds this library).
Made sure tests are passing.
Updated @castlabs/prestoplay from 6.2.6 to 6.2.7
Filled in missing changelog entries and polished the changelog.
Fixed some bugs, fixes related to the UI were:

Make sure spinner is visible.

Fix styling of play/pause button.

Add a margin around UI buttons.

I will squash before merge (3 commits - changelog, new fixes, rollup).

Moved to https://github.com/castlabs/prestoplay-react-components/pull/6


This SqlTransaction has completed; it is no longer usable.

InvalidOperationException: This SqlTransaction has completed; it is no longer usable.
System.Data.SqlClient.SqlTransaction.ZombieCheck()
System.Data.SqlClient.SqlTransaction.Commit()
YesSql.Core.Services.Session.Dispose()
YesSql.Core.Services.Store.<InitializeAsync>d__10.MoveNext()

Happens when the tables already exist during the setup

Is it desired behavior?  I wanted to call InitalizeAsync method in a Startup class of my project, but it can be called only when a db does not contains the tables.


Adding subtabs to item sheet details

This will add subtabs on the "Details" tab of item sheets to organize the features a bit better. Primarily aimed at cleaning up weapon sheets in preparation for adding new calculation process for NPC attack and damage bonuses.
Some progress screenshots added below:

This NPC-specific tab is a placeholder for now

This is the same as my previous PR, which I'll close. The previous PR wasn't based off dev and generally had some really messy, ugly git practices. Only one commit from the previous PR was actually new code, and it also had some extraneous code from a different feature that is in its own PR, https://github.com/foundryvtt-starfinder/foundryvtt-starfinder/pull/1101.
I've rebuilt the branch based freshly off the current development branch. This one is cleaner and should hopefully be simpler to review.

Some progress

I've completed

Ammunition
Armor
Armor Upgrades
Augmentations
Containers
Goods
Weapons

Still to Complete

Hybrid Items
Magic Items
Shields
Technological Items
Weapon Accessories
Weapon Fusions

Also need to review classes, themes, spells, starship parts, etc.

I've completed
* Ammunition

* Armor

* Armor Upgrades

* Augmentations

* Containers

* Goods

* Weapons

Still to Complete
* Hybrid Items

* Magic Items

* Shields

* Technological Items

* Weapon Accessories

* Weapon Fusions

Also need to review classes, themes, spells, starship parts, etc.

All equipment items sheets are done, on to reviewing the remaining item sheets for "breakages".

Archetype
Class
Species
Theme
Actor Resource
Feature
Spell
Ability Score Increase
Drone Chassis/Mod
Vehicle Attack
Vehicle System
All Starship Parts

Implementation of code from this PR also needs to be added.

[ ] https://github.com/foundryvtt-starfinder/foundryvtt-starfinder/pull/1046

Everything is done, all item sheets have been reviewed and updated to make their details tabs easier to read. Subtabs have been added where appropriate.
This also incorporates the changes made in the following PRs

https://github.com/foundryvtt-starfinder/foundryvtt-starfinder/pull/1046

This also partially resolves issue https://github.com/foundryvtt-starfinder/foundryvtt-starfinder/issues/1110 as it applies to vehicle damage.

Example of final styling


cannot be used with new Devise "reconfirmable" feature....

if I set :
config.reconfirmable = true   in Devise initializer...
then the email address is not filled , but rather is the 'unconfirmed_email'
this raises 2 bad behaviors :

the confirmation request email is sent by Devise  + the invitation email...
the flash notice doesn't have the user email address in the string ...

don't know if there is any way to have a correct behavior ..  ( I commented this feature in Devise initializer)
whatever I am using inevitable with Devise + subdomains ( subdomain admins are able to invite users in their subdomains...) it runs well ...

I have to read code about reconfirmable feature, it's a new feature for me


Matrices tex problems

I have been trying to insert variables into TexMobjects involving matrices like so:
class Test(Scene):
    def construct(self):
        a = 3
        matrix1 = TexMobject(r"\begin{pmatrix} ", a, r" \\ 4 \end{pmatrix}")
        self.add(matrix1)

but I get errors of the following type...

Any help much appreciated!

Use Matrix,  you can't break LaTeX environments in manim, just formulas
class Matriz(Scene):
    def construct(self):
        matrix1 = Matrix([[1],[2]])
        self.add(matrix1)

Thanks


[DNM] WIP: Prjxray stabilization test

This PR is to test all stabilization related fixes together on the GCS runner.
There are separate PRs for each of the commits.

All stabilization related PRs have been merged. This DNM PR can be closed now.


Array destruction

Current coverage is 98.49%

Merging #1 into master will increase coverage by +0.01% as of 115a46c

@@            master      #1   diff @@
======================================
  Files           68      69     +1
  Stmts          728     732     +4
  Branches       105     105
  Methods          0       0
======================================
+ Hit            717     721     +4
  Partial          5       5
  Missed           6       6

Review entire Coverage Diff as of 115a46c

Powered by Codecov. Updated on successful CI builds.


Add Greek Translation

Published: you should now see https://mlocati.github.io/timezone-converter in Greek (if your browser is configured to prefer the Greek language).
Thanks!


[Question] How to set the gpu id to use?

Hi, in nnunet, use CUDA_VISIBLE_DEVICES=1 nnUNet_train .... can set which gpu to use in the training process.
Have tried this in nnDetection, seems not working.
Currently is there a mechanism to set the preferred gpu id in a multi-gpu machine to use in the nnDetection?
Thanks!

Hi @ZheMin-1992 ,
sorry for the delay.
There is no explicit way of setting the GPU id because CUDA_VISIBLE_DEVICES=1 nndet_train ... works correctly on my workstation and in multiple cluster environments.
The Id could be changed here:
https://github.com/MIC-DKFZ/nnDetection/blob/a53f4fc72b3a20c8c32f688eb50522c16bd89d4d/scripts/train.py#L270
Best,
Michael


Docstring for NUTS has a wrong name for a named arg

https://github.com/TuringLang/Turing.jl/blob/dd5ce7eded5bd804377bd67cfb84307aeec0aea8/src/inference/hmc.jl#L337
Under the existing API, the last named arg ϵ, should actually be init_ϵ when given as a named arg (and for that specific constructor).
On the other hand, the actual field in the struct is called ϵ, but changing this around would break the existing API.
EDIT: an alternative solution that possibly (?) improves the API while not breaking it involves allowing that particular constructor to also understand ϵ as a named argument, in addition to init_ϵ

Fixed by https://github.com/TuringLang/Turing.jl/pull/1649


userEvent.keyboard doesn't release pressed key with playwright provider

Describe the bug
Interactive API doesn't seem to release keys across consecutive tests or reruns.
Related issue: https://github.com/vitest-dev/vitest/issues/6082 (fixed)
Reproduction
https://github.com/kwangure/vitest-matcher-rerun-repro
System Info
System:
    OS: Windows 10 10.0.19045
    CPU: (4) x64 Intel(R) Core(TM) i5-6300U CPU @ 2.40GHz
    Memory: 420.47 MB / 3.86 GB
  Binaries:
    Node: 20.16.0 - ~\.local\share\pnpm\node.EXE
    npm: 10.8.1 - ~\.local\share\pnpm\npm.CMD
    pnpm: 9.7.1 - ~\AppData\Local\Microsoft\WinGet\Packages\pnpm.pnpm_Microsoft.Winget.Source_8wekyb3d8bbwe\pnpm.EXE
  Browsers:
    Edge: Chromium (127.0.2651.74)
    Internet Explorer: 11.0.19041.4355
  npmPackages:
    @vitest/browser: ^2.1.2 => 2.1.2
    vite: ^5.4.8 => 5.4.8
    vitest: ^2.1.2 => 2.1.2

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I cannot reproduce following your reproduction steps. The test always succeeds.

I've pulled a clean copy of the linked reproduction and I'm still able to trigger the bug. It succeeds on the initial run. Then I click the green "Rerun current test". In the new run, Tab doesn't focus the input because Alt+Tab (Alt from the previous run) is being pressed instead.
I'm using latest Chromium (1140) that playwright installs.

Also able to reproduce with a 2 test setup as mentioned in the repro's readme:
This succeeds:
import { expect, test } from "vitest";
import { render } from "vitest-browser-svelte";
import { userEvent } from "@vitest/browser/context";

import Combobox from "./component.svelte";

test("1", async () => {
	// await userEvent.keyboard("{Alt>}"); // Not okay. I didn't release `{Alt>}` with `{/Alt}`
});

test("2", async () => {
	const screen = render(Combobox);

	await userEvent.keyboard("[Tab]"); // focus input

	await expect.element(screen.getByRole("combobox")).toHaveFocus();
});

Quit Vitest watch mode. Uncomment the keypress.
This fails:
import { expect, test } from "vitest";
import { render } from "vitest-browser-svelte";
import { userEvent } from "@vitest/browser/context";

import Combobox from "./component.svelte";

test("1", async () => {
	await userEvent.keyboard("{Alt>}"); // Not okay. I didn't release `{Alt>}` with `{/Alt}`
});

test("2", async () => {
	const screen = render(Combobox);

	await userEvent.keyboard("[Tab]"); // `Alt + Tab`

	await expect.element(screen.getByRole("combobox")).toHaveFocus(); // fails
});

I'm able to reproduce both scenarios in Chromium 1140 and  Firefox 1465, on both Windows and Linux (WSL).

I can confirm it fails on my PC. Checking keydown also shows alt is being pressed for re-run.
	document.addEventListener('keydown', (e) => {
		console.log([e.key, e.altKey])
	})

// initial run
stdout | src/component.test.ts > keydown repro
Tab false
stdout | src/component.test.ts > keydown repro
Alt true

// watch re-run
stdout | src/component.test.ts > keydown repro
Tab true
stdout | src/component.test.ts > keydown repro
Alt true


Always-available comment buttons in page editor

Discussed during design review with @benenright as a simple improvement to the commenting UX.

[ ] The "Add" and "view existing comments" buttons could always be available, triggering the "comments" side panel to reveal when clicked.
[ ] The "Add" button in rich text should also always be there
[ ] Remove extra spacing to the right of fields

Related: #8147

Once we implement this, we need to make sure to do this for the two implementations of comment buttons’ rendering, in Django Templates and JS.

Worth noting this should only be available with WAGTAILADMIN_COMMENTS_ENABLED = True

Would it be possible for me to attempt resolving this issue?

Implemented via #8345.


Inventory script usage unclear

Description
The documentation for the inventory script is unclear to me.
I installed ionos cloud via ansible galaxy. Creating/Updating servers is working fine. But I get confused how I could use the inventory.
If I run ansible-playbook -i ~/.ansible/collections/ansible_collections/ionoscloudsdk/ionoscloud/plugins/inventory/inventory.py test.yml
I get
[WARNING]:  * Failed to parse /home/kfritsche/.ansible/collections/ansible_collections/ionoscloudsdk/ionoscloud/plugins/inventory/inventory.py with script plugin: problem running /home/kfritsche/.ansible/collections/ansible_collections/ionoscloudsdk/ionoscloud/plugins/inventory/inventory.py --list ([Errno 13] Permission denied:
'/home/kfritsche/.ansible/collections/ansible_collections/ionoscloudsdk/ionoscloud/plugins/inventory/inventory.py')

Added +x to the inventory script results in:
[WARNING]:  * Failed to parse /home/kfritsche/.ansible/collections/ansible_collections/ionoscloudsdk/ionoscloud/plugins/inventory/inventory.py with script plugin: Inventory script (/home/kfritsche/.ansible/collections/ansible_collections/ionoscloudsdk/ionoscloud/plugins/inventory/inventory.py) had an execution error: Traceback (most recent call last):
File "/home/kfritsche/.ansible/collections/ansible_collections/ionoscloudsdk/ionoscloud/plugins/inventory/inventory.py", line 100, in <module>     import six ImportError: No module named six

pip install six tells me its already installed.
In theory I should be able to use the inventory as a plugin.
So I created a ionos.yml file with this content:
plugin: ionoscloudsdk.ionoscloud.inventory
server_name_as_inventory_hostname: true

And then using ansible-playbook -i inventories/ionos.yml test.yml
This seems to work better as I can even revert the +x on the inventory.py but results in:
usage: ansible-playbook [-h] [--list] [--host HOST] [--datacenters] [--fwrules] [--images] [--lans] [--locations] [--nics] [--servers] [--volumes] [--refresh]
ansible-playbook: error: unrecognized arguments: inventories/ionos.yml hub-test-start.yml

Even it it says ansible-playbook the returend message comes from the inventory script. So ansible is calling it, but the script returns this message and no inventory.
So I'm not sure how to use the inventory.py. Any help would be appreciated.
Expected behavior
Either when calling the inventory.py directly or using the better way as a inventory plugin should work to retrieve the dynamic inventory.
Environment
Ansible version:
ansible 2.9.6
  config file = /etc/ansible/ansible.cfg
  configured module search path = ['/home/kfritsche/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python3/dist-packages/ansible
  executable location = /usr/bin/ansible
  python version = 3.8.5 (default, May 27 2021, 13:30:53) [GCC 9.3.0]

Module version:
5.0.2

OS:
Linux 5.4.0-77-generic Ubuntu focal

Configuration Files
test.yml
- hosts:
    - drp
  tasks:
    - debug:
        var: ansible_host

Does the example of the README in https://github.com/ionos-cloud/module-ansible/tree/master/plugins/inventory work?

I found this back then, but it doesn't describe how to use it with ansible-playbook. In combination with ansible-playbook I get the results describe above and didn't get it running.

Regarding the plugin approach which returned w/ unrecognized arguments I thinks it's due to the default behavior of argparse.
To be used in this context it, parsing should be done by ArgumentParser.parse_known_args() instead of ArgumentParser.parse_args().
Regarding the six module error I have no clue. Do you have only one Python installation?


Questions about SDD dataset

Hello Nachiket,
I am working on trajectory forcasting modeling and I am impressed with your model. I have questions about the SDD dataset. The raw SDD data is available on the website, but I am not sure that my short trem data processing is the same way as yours. The frames are sampled at FPS = 2.5 and for each pedstrains in each videos, I intercept the first 8s (3.2s observation and 4.8s prediction) of each pedstrain trajectories.  As shown in the code below. I want to know if this process is the same as yours.
Thank you for your time, and I look forward to hearing from you.
--Lue Li
`
import numpy as np
import pathlib
import pickle
from tqdm import tqdm
import random
labels_object = {'"Pedestrian"': 0,'"Skater"': 1,'"Biker"': 2,'"Car"': 3,'"Cart"': 3,'"Bus"': 3}
SDD_RAW_DIR = pathlib.Path('../datasets/SDD')
def get_annotation(data_dir):
global labels_object
print(data_dir)
with open(data_dir, 'r') as f_a:
    lines = []
    for line in f_a:
        line = line.strip().split(" ")
        line = [int(line[5]), int(line[0]), (float(line[2]) + float(line[4])) / 2.0,
                (float(line[1]) + float(line[3])) / 2.0, labels_object[line[9]]]
        # frame object_id x y type
        if (line[4]== 0):
            lines.append(np.asarray(line))
lines = np.stack(lines)
return lines

def process_data(file_dir,setname):
scene_dir = file_dir / 'label.jpg'
data_a = get_annotation(file_dir/'annotations.txt', scene_dir)
all_obj = np.unique(data_a[:, 1])
all_data = None
for obj in all_obj:
    obj_allframe = data_a[(data_a[:,1]==obj), :]
    mask = np.arange(0, obj_allframe.shape[0],12)
    obj_sampleframe = obj_allframe[mask,:]
    if (obj_sampleframe.shape[0] < 20):
        continue
    index = 0
    if (all_data is None):
        all_data = obj_sampleframe[index: index+20,:]
    else:
        all_data = np.concatenate((all_data, obj_sampleframe[index: index+20,:]))
human_num = all_data.shape[0]
print("human_num = {}".format(human_num))
path = '../trajnet_process_onlyped/' +setname + '/stanford/' + str(file_dir.parents[0].name) + '_' + str(file_dir.name)[-1] + '.txt'
np.savetxt(path,all_data,fmt='%.01f')
return human_num

if name == 'main':
  train_set = {'bookstore': [0, 1, 2, 3], 'coupa': [3], 'deathCircle': [0, 1, 2, 3, 4], 'gates': [0,1,3,4,5,6,7,8],
             'hyang': [4, 5, 6, 7, 9], 'little': [], 'nexus': [0, 1, 2, 3, 4, 7, 8, 9], 'quad': []}
test_set = {'bookstore': [], 'coupa': [0, 1], 'deathCircle': [], 'gates': [2],
            'hyang': [0, 1, 3, 8], 'little': [0, 1, 2, 3], 'nexus': [5, 6], 'quad': [0, 1, 2, 3]}
# validation_set = {'bookstore': [4, 5, 6], 'coupa': [2], 'deathCircle': [], 'gates': [0],
#                   'hyang': [2, 10, 11, 12, 13, 14], 'little': [], 'nexus': [10, 11], 'quad': []}
data_setsplit = [train_set, test_set]
set_dir = {}
set_name = {0: 'train', 1: 'test'}
for i, setdir in enumerate(data_setsplit):
    for key, value in setdir.items():
        if value == []:
            continue
        for idx in value:
            video_num = 'video' + str(idx)
            set_dir.setdefault(set_name[i], []).append(SDD_RAW_DIR / key / video_num)
print(set_dir)
for setname, setdir in set_dir.items():
    total_human = 0
    for file_dir in tqdm(setdir, desc='Process ' + setname):
        print(file_dir)
        human_num = process_data(file_dir,setname)
        total_human += human_num
    print('{} human num = {}'.format(setname,total_human))

`

Thank you for your interest in the work.  We use all tracks in the SDD raw files, including skateboarders, bicyclists and golf carts.
Also, since several tracks are longer than 8 sec, they yield several 8 sec snippets that serve as multiple train and test instances.
The train/val/test sets correspond to different scenes in SDD, listed in table 1 of the paper.
Hope this helps!

Thank you very much for your reply!

Hello Nachiket,
Thank you for your wonderful work and open-source code! I want to comment on this question because I am struggling to find SDD data split provided by trajnet, which seems to be expired. Based on the discussion here, may I confirm the data files in https://github.com/nachiket92/P2T/tree/main/data/sdd correspond to the data split of Table I, which is essentially the trajnet split for SDD? And if we want to use the split used in PECNet, we have to process the data provided by PECNet github repo, is that true?
Best,
Zhe Huang


Error with vulcanized core and paper elements js file

In relation to this Stackoverflow question:
http://stackoverflow.com/questions/26629641/polymer-vulcanize-core-elements-and-paper-elements-to-one-file
With the latest Polymer (0.5.1) the vulcanized files do not work anymore using the same command.
I get a "Uncaught Element name could not be inferred." error from the generated js file after vulcanizing

Closing this issue due to age and the release of version 1 of Polymer - please feel free to re-open if this is incorrect.


[networkx] remove experimental tag of nx module

What do these changes do?

rename macro definition EXPERIMENTAL_ON to NETWORKX in engine side and make it default ON.
remove experimental tag of nx module
add install folly dependency in install_dependencies.sh

Related issue number
#62

Codecov Report

Merging #273 (d5a8138) into main (72e79dd) will decrease coverage by 19.47%.
The diff coverage is 50.00%.

@@             Coverage Diff             @@
##             main     #273       +/-   ##
===========================================
- Coverage   77.89%   58.42%   -19.48%
===========================================
  Files          57       57
  Lines        5294     5287        -7
===========================================
- Hits         4124     3089     -1035
- Misses       1170     2198     +1028

Impacted Files
Coverage Δ

python/graphscope/__init__.py
79.16% <ø> (ø)

python/graphscope/framework/app.py
70.12% <ø> (-17.69%)
:arrow_down:

python/graphscope/framework/graph_builder.py
62.50% <0.00%> (-30.00%)
:arrow_down:

python/graphscope/framework/graph.py
79.12% <100.00%> (-13.11%)
:arrow_down:

python/graphscope/analytical/udf/patch.py
3.47% <0.00%> (-96.53%)
:arrow_down:

python/graphscope/analytical/udf/wrapper.py
25.80% <0.00%> (-74.20%)
:arrow_down:

python/graphscope/analytical/udf/decorators.py
21.25% <0.00%> (-73.75%)
:arrow_down:

python/graphscope/analytical/udf/compile.py
25.41% <0.00%> (-65.09%)
:arrow_down:

python/graphscope/deploy/hosts/cluster.py
32.25% <0.00%> (-54.84%)
:arrow_down:

python/graphscope/deploy/hosts/utils.py
50.00% <0.00%> (-50.00%)
:arrow_down:

... and 22 more

Continue to review full report at Codecov.

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


Incompatible with ipywidgets >=8

I cannot run these instructions:

OK: git clone https://github.com/johnomernik/qgrid
OK: cd qgrid
OK: python setup.py install
NOPE OK: pip install qgrid2-1.1.3-py3-none-any.whl

The problem is that python setup.py install does not (and couldn't ever) build any wheel package,
but creates a dist/qgrid-1.3.1-py3.10.egg "link" file.
If i try to build a wheel out of this repo:
python setup.py bdist_wheel

...i get a dist/qgrid-1.3.1-py2.py3-none-any.whl file, but not qgrid2-... that is given in the instructions.
And if i install this wheel, the following notebook-cell fails to render a grid:
import pandas as pd
import numpy as np
import qgrid

df=pd.DataFrame(np.random.rand(200, 30), columns=[str(i) for i in range(30)])
qgrid.show_grid(df)

...rendering instead this error output:

...which when i click it, it reveals:
[Open Browser Console for more detailed log - Double click to close this message]
Failed to load model class 'QgridModel' from module 'qgrid'
Error: No version of module qgrid is registered
    at f.loadClass (http://localhost:8888/lab/extensions/@jupyter-widgets/jupyterlab-manager/static/134.40eaa5b8e976096d50b2.js?v=40eaa5b8e976096d50b2:1:74856)
    at f.loadModelClass (http://localhost:8888/lab/extensions/@jupyter-widgets/jupyterlab-manager/static/150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:10729)
    at f._make_model (http://localhost:8888/lab/extensions/@jupyter-widgets/jupyterlab-manager/static/150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:7517)
    at f.new_model (http://localhost:8888/lab/extensions/@jupyter-widgets/jupyterlab-manager/static/150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:5137)
    at f.handle_comm_open (http://localhost:8888/lab/extensions/@jupyter-widgets/jupyterlab-manager/static/150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:3894)
    at _handleCommOpen (http://localhost:8888/lab/extensions/@jupyter-widgets/jupyterlab-manager/static/134.40eaa5b8e976096d50b2.js?v=40eaa5b8e976096d50b2:1:73393)
    at b._handleCommOpen (http://localhost:8888/static/lab/jlab_core.3e11c5239452d671150c.js?v=3e11c5239452d671150c:2:1037311)
    at async b._handleMessage (http://localhost:8888/static/lab/jlab_core.3e11c5239452d671150c.js?v=3e11c5239452d671150c:2:1039301)

...and the browser-console is this:
150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1 Failed to fetch ipywidgets through the "jupyter.widget.control" comm channel, fallback to fetching individual model state. Reason: Control comm was closed too early
_loadFromKernel @ 150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1
3134.40eaa5b8e976096d50b2.js?v=40eaa5b8e976096d50b2:1 Error: widget model not found
    at f.get_model (150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:3461)
    at w.renderModel (134.40eaa5b8e976096d50b2.js?v=40eaa5b8e976096d50b2:1:72019)
renderModel @ 134.40eaa5b8e976096d50b2.js?v=40eaa5b8e976096d50b2:1
150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1 Error: No version of module qgrid is registered
    at f.loadClass (134.40eaa5b8e976096d50b2.js?v=40eaa5b8e976096d50b2:1:74856)
    at f.loadModelClass (150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:10729)
    at f._make_model (150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:7517)
    at f.new_model (150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:5137)
    at f.handle_comm_open (150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:3894)
    at _handleCommOpen (134.40eaa5b8e976096d50b2.js?v=40eaa5b8e976096d50b2:1:73393)
    at b._handleCommOpen (default.js:1095:19)
    at async b._handleMessage (default.js:1262:43)
loadModelClass @ 150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1
DevTools failed to load source map: Could not load content for chrome-extension://cfhdojbkjhnklbpkdaibdccddilifddb/browser-polyfill.js.map: System error: net::ERR_FILE_NOT_FOUND
DevTools failed to load source map: Could not load content for chrome-extension://pgjjikdiikihdfpoppgaidccahalehjh/webspeed.js.map: System error: net::ERR_BLOCKED_BY_CLIENT

What am i missing?

Are you behind a proxy? I see a net error…
On Mon, Mar 20, 2023 at 7:39 AM Kostis Anagnostopoulos <
@.***> wrote:

I cannot run these instructions:
OK: git clone https://github.com/johnomernik/qgrid
OK: cd qgrid
OK: python setup.py install
NOPE: pip install qgrid2-1.1.3-py3-none-any.whlcd ..
The problem is that python setup.py install does not (and couldn't ever)
build any wheel package,
but creates a dist/qgrid-1.3.1-py3.10.egg "link" file.
If i try to build a wheel out of this repo:
python setup.py bdist_wheel
...i get a dist/qgrid-1.3.1-py2.py3-none-any.whl file, but not qgrid2-...
that is given in the instructions.
And if i install this wheel, the following notebook-cell fails to render a
grid:
import pandas as pdimport numpy as npimport qgrid
df=pd.DataFrame(np.random.rand(200, 30), columns=[str(i) for i in range(30)])qgrid.show_grid(df)
...rendering instead this error output:
[image: image]
https://user-images.githubusercontent.com/501585/226341105-4c075941-2b14-48cc-b52b-b72a82645114.png
...which when i click it, it reveals:
[Open Browser Console for more detailed log - Double click to close this message]Failed to load model class 'QgridModel' from module 'qgrid'
Error: No version of module qgrid is registered
at f.loadClass @./jupyterlab-manager/static/134.40eaa5b8e976096d50b2.js?v=40eaa5b8e976096d50b2:1:74856)
at f.loadModelClass @./jupyterlab-manager/static/150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:10729)
at f._make_model @./jupyterlab-manager/static/150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:7517)
at f.new_model @./jupyterlab-manager/static/150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:5137)
at f.handle_comm_open @./jupyterlab-manager/static/150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:3894)
at _handleCommOpen @./jupyterlab-manager/static/134.40eaa5b8e976096d50b2.js?v=40eaa5b8e976096d50b2:1:73393)
at b._handleCommOpen (http://localhost:8888/static/lab/jlab_core.3e11c5239452d671150c.js?v=3e11c5239452d671150c:2:1037311)
at async b._handleMessage (http://localhost:8888/static/lab/jlab_core.3e11c5239452d671150c.js?v=3e11c5239452d671150c:2:1039301)
...and the browser-console is this:
150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1 Failed to fetch ipywidgets through the "jupyter.widget.control" comm channel, fallback to fetching individual model state. Reason: Control comm was closed too early_loadFromKernel @ 150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:13134.40eaa5b8e976096d50b2.js?v=40eaa5b8e976096d50b2:1 Error: widget model not found
at f.get_model (150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:3461)
at w.renderModel (134.40eaa5b8e976096d50b2.js?v=40eaa5b8e976096d50b2:1:72019)renderModel @ 134.40eaa5b8e976096d50b2.js?v=40eaa5b8e976096d50b2:1150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1 Error: No version of module qgrid is registered
at f.loadClass (134.40eaa5b8e976096d50b2.js?v=40eaa5b8e976096d50b2:1:74856)
at f.loadModelClass (150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:10729)
at f._make_model (150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:7517)
at f.new_model (150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:5137)
at f.handle_comm_open (150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:3894)
at _handleCommOpen (134.40eaa5b8e976096d50b2.js?v=40eaa5b8e976096d50b2:1:73393)
at b._handleCommOpen (default.js:1095:19)
at async b._handleMessage (default.js:1262:43)loadModelClass @ 150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1
DevTools failed to load source map: Could not load content for chrome-extension://cfhdojbkjhnklbpkdaibdccddilifddb/browser-polyfill.js.map: System error: net::ERR_FILE_NOT_FOUND
DevTools failed to load source map: Could not load content for chrome-extension://pgjjikdiikihdfpoppgaidccahalehjh/webspeed.js.map: System error: net::ERR_BLOCKED_BY_CLIENT
What am i missing?
—
Reply to this email directly, view it on GitHub
https://github.com/JohnOmernik/qgrid/issues/9, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOXGUOXLDFSB7H3AK7XUKLW5BFYFANCNFSM6AAAAAAWBA4XNI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Sent from my iThing

Yes, i'm behind a proxy.
Additionally, i discovered a big mistake, and apologies for the title of this issue, actually the steps run quite well
(but unfortunately cannot make them work).
You  see, i had run at some point python setup clean and it had removed the pre-build wheel - i restored it and after following the instructions, the `qgrid.shod_ simply didn't work.
I will list here what i used:
python3.10 -m venv --prompt qgrid ../venv-qgrid
. ~/Work/venv-qgrid/bin/activate
pip install jupyterlab  ipywidgets pandas
cd  qgrid-dev
python setup.py install
pip install qgrid2-1.1.3-py3-none-any.whl
cd ..

And then in the cells above i get exactly the same error as before:
[Open Browser Console for more detailed log - Double click to close this message]
Failed to load model class 'QgridModel' from module 'qgrid'
Error: No version of module qgrid is registered
    at f.loadClass (http://localhost:8888/lab/extensions/@jupyter-widgets/jupyterlab-manager/static/134.40eaa5b8e976096d50b2.js?v=40eaa5b8e976096d50b2:1:74856)
    at f.loadModelClass (http://localhost:8888/lab/extensions/@jupyter-widgets/jupyterlab-manager/static/150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:10729)
    at f._make_model (http://localhost:8888/lab/extensions/@jupyter-widgets/jupyterlab-manager/static/150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:7517)
    at f.new_model (http://localhost:8888/lab/extensions/@jupyter-widgets/jupyterlab-manager/static/150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:5137)
    at f.handle_comm_open (http://localhost:8888/lab/extensions/@jupyter-widgets/jupyterlab-manager/static/150.b0e841b75317744a7595.js?v=b0e841b75317744a7595:1:3894)
    at _handleCommOpen (http://localhost:8888/lab/extensions/@jupyter-widgets/jupyterlab-manager/static/134.40eaa5b8e976096d50b2.js?v=40eaa5b8e976096d50b2:1:73393)
    at b._handleCommOpen (http://localhost:8888/static/lab/jlab_core.48a41b637653e18e773e.js?v=48a41b637653e18e773e:2:1039756)
    at async b._handleMessage (http://localhost:8888/static/lab/jlab_core.48a41b637653e18e773e.js?v=48a41b637653e18e773e:2:1041746)

I think i nailed it!
First, i suggest to retrofit slightly the installation instructions to build a wheel out ofthe sources,
and use it whenever it is needed (instead of installing from the sources, which i more brittle eg if the git-repo is removed.
I played with the versions of the various core jupyter packages and discovered that qgrid fails to work when ipywidgets is 8+.  To reproduce, these are my (linux0 commands:
cd qgrid-dev
/usr/bin/python3.10 -m venv --prompt qgrid venv && source venv/bin/activate
pip install wheel
python setup.py bdist_wheel  # build a wheel inside `dist/`
pip install  pandas  jupyterlab  'ipywidgets<8'  dist/qgrid-1.3.1-py2.py3-none-any.whl  qgrid2-1.1.3-py3-none-any.whl

I will post a PR to fix the dependencies of this project.

Sure, a rewrite is very welcomed.
Particularly now that the new notebook-v7 API is coming out of alphas (i don't know if it would also affect this widget)
Till then a PR with just this dependency capped will save many wasted days from other unlucky programmers on the planet.
I included also shrink-wrapped installation instructions, directly from Github :-)

btw, i notice that it is not a git-fork from the original repo, the history of commits has been clipped.
is that on purpose?

Yes, because I am not a developer, just a guy who wanted to use qgrid on a
newer version of jupyter lab.  I honestly have no idea how any of this
works and am actually away from my dev box for 2 weeks. Sorry if my
responses are delayed
On Tue, Mar 21, 2023 at 8:38 AM Kostis Anagnostopoulos <
@.***> wrote:

btw, i notice that it is not a git-fork from the original repo, the
history of commits has been clipped.
is that on purpose?
—
Reply to this email directly, view it on GitHub
https://github.com/JohnOmernik/qgrid/issues/9#issuecomment-1477854907,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOXGUKZONPFB4GZZHE2GRLW5GVO3ANCNFSM6AAAAAAWBA4XNI
.
You are receiving this because you commented.Message ID:
@.***>

--
Sent from my iThing

No problem. I am not sure the best approach approach here as I won’t be
able to
Merge this for a while (sorry, just not at a place where I can do it)
I know that my hack to make it work is a bit off, I was trying to find a
way to build it it in such a way that didn’t require building on everyone’s
machine (ie not needing node.is). I wonder hat would be the best way to to
garner interest I this. I think the bones of qgrid are real good. I like
the ability to edit and pull the edited data frame off, I like resizing of
columns, etc . Just need to figure out how to simplify and get it future
proof
On Tue, Mar 21, 2023 at 8:46 AM Kostis Anagnostopoulos <
@.***> wrote:

No problem.
I like tidy git histories, i can help you fix this, if you want
btw, PR#10 served!
—
Reply to this email directly, view it on GitHub
https://github.com/JohnOmernik/qgrid/issues/9#issuecomment-1477866301,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOXGUNKK3QGI67ZKDM4JNDW5GWJVANCNFSM6AAAAAAWBA4XNI
.
You are receiving this because you commented.Message ID:
@.***>

--
Sent from my iThing

Is the intention of this particular issue to fix the compatibility with ipywidgets>=8, or rather accept the fact and accept the fix in #10?

The latter, accept the fact that it dosn't work with ipywidgets>=8.

@ankostis did you look into what the incompatibility is? I might be interested to spend some time and try to fix it, at least in case it doesn't require huge amounts of work.

No

But maybe you can take some ideas from a respective conda package (if indeed it works).

Highly recommend people check out https://github.com/zhihanyue/qgridnext


hurry up

plz

Wanna help?

LOL.

Is there any way I can help? Although I have to admit, I'm not great with C, I have some bison, lex, Nd yacc experiance.

@christopherdumas Look at the code, figure out how it works, figure out how to improve it, and send pull requests.

Don't take matz away from ruby!
Also, this issue is not exactly useful FredGoo. ;)

@shevegen  lol
just for fun

see https://github.com/matz/streem/issues/113


Upgrade docker

Docker > 1.11.0 would be great to fix some issues.

Fixed by 22e863ba05f1d8ef36bbba363a69826ff222e43c and new images built with new docker.


Make theme configurable

resolves #12

@dontlaugh
https://github.com/hatoo/Accepted/blob/5dec79fdb817982a760d638220a9eaf328209e40/assets/default_config.toml#L1-L9
You can set the theme in config file now.
Does it solves your issue?

I will test this soon. Thank you.

My background is still opaque, even when making the colors match, but it's still a good change :+1: At least it lets me experiment with the colors without rebuilding.


Question about SHPs calculation

Hi,
I am trying to understand the SHPs calculation implementation.
I have a simple question about SHP_SelPoint.m
It seems that you take the SHPs from a pixel only if these are connected to the pixel coordinate within the search window.
(lines 64-65)
% connection component
LL = bwlabel(SeedPoint);
SHP.PixelInd(:,num)=LL(:)==LL(InitRow,InitCol);
I wonder, why don't take all the pixels within the search window who pass the statistical test as SHPs, regardless if these are not connected spatially to the pixel?
Thanks a lot for your answer,
Erik

Hello, I think the reason for doing this is because in the real world, adjacent pixels have more similarity in physical representation, and in SAR images, because the resolution is mostly a few meters or more, non adjacent pixels may have sudden changes in physical representation. This is my own understanding. If there is anything inappropriate, please forgive me


Postgres provisioning fixes

For support of Postgres 15 where the CREATE permission isn't automatically granted on the 'public' schema.
Adds a new optional 'schema' parameter for postgres connection URLs (ie. postgres://user:pw@host/db?schema=public). When not set, the schema now defaults to the username when creating new tables.
For new stores,

If the database is newly created, then the database owner is set explicitly
A new schema is created in the database, with the same name as the user. This is consistent with the default search path: ('$user', public)
All tables are created in the specified schema

For existing stores,

If the schema isn't specified, then all schemas in the search path are checked for the config table (not just 'public')
When the schema is set explicitly in the connection URL, the search path is restricted just to that schema

Fixes #129 ?

#178 should fix the test failure on Windows and it might be best to merge that PR first.


Flake8cfg

@gillins
I just wanted to flag with you that I had to make some changes in the parallel_examples do_prepare.py code, as flake8 was complaining a lot about the indentation. I separated out the containerOverrides parts, so that I could indent that separately. The way you had it was perfectly clear, but flake8 couldn't cope, so this seemed the best compromise.

Thanks - all good with me!


Remove custom queue package

Signed-off-by: Gaius<EMAIL_ADDRESS>
Description

Remove custom queue package.

Related Issue
#274

Motivation and Context
The library is not referenced in the project.

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist:

[x] Code compiles correctly.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Codecov Report

Merging #275 (4d98c4a) into main (accfef0) will decrease coverage by 1.93%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #275      +/-   ##
==========================================
- Coverage   44.83%   42.89%   -1.94%
==========================================
  Files          54       52       -2
  Lines        4262     4119     -143
==========================================
- Hits         1911     1767     -144
- Misses       2113     2115       +2
+ Partials      238      237       -1

Flag
Coverage Δ

unittests
42.89% <ø> (-1.94%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/util/net/iputils/ip_utils.go
61.22% <0.00%> (-4.09%)
:arrow_down:

client/daemon/peer/peertask_base.go
32.92% <0.00%> (-0.74%)
:arrow_down:


Not all async methods in ForceClient are implemented asynchronously

Multiple methods in the ForceClient are not implemented asynchronously despite having asynchronous method names.
For example, in ForceClient.cs, there is a method declared as:
public Task<QueryResult<T>> QueryAsync<T>(string query)
QueryAsync implies that it queries asynchronously however it is currently done synchronously. Many methods in that file suffer the same issue.
The correct definition should look like:
 public async Task<QueryResult<T>> QueryAsync<T>(string query)

@KE7 I think you're getting a bit mixed up here.
The async keyword is only required when a method contains the await in the method body. A method that returns a Task<T> or Task can still be asynchronous.


Use ModuleScanners

The idea is simple, provide a way to find all modules in a given source; be it a directory or an array of class names. These scanners would return an array with all (valid) module names.

Done!


feat: configuration file is no longer mandatory

I removed the need to setup a configuration file in the repo. If there is no file, the default configuration is loaded, as it was in previous versions.

@rodripf hello, very good idea, thank you!


Try It Out Image is cut off

Using the attached image in try it out, the image is cut off. I think we're passing the full thing to training. We should display the full image

The full image:

do't we have image size restriction?

Validated.


Bug: Sentence grammer is not correct

In the second point of individual users, it should be "every positive effort" and not "everyone"
GSSoC'23

Hello gitssz, thanks for opening a issue, your contribution is valuable to us. The maintainers will review this issue and provide feedback as soon as possible.


Hybrid work vs in-office

[Can we possibly add something about avoiding the use of in-office, office work, etc., references? I've gotten CELA pushback about referencing and it would be great to have something in the guide about it.]

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

ID: 1c4ab3f5-d7de-1853-6f11-16e761b9698e
Version Independent ID: 3901f044-2b3e-5f86-79b8-0cce3382c49c
Content: Welcome - Microsoft Style Guide
Content Source: styleguide/welcome/index.md
Product: non-product-specific
GitHub Login: @pallep
Microsoft Alias: pallep

Hi Brittany,
I am the Editor for the Microsoft Writing Style Guide. Thank you for providing feedback on this.
Could you please clarify your request? I do not see any references to "in-office" work in the page you linked to. Are you suggesting that such references should be removed globally? If so, could you please explain the rationale for this request so I can bring it to the attention of the relevant parties?

I work for a contractor, and we manage social channels for Microsoft
products. Channel managers have told me more than once that we're steering
away from using "in-office" in reference to working in person. However, we
don't have anything in the style guide that I can point to as justification
for why I'm removing it from copy. So I was mostly curious if this is
coming from CELA or simply stakeholder preference.
On Fri, Oct 14, 2022 at 2:13 PM S-LRaillard @.***>
wrote:

Hi Brittany,
I am the Editor for the Microsoft Writing Style Guide. Thank you for
providing feedback on this.
Could you please clarify your request? I do not see any references to
"in-office" work in the page you linked to. Are you suggesting that such
references should be removed globally? If so, could you please explain the
rationale for this request so I can bring it to the attention of the
relevant parties?
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/microsoft-style-guide/issues/342#issuecomment-1279474156,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZIHY7ZD6JP7N6OYKZTZQKTWDHEILANCNFSM57XIBVTA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi @brittanyshands, sorry for the long delay. I investigated with CELA and there is no guidance coming from them about this issue, so I imagine it is stakeholder preference. Thanks for raising the issue!


Shim Layer to support multiple Spark versions

fixes #355
This adds a shim layer to support multiple Spark versions.  This PR adds the framework and support for Apache Spark 3.0 and Apache Spark 3.1.  All the tests pass for Spark 3.0 but there are a few failures still for 3.1. We can finish resolving those after this is merged.  The problem I keep running into is upmerged to spark 3.1 as well as the plugin itself keeps having conflicts and requires continuous retesting and fixing those. So if we can get this part reviewed and in, I think it will be easier to finish the remaining things and allow multiple people to look at it.
Note that to shim the ShuffleManger, it requires the user to specify the spark package version right now.
A lot of the changes are around Join. HashJoin had changes which requires a lot of things to be copied into the shim layer. There may be ways it improve this to share code but would like to look more at that in followup, if you have ideas let me know.  If you diff those files there should be very little differences. The other issue there is BuildSide, BuildRight, BuildLeft all moved packages.
Spark 3.1 had api changes for other things - TimeSub, ScalaUDF, MapOutputTracker, ShuffleManager, FileSourceScan, First and Last
For the Shimlayer itself we use service loaders for each of the versions. There is a lightweight loader class that first determines if the loader applies to that Spark version, then it has a buildShim function to load the entire shim. This keeps us from loading a bunch of classes we don't really need.
I added a profile spark31tests that you can specify to run the tests on Spark 3.1.  Once we have things working we can hook that up to CI along with running integration tests on both versions.  slf4j in the tests had conflicts with the cudf versions, so for now I made them pull in the newer version. I want to update CUDF to have the same version as well.
Note that I tested the core sql-plugin code by building against both Spark 3.0 and Spark 3.1 to ensure I didn't miss anything there. Tests were run against both versions. 1 unit test failure on 3.1 and 3.1 has some integration test failures - some FullOuter join tests, log test, and TimeSub tests,

build

build

I think we need to have a process in place for removing supported versions. I am concerned about code bloat and a lot of copy/paste with this solution.  I personally cannot think of a better solution at this point, but we should spend some time to have a rule about what versions/how many we do support so that we can keep the copy/paste to a minimum.

I'll look at at spark3.0.1 quick

Also note, to be clear, there is nothing in this that prevents us from having one shim layer depend on another one. For instance if spark301 depends on spark30, you could have Spark301Shims inherit from Spark30Shims. As long as you don't get into circular dependency. It doesn't do that currently and I didn't try to create common modules.  I should have been more clear on that in my previous response.
I'm looking at spark 3.0.1 now to see if there are other complications there.

build

build

build

I added support for spark 3.0.1 and this shows how you can inherit from other versions of Shim layers.  it allows you to reuse code.
https://github.com/NVIDIA/spark-rapids/compare/branch-0.2...tgravescs:shimBranch0.2-spark301?expand=1#diff-07d276bf63961b60ca0251a61c029aabR26
I did also add a bit to reuse execs/exprs, but I'm not sure on that as it may get ugly quickly - just for reference you can see that here: https://github.com/NVIDIA/spark-rapids/compare/branch-0.2...tgravescs:shimBranch0.2-spark301?expand=1#diff-77438fb2622c8043fee4391edcbfdf56R118
Those are just built on top of this PR.

With the above approach it makes more sense to use spark300 and spark310.  Let me know if you have any high level thoughts on that approach and if that seems ok I can do that and put up a different PR to just include 3.0.1

I'm going to add in 3.0.1 support and do some renaming of things, so closing this one and will open another one.


Cannot remove cape

Describe the bug
i've been attempting to remove my cape but it just keeps sticking to me, i cannot wear most back bling because it keeps putting the cape back on
To Reproduce
Steps to reproduce the behavior:

press customise cosmetics
put on back bling
go to capes
remove cape

Expected behavior
i had expected that the cape would be gone and id see my back bling
Screenshots
n/a
Setup (please complete the following information):

OS: windows 10
Minecraft Version: firefox?
Version: 1.18.2

Additional context
even the invisibble capes hide my back bling

Send a screenshot of the cape you're stuck wearing as well as your username. You can always disable your cape in skin customization settings.

Send a screenshot of the cape you're stuck wearing as well as your username. You can always disable your cape in skin customization settings.

i see. it was a me problem. sorry about that. im not smart


docs(material/theming): add section for overrides api to theming.md

Documentation Feedback
Add a section to theming.md to describe the 3 new override APIs. The section should describe:

The benefits of using these APIs.
How to use them.
What use case each is best suited for.

This information is already outlined in go/angular:mat-theming-overrides-api.
Affected documentation page
https://material.angular.io/guide/theming

I urgently need the usage documentation for "override APIs".

Closing since the docs site starting in v19 will have documentation for all the different componenent's override functions. This is an preview for button: https://next.material.angular.io/components/button/styling


Add easy way to transform nodes using typescript compiler

Basically, wrap the ts.transform function.

I'm not going to do this for the time being. The ts.transform function does not seem to change the text of the source file (unless I'm missing something).


[TOPIC-BLE-LLCP] Initial demo implementation of LLCP system framework

I havn't fixed the line width problems

@cvinayak  Merged with DNM label?


Recreate token cache if token refresh fails

@dermesser This fixes my problem, by recreating the token cache if a token refresh fails in any way. What do you think to that general approach?
Fixes https://github.com/dermesser/yup-oauth2/issues/163

Oh I'm very sorry - this slipped through my inbox twice, somehow :(
Looking at the code, it seems reasonable to request a completely new access token on this failure, just as if no refresh token were found. In fact, the existing comment already mentions your failure mode.

It seems that we need to restructure the test_refresh test a bit to deal with the updated flow.

I merged this PR manually. Thank you for the effort, and apologies again for letting you hang for so long.

@dermesser wrote:

I merged this PR manually. Thank you for the effort, and apologies again for letting you hang for so long.

No problem and thanks for the merge. FYI my project which was depending on yup-oauth2 is now archived.

I hope that at least other projects will benefit from your change, then!


Exception The process cannot access the file 'C:\tools\VSTSSyncMigration\export while Attachment migration for Bugs

I am getting the following error continuously for 4 times now which doing a WIT migration. There is one attachment where i get the following error which is really strange. As of now i have removed export and import and run the migration again. Any pointers will help.
vstssyncmigrator.exe Information: 0 :  Query Complete: found 1 work items in 233ms
777 of 1227 - Import https+----xxxx.visualstudio.com----teamproject--1380xx#No UI on UI confirmation.mp4 to 119x
Web method running: [https://target.visualstudio.com/_apis/connectionData?connectOptions=1&lastChangeId=-1&lastChangeId64=-1] (GET)connectionData[connectionData]
Web method response: [https://target.visualstudio.com/_apis/connectionData?connectOptions=1&lastChangeId=-1&lastChangeId64=-1] (GET)connectionData[connectionData] 918 ms
Web method running: [https://target.visualstudio.com/_apis/wit/attachments?uploadType=chunked] (POST)attachments[wit]
Web method response: [https://target.visualstudio.com/_apis/wit/attachments?uploadType=chunked] (POST)attachments[wit] 183 ms
Web method running: [https://target.visualstudio.com/_apis/wit/attachments/1ff0e575-82c2-4372-9f5f-a8a0a9ab3294?uploadType=chunked] (PUT)1ff0e575-82c2-4372-9f5f-a8a0a9ab3294[wit]
Web method response: [https://target.visualstudio.com/_apis/wit/attachments/1ff0e575-82c2-4372-9f5f-a8a0a9ab3294?uploadType=chunked] (PUT)1ff0e575-82c2-4372-9f5f-a8a0a9ab3294[wit] 2916 ms
Web method running: [https://target.visualstudio.com/_apis/wit/attachments/1ff0e575-82c2-4372-9f5f-a8a0a9ab3294?uploadType=chunked] (PUT)1ff0e575-82c2-4372-9f5f-a8a0a9ab3294[wit]
Web method response: [https://target.visualstudio.com/_apis/wit/attachments/1ff0e575-82c2-4372-9f5f-a8a0a9ab3294?uploadType=chunked] (PUT)1ff0e575-82c2-4372-9f5f-a8a0a9ab3294[wit] 329 ms
vstssyncmigrator.exe Warning: 0 :   [EXCEPTION] The process cannot access the file 'C:\tools\VSTSSyncMigration\export\https+----xxx.visualstudio.com----teamproject--1380xx#No UI on UI confirmation.mp4' because it is being used by another process.

Is this happening on the export or the import?
Do you know how big that file is?

I see the file is present in export folder and has been attached to workitem each time the migration is run. The file is 21648 KB. I also tried deleting the file from source and then re-ran the migration. But still it adds the mp4 file again on migration, don't know how.
SessionID: d859c19a-6c46-469b-a0f9-9e4a9374ebb9

So there are two "processors" one for exporting the attachemtns, and the other for importing them. The Export saves the files in a format that has the work item ID in it. The Import just iterates through the images folder and uploads them, deleting them after they are complete.
I can see the error message, and I am not sure what the issue might be...

Could it be AV on such a big file?

Have you had any luck?

Hi @MrHinsh having same issue here, this seems to work for several attachments but suddenly this error.
here is the log :
VstsSyncMigrator.exe Information: 0 :  Query Complete: found 1 work items in 74ms  219 of 772 - Import https+----myvsts.visualstudio.com--My Project--_workitems--edit--3252#my file name.mp4 to 11325 Web method running: [https://myvsts.visualstudio.com/_apis/connectionData?connectOptions=1&lastChangeId=30031905&lastChangeId64=30031905] (GET)connectionData[connectionData] Web method response: [https://myvsts.visualstudio.com/_apis/connectionData?connectOptions=1&lastChangeId=30031905&lastChangeId64=30031905] (GET)connectionData[connectionData] 201 ms Web method running: [https://myvsts.visualstudio.com/_apis/wit/attachments?uploadType=chunked] (POST)attachments[wit] Web method response: [https://myvsts.visualstudio.com/_apis/wit/attachments?uploadType=chunked] (POST)attachments[wit] 107 ms Web method running: [https://myvsts.visualstudio.com/_apis/wit/attachments/cc10e711-867a-4b59-b5f1-5c9300996134?uploadType=chunked] (PUT)cc10e711-867a-4b59-b5f1-5c9300996134[wit] Web method response: [https://myvsts.visualstudio.com/_apis/wit/attachments/cc10e711-867a-4b59-b5f1-5c9300996134?uploadType=chunked] (PUT)cc10e711-867a-4b59-b5f1-5c9300996134[wit] 7511 ms Web method running: [https://myvsts.visualstudio.com/_apis/wit/attachments/cc10e711-867a-4b59-b5f1-5c9300996134?uploadType=chunked] (PUT)cc10e711-867a-4b59-b5f1-5c9300996134[wit] Web method response: [https://myvsts.visualstudio.com/_apis/wit/attachments/cc10e711-867a-4b59-b5f1-5c9300996134?uploadType=chunked] (PUT)cc10e711-867a-4b59-b5f1-5c9300996134[wit] 6747 ms VstsSyncMigrator.exe Warning: 0 :   [EXCEPTION] The process cannot access the file 'C:\tools\VstsSyncMigrator\export\my file name.mp4' because it is being used by another process. MigrationEngine: The Processor AttachementImportMigrationContext entered the failed state...stopping run

If you can, please reproduce this again using the latest version, which might have more information in the exception.
And I agree with @MrHinsh, it might be AV locking the file

I also got this with a slow computer and added a bunch of checks to basically ignore the delete action failing as it does not prevent us moving on


Currently import from Redis Enterprise to Premium tiers is not supported

Currently import from Redis Enterprise to Premium tiers is not supported.

@flang-msft
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


make setSize check for local vs. non-local assignment

The makes the R version of setSize determine whether it should make local or non-local assignment in the calling frame.
Fixes #724

This passed tests on travis but failed on appveyor, so I suspect an appveyor glitch.  I will restart it.


Indented codeblocks throw off all future codeblocks

rubocop.yml
require:
  - rubocop-md

AllCops:
  TargetRubyVersion: 3.2
  NewCops: disable

test.md since I still don't know how to properly escape markdown here.
I'll try to check this out myself if that's fine with you @palkan, but in case I don't manage like last time I'll let you know. That good?

I'll try to check this out myself if that's fine with you @palkan, but in case I don't manage like last time I'll let you know. That good?

Yeah, sounds good. Feel free to reach out / submit a draft PR if you encounter any struggles

What would you consider a solution here? Since we lint MD files by "converting" them into Ruby files, the Layout/InitialIndentation is an expected behaviour (though annoying, for sure). Probably, we can un-indent code blocks during pre-processing and restore them back in the end; the question is how to distinguish between valid (within lists) and  invalid indentation.

Is this extension supposed to care about the indentation of codeblocks themselves in the first place? If I put my codeblock 10 spaces to the right then all I really care about is that the code inside is properly indented relative to that. Anything else feels out of scope, to me at least.
One thing I feel that would be a big help here is by using rubocops 3rd party template support that got added in 1.45. https://github.com/rubocop/rubocop/pull/10839
It allows extensions like this one to process non-rb files and preprocess them appropriately. Basically what is already happening now but "officially sanctioned". Autocomplete like this would be easier to implement I believe, without the comment workaround going on right now.
Would you be open to exploring that? It is pretty involved for this issue and probably more work that necessary. Just want to get your feelings on this, perhaps for something in the future.

using rubocops 3rd party template support that got added in 1.45.

Oh, haven't seen this; awesome! Yeah, we should migrate to this new API.

Great to hear! I think I'll try to start with that as I feel it will make fixing this as a second step easier. I feel like nobody has complainted about this issue yet so you can take a bit of time for this.

Coming here as I tested this to link our 2k+ markdown doc files, and see the same issue.
@palkan Re: Probably, we can un-indent code blocks during pre-processing and restore them back in the end; the question is how to distinguish between valid (within lists) and invalid indentation.
My suggestion would be to count the number of spaces before the start of the code block, and trim that amount until the end of the code block. So:
List item:

- ```ruby
  valid, trim 2 characters from every line.
  ```

- Nest some more:

  - ```ruby
    valid, trim 4 from every line
    ```

  - ```ruby
      invalid indentation, trim 4 from every line, but this line still ends up with 2 extra spaces
    ```

After trimming:
List item:

```ruby
valid, trim 2 characters from every line.
```

- Nest some more:

```ruby
valid, trim 4 from every line
```

```ruby
  invalid indentation, trim 4 from every line, but this line still ends up with 2 extra spaces
```


Support for podman/podman-compose as an alternative to docker/docker-compose

Is your feature request related to a problem? Please describe.
Currently, OpenSearch testing has a hard dependency on Docker and Docker-compose. This makes the development on non-Linux platforms more complicated and introduces a barrier for new contributors.
Describe the solution you'd like
A Podman and Podman-compose are open-source alternatives and they had matured a lot. It would be great if developers had the option to choose between docker and podman.
Describe alternatives you've considered
My primary development environment is MacOS (M1) and I am not sure if there are "non-hackich" alternatives (for example Minikube).
Additional context
I was recently experimenting with podman and podman-compose:
# You can install podman directly from brew of as part of https://podman-desktop.io/
% podman version
Client:       Podman Engine
Version:      4.4.2
API Version:  4.4.2
Go Version:   go1.20.1
Git Commit:   74afe26887f814d1c39925a1624851ef3590e79c
Built:        Thu Feb 23 16:59:21 2023
OS/Arch:      darwin/arm64

Server:       Podman Engine
Version:      4.4.2
API Version:  4.4.2
Go Version:   go1.19.6
Built:        Wed Mar  1 12:22:39 2023
OS/Arch:      linux/arm64

# You can install podman-compose using pip3
% podman-compose version
['podman', '--version', '']
using podman version: 4.4.2
podman-composer version  1.0.3
podman --version
podman version 4.4.2
exit code: 0

And I have to say that I was able to run many example applications from both the https://github.com/docker/awesome-compose and https://github.com/containers/podman-compose/tree/devel/examples.
The experience seem to be really smooth these days. Podman-compose uses Qemu VMs and CLI commands podman and podman-compose work really effortlessly. You create new podman machine instance (which is typically a fresh fedora), do some basic settings (like installing podman-mac-helper, export DOCKER_HOST). Then you navigate to docker-compose examples folder, fire podman-compose up and that's it...
So far so good. But when I was trying to convince OpenSearch to use podman-compose instead of docker-compose I realized that the main issue is that it rely on gradle-docker-compose-plugin which is not compatible with podman-compose. See https://github.com/avast/gradle-docker-compose-plugin/issues/387
To me it seems that missing gradle plugin for podman is the last bit that is holding us back. Perhaps the most logical way would be to fork the docker plugin and adapt it for podman (but I am not saying this is on my near roadmap, although a prototyping should not be that difficult).
I would like to hear from others, WDYT?

What's the issue you're running into using Docker on M1?

As far as I understand it is available only as a part of paid subscription (at least that would be my case)
I would like to use Podman (and I believe there will be more people going forward)

+1 on podman because of licensing

+1 on additional options other than docker for we macOS folks!

@lukas-vlcek looks like we have agreement on podman, would you like to contribute the changes?

Yes, I would like to give it a try. It is on my radar.


Test pass for a folder but fail for all

What Ruby, Rails and RSpec versions are you using?
Ruby version: 3.1.2
Rails version: 7
RSpec version:
RSpec 3.11
  - rspec-core 3.11.0
  - rspec-expectations 3.11.0
  - rspec-mocks 3.11.1
  - rspec-rails 6.0.0.rc1
  - rspec-support 3.11.0

Observed behaviour

Several times I have experienced this issue that there is no consistency between running tests individually, in folder and as a whole. Before I had case of test passes when running all tests but fail if be run on individual tests. I don't have reference for that but now I have this case to present.
When we run rspec or rspec spec we have 2 failure (issue was maching dynamic date with hardcoded ones so not related to other parts) but running rspec /spec/requests or running the exact test file, test could pass without issue.

Expected behaviour

We should have consistency over the tests. In this case, it was good that we had failure when running rspec but before it was opposite which was very bad for CI/CD.
Can you provide an example app?

I can share the code in two or three weeks or if needed, I can share the code privately till then.

I can share the code in two or three weeks or if needed

Please come back with a reproduction repository when you can share it.
Before that, I can suggest you to rspec bisect.

This looks like a pretty normal test interdependency issue where you have some form of shared state which isn't being handled properly so it leaks between tests, in particular its probably a leaking database entry which is causing both the last record to be unexpected, and presumedly preventing an entry from being saved. As RSpec doesn't handle these (its usually something like database cleaner or Rails transactional test helpers) its not something we can fix on our end.
I'm closing this for now but feel free to re open if you find something in RSpec itself that could be improved.

MHi
My initial thought was the same but what was causeing the test to fail was a simple mistake in checking date. That should fail in all death of testing.
I can share whole spec folder if it can help. I just dont know how to do it. A temporary repo? Zip file?
در ۱۷ ژوئیهٔ ۲۰۲۲ ۱۱:۲۹:۴۱ (GMT+03:00)، Jon Rowe @.***> نوشت:

This looks like a pretty normal test interdependency issue where you have some form of shared state which isn't being handled properly so it leaks between tests, in particular its probably a leaking database entry which is causing both the last record to be unexpected, and presumedly preventing an entry from being saved. As RSpec doesn't handle these (its usually something like database cleaner or Rails transactional test helpers) its not something we can fix on our end.
I'm closing this for now but feel free to re open if you find something in RSpec itself that could be improved.
--
Reply to this email directly or view it on GitHub:
https://github.com/rspec/rspec-rails/issues/2613#issuecomment-1186444643
You are receiving this because you authored the thread.
Message ID: @.***>

Your best option is to run rspec --bisect to find the minimalist failing set of specs, then reduce those further until you narrow down the source of the interdependant failure. Likely that will be in your codebase, if its not please gist a small snippet or use our issue template to produce the issue in a clean rails app, we don't have the bandwidth to debug your project for you I'm afraid.


Do not assume avatar needs be changed from gravatar.com

Do not assume avatar needs be changed from gravatar.com
Always send user to settings screen to change avatar.
Drops "change_custom_avatar" localized message, keeps "change_avatar"
for the generic one.

Looks good to me
@Unknwon thoughts?

Thanks!


packages on linux don't show icon in desktop entry

Haven't seen an issue opened on this yet, so here you go.

Hello @citrusui
It is a known issue I'm planning to tackle quick.
You're one of the first to try the .deb packaging and your feedback and patience is valuable, so thanks!

Docs: https://developer.gnome.org/integration-guide/stable/desktop-files.html.en

@xseignard  Will be fixed tomorrow in the electron-complete-builder (and our projects will be merged next week —https://github.com/loopline-systems/electron-builder/issues/138#issuecomment-183715299).
48x48 PNG icon is required, but we will automatically extract it from OS X icns file (if not explicitly provided).