Unable to override Slick slide width and height, and the margins, on resize

I am trying to use the amazing Slick carousel for a simple slider, but am having difficulties with Slick's responsiveness, since I cannot 'turn this off'/modify this manually.
The slider should be full width with square slides inside. The square dimensions are based on the screen width or height, depending which of these is the smallest. I have written the JS to calculate these dimensions and size the slides as well as center them using margins and it works perfectly fine.
However, as soon as I add Slick, the dimensions of the slides (or their margins) get fixed after page load. In the case I use default settings the left and right margins get fixed after page load, so that only the slide itself resizes on a window resize. In the case I use the variableWidth property set to "true", the margins are variable, but the 'content' width gets fixed.
Please see the pen I made to illustrate this:
http://codepen.io/anon/pen/LVwobo
Here I have used the default Slick settings. I have replaced the images which I normally use with blue divs to simplify the example. On each page refresh/load, each of the blue divs is a square which is positioned in the center due to its margins, all calculated and added via the JS. Its dimensions are also calculated by JS.
However, as soon as I resize the page, the divs lose their square shape. Even though the function to size them to a square is called on resize, the dimensions are overwritten by Slick's JS. If I use the variableWidth option, it is the same but with the margins.
Is there a way to 'disable' the resizing done by Slick? I know this is a handy feature in view of its responsiveness, but in this specific case it is not working for me. My function makes sure the slides are always 100% viewport width and also adjusts the slider height if needed, so Slick should then have no issue just 'sliding' the slides without first resizing them, if I am correct?

No IDEA to fix that ?
i've the "same pbm" with flexbox div (who is the slider-parent).
The only answser i've is to fix margin to the slide but i wonder if the simple way to work
my slide are sticked each other also


SSCS-5054: Fix Appellant subscription updated for old email/sms

The appellant does not get a subscription updated message to their old emails address.

Can we put an integration test in to cover this scenario please?

Closing this PR, as I've re-written this.


design_paper and short_paper: Some edits

Design_paper:
Rewrote the section Design Requirements -> Payment Example.
It's a far reaching rewrite, in draft state, pulled for inspection and to keep it up to date.
Short_paper:
Added the section Design Requirements -> Delivery side.

@colourful-land please rebase!!!


Need delete porn content

We found a few sites that also have to be removed from your database (adult materials):
http://httparchive.org/viewsite.php?pageid=4438119
http://httparchive.org/viewsite.php?pageid=15062825
http://httparchive.org/viewsite.php?pageid=15007643
http://httparchive.org/viewsite.php?pageid=7004854
http://httparchive.org/viewsite.php?pageid=66769548
http://httparchive.org/viewsite.php?pageid=34552743
http://httparchive.org/viewsite.php?pageid=32177293
http://httparchive.org/viewsite.php?pageid=66464202
Thanks!

These sites have been flagged as adult content so we no longer show their filmstrips in HTTP Archive.

Display an insufficient measure flag. From the pages you can make the transition to some of the materials sites. For example the table of requests - where the reference to the prohibited materials, as well as on the Filmstrip, Video - which also can be seen prohibited content.It is necessary to exclude domains of Databases!!!Requests were sent to the domain registrant (AHNames), archive.org, icann.orgarchive.org - are already excluded from the Databases; Domains list: sadomax.com bizarre-tube.com myfuck.tv bondagezilla.com my18teens.com myteenvideo.com pornmage.com xxxshemaleporn.comThank you for your cooperation
At 6 Aug 2016 16:15:20 +0000 (+00:00) from Steve Souders<EMAIL_ADDRESS>These sites have been flagged as adult content so we no longer show their filmstrips in HTTP Archive.
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.


unixpb: Add Xcode11 installation tasks to Unix playbook

[x] commit message has one of the standard prefixes
[ ] faq.md updated if appropriate
[ ] other documentation is changed or added (if applicable)
[ ] playbook changes run through VPC or QPC (if you have access)
[ ] VPC/QPC not applicable for this PR
[ ] for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

ref https://github.com/adoptium/infrastructure/issues/2536
Xcode11.7 is required on m1 mac machines in order to cross compile x64 jdk8
Our usual method of installing xcode using xcversion is no longer supported https://github.com/xcpretty/xcode-install/blob/master/MIGRATION.md and is causing errors when I use it to install xcode11.7, see https://github.com/adoptium/infrastructure/issues/2536#issuecomment-1733623397
xcodes is another tool used to install different versions of xcode but I think it is dependent on Xcode13 which I believe requires macOS 11.3+ (our M1 machines are 11.0)

The transfer process takes around 40mins. The binary is 7.6G.
And this may be due to timeout reasons, but after the transfer is complete ansible hangs. It hung for about 30mins before I interrupted the execution. I'll test to see if AWX shows the same behaviour

Awx run https://awx2.adoptopenjdk.net/#/jobs/playbook/1910?job_search=page_size:20;order_by:-finished;not__launch_type:sync

Tested this pr on test-macstadium-macos11-arm64-1. Works as expected

Running from AWX, the transfer takes the same time as running it from my machine, around 40mins. I dont think this is too bad as we are not seeing timeout errors

I've rolled out xcode11.7 onto the 4 arm64 mac machines

Running from AWX, the transfer takes the same time as running it from my machine, around 40mins. I dont think this is too bad as we are not seeing timeout errors

Yeah that's fine sine it's a one off and won't happen on subsequent runs of the playbook :-)

I'm going to go ahead and push this as it's blocking my Orka review


fix: disable REACT_DEVTOOLS_GLOBAL_HOOK in production mode

SUMMARY
try to fix https://github.com/apache/superset/issues/20815
refer: https://stackoverflow.com/questions/42196819/disable-hide-download-the-react-devtools
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
BEFORE

AFTER

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[x] Has associated issue: #20815
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Seems like it has been fixed.
Closed.


compose: Add stream privacy decorations in banner box.

This PR is for issue #19878
Fixes part (1/2) -
Add stream privacy decorations in banner box.
Part (2/2) in another PR as suggested by @alya in Zulip chat https://chat.zulip.org/#narrow/stream/9-issues/topic/missing.20hash.20before.20stream.20name/near/1261461 -
Add stream privacy decorations in compose box.
Testing plan:
Lint test.
Node test.
GIFs or screenshots:

Changes explained:
Earlier sent message notification - "Sent! Your message is outside current narrow. Narrow to test > blue"
Updated sent message notification - "Sent! Your message is outside current narrow. Narrow to #test > blue"

Hm, that hash looks overly large to me.
Could you also:

Add screenshots for a private stream and a world public stream?
Make sure your PR passes checks.

Thanks!

Hm, that hash looks overly large to me.
Could you also:

Add screenshots for a private stream and a world public stream?
Make sure your PR passes checks.

Thanks!

Yes, I will adjust the hash size and also add screenshots for mentioned cases.
I will make sure that my PR passes checks as well.
Thanks for reviewing my PR.

Hm, that hash looks overly large to me.
Could you also:

Add screenshots for a private stream and a world public stream?
Make sure your PR passes checks.

Thanks!

Fix vertical alignment and size of stream privacy decoration icons in banner box.
Screenshots-
Public stream -

Private stream -

Web Public stream-

Thanks for working on this @srdeotarse! I have a review regarding the commits.

Also you might want to clean up your commit history, check out - https://zulip.readthedocs.io/en/latest/git/fixing-commits.html.

Thanks! It looks to me like there's not enough spacing to the left of the lock and globe icons -- there should be as much space as you normally have between words.
For the hash, can we try having less space between the # and the name of the stream?

Thanks! It looks to me like there's not enough spacing to the left of the lock and globe icons -- there should be as much space as you normally have between words.
For the hash, can we try having less space between the # and the name of the stream?

Adjusted spacing to the left of globe and lock icons in banner box. Also, reduced spacing between # and name of stream.

@srdeotarse  Could you add updated screenshots here? Thanks!

@alya  Adding the updated screenshots -
Added spacing to the left of lock icon -

Added spacing to the left of globe icon -

Reduced spacing between hash and stream name -

@raghupalash I have cleaned my commit history and changed commit messages according to commit guidelines.

I think you'd want to squash all the commits in one single commit (as they all belong to a single change). This might help https://zulip.readthedocs.io/en/latest/git/fixing-commits.html#fixing-the-last-commit. Feel free to bump a thread on the development-help stream if you can't figure it out.

@amanagr perhaps you could review and/or finish up this PR as well?

I removed the underline on link hover since it was not playing nicely with the icons. It just has an increased opacity effect on hover now.

@amanagr I have changed the author of the last two commits to me.

@srdeotarse did you review the changes I made? Do they look good to you?

Thanks for updating this!
I'm not sure about the globe and lock, but the font on # looks odd to me. Could we make it less bold to match the rest of the text?

Seeing the icons added, it now feels odd not to have the appropriate stream privacy icon in a couple of other places in the compose box area:

The "Message..." button

The "Message..." placeholder text

Would it be possible to make these be based on the stream privacy setting as well?

@srdeotarse , will you have time to address the remaining feedback on this PR, or should @amanagr take over from here?

Let's just complete the banner icon changes in this PR.


[table] multisort not working with advanced filters

What package is this for?

[x] React
[ ] Angular

Describe the bug
The multisort doesn't work when hasAdvancedFilter is true
To Reproduce
Steps to reproduce the behavior:

Go to https://next.carbon-addons-iot-react.com/?path=/story/1-watson-iot-table--playground
Click on knobs tab "Sort & filter"
Enable isSortable, hasMultiSort, and hasAdvancedFilter
Click the overflow button for the Select column and select "Multi-sort"
Click the "Sort" button in the dialog
See that the result is empty, which is incorrect

Please create a reduced test case in CodeSandbox:
https://codesandbox.io/s/github/carbon-design-system/carbon-addons-iot-react/tree/next/packages/react/examples/codesandbox

Expected behavior
Result should be sorted rows instead of empty
Environment/versions:

OS: [e.g. MacOS, Windows]
Browser: [e.g. chrome, safari]
carbon-addons-iot-react version: [e.g. v2.60.0]

Additional context

Specific timeline issues / requests
Do you want this work within a specific time period? Is it related to an
upcoming release?
NB: The core contributors will try to work with your timeline, but it's not
guaranteed. The earlier you make a request in advance of a desired delivery
date, the better!

If this turns out to be a "bug" in the story configuration then we can lower the severity


Add License

There doesn't seem to be a license included with this. So it's not clear what users can and cannot do with the code.

It took a while, but I have added a license. Thanks!


Is there anyway to get nearby drivers and lat lng as an address in flutter

In my taxi app, I need to get the nearby drivers to customer and, retrieving lat lng as an address from firestore.
Thanks

Questions like these are better suited for Q&A sites like StackOverflow or on one of the flutter community sites at https://flutter.dev/community


fix error initialising flow tab editor

fixes #3577
Types of changes

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes
fix error initialising flow tab editor as described here: #3577

Coverage increased (+0.04%) to 68.5% when pulling bf8bfa582a4aa2a7ac4d1351b9867a7c4ce9a9eb on Steve-Mcl:fix-flow-prop-tab into 55ecc7a92c264ed0613222c40742005333941057 on node-red:dev.


Ro crate data model API

This PR will bundle the work for closing #433
We decided to merge this before full completion to unblock other PRs, will continue on another PR

continuation in #478


remove the end year in copyright notice

Remove the end year of the copyright notice.
The start year of most copyright notices was 2015, but only the files in hal were 2016.
Keep it this way, or unify the start year to 2015 ?

The year of termination of the copyright notices are unified to 2015, including hal related files.


GitHub link

The link to GitHub now opens in an external browser in Electron. Will need to manually update GitHub counts.

Also, I have no clue how releases got in there.


[cc2538] fix radio disabled for entropy get

RFCORE_XREG_FRMCTRL0 is not restored after generating random.

This fixes #3859

LGTM :+1:
Thanks for fixing this! I wonder if we should include a change from #3861.

@KaSroka ,
maybe not appropriate to raise here however I have some questions for #3818 [entropy] allow for entropy source sharing which causes the issue this PR tries to fix.
Not sure about other platforms yet, at least cc2538 itself could generate pseudo random number, why we unify all the platforms to use the same Random::NonCrypto in #3818 ?
If we don't want to use the pseudo random number, maybe we don't need to do anything in cc2538RandomInit() which mainly prepares for pseudo random number generation?
https://github.com/openthread/openthread/blob/master/examples/platforms/cc2538/entropy.c#L72

@librasungirl There were few ideas standing behind these changes. First, it's easier for the platform to provide only the entropy source. Many platforms defined their own pseudo random generator or used rand() (cc2538 was the only one using something else). Also from a developer point of view, it's easier to distinguish between NonCrypto and Crypto random and decide whether to use them. This would help to avoid the usage of noncryptographic random numbers when cryptographic numbers are needed (at least that's what I hope).
Regarding the cc2538 I have no idea what's this init is doing and if it's connected with RNG or not - that's why I left it here. If it's really not needed then we can remove it.


Provide code and documentation that generates ThirdEye Docker image

Generate Dockerfile and ancillary code that generates a ThirdEye (TE) demo Docker image for easy and reproducible evaluation, development, and testing.
This can help prospective TE community members evaluate the project and contribute.
This containerization can later be extended to wrap context around the running of TE by connecting to various data sources, running in different compute environments such as in k8s, or other user-specified purposes.

Sounds like a great idea!


[Bug Report]:

Describe the Bug
PDF exported from our site has problems with images (check images).
p.s Since our wiki is for internal usage I can't share the link, sorry.
Steps to Reproduce

Go to<EMAIL_ADDRESS>Click Export
Select PDF
Check downloaded PDF or picture below

Expected Behaviour

Screenshots or Additional Context
PDF:

Exact BookStack Version
v21.12.2
PHP Version
No response
Hosting Environment
CentOS 8

Hi @Kosstrov,

Does this occur for images that are uploaded fresh via BookStack's image manager?
If you look at the source code for the page content being exported (Code button to the right of the editor toolbar in page editor) are these images (<img src="<link>"/> parts)  linked to your BookStack instance URL or an external URL?

Thank you @ssddanbrown !
Everything working fine ... however pics are not aligned in PDF:

Just readd all pics and it helped, thank you.


Feature: Add Git Undo Command

Hmm, yeah, something as simple as this should be fine. Add it also to the readme and I can merge that

I changed it to use git reset --mixed HEAD~ which I think is a good compromise. I might add --soft and --hard as options later if I find it useful.

Hmm, thinking about it, I think "undo" is a bit too generic of a name. Maybe something like "undoLastCommit"?

lgtm!


Telemetry: update network/interfaces to a list of dict

This is to workaround a limitation of ES.

Done in e475f33993e7b7ffbe370082fe9bc34030accd54


Bug: typeorm-extension: seed:run Not Executing Specific Seed File (Version 3.5.0)

Versions

Node: v18.18.2
OS: Ubuntu

Reproduction

I'm encountering an issue with typeorm-extension version 3.5.0 where the seed:run command doesn't seem to be executing the specified seed file, despite providing the correct path.
Additional Details

Steps to reproduce
"seed:specific": ts-node ./node_modules/typeorm-extension/bin/cli.cjs seed:run --dataSource typeorm-data-source.ts --name src/seeders/user.seeder.ts

this is my pacakge.json script cmd which i run for specific seed file
What is Expected?
The command should execute the seed logic defined in user.seeder.ts after successfully locating and loading the seed file.
What is actually happening?

<EMAIL_ADDRESS>seed:specific
ts-node ./node_modules/typeorm-extension/bin/cli.cjs seed:run --dataSource typeorm-data-source.ts --name src/seeders/user.seeder.ts
DataSource Directory: /home/anas/Documents/kwanso/boss-economics/backend                                                                                > DataSource Name: typeorm-data-source.ts
Seed Name: src/seeders/user.seeder.ts

The response only displays the data source name, directory, and file name, and doesn't actually trigger the execution of the seed logic within user.seeder.ts.
@tada5hi
@guillaumedeslandes
@ckaeslin
@faloi
@fossamagna

@anassarfraz can you provide a reproduction (git) repository ?

@anassarfraz can you provide a reproduction (git) repository ?

yes sure i will create and share repository

@tada5hi
this code just related to problem i remove extra code from this repository.
https://github.com/anassarfraz/typeorm-extension-issue
just try to run npm run seed:specific for run the seed for specific file i mentioned in package.json file.
actually it runs the commands in cli but not save the data in database.
but if i run npm run seed:run it will save the data in database.
Thanks for your response.

You have to adjust the paths for seeds, migrations & factories in your typeorm-data-source.ts file:
After the adjustment everything works fine :)
const options: DataSourceOptions & SeederOptions = {
  type: 'postgres', // IF you are using a different type of connector, please mention here explicitly.
  database: process.env.DATABASE_NAME || '',
  host: process.env.DATABASE_HOST || 'localhost',
  port: parseInt(process.env.DATABASE_PORT) || 5432,
  username: process.env.DATABASE_USERNAME,
  password: process.env.DATABASE_PASSWORD,
  migrationsRun: true,
  entities: ['src/**/**/*.entity.{ts,js}'],
  migrations: ['src/migrations/*{.ts,.js}'],
  seeds: ['src/seeders/*.seeder{.ts,.js}'],
};

@anassarfraz please close the issue if this solved your issue.

i will check in a while then if resolved for me i will closed it.
thanks

your welcome!

@tada5hi
did you try to run code which i gave to you (https://github.com/anassarfraz/typeorm-extension-issue)
my typeorm-data-source.ts setting are fine and its paths for seed and migrations are fine , i am using swc compiler for compailing nestjs code.
my path for seeders and migrations are corrects thats why my code create table in migration and run all seeds on seed:run i just want to ask run seeder only for specific file not working ..

@tada5hi did you try to run code which i gave to you (https://github.com/anassarfraz/typeorm-extension-issue)
yes, without modifications, the run seed:specific does not work.
my typeorm-data-source.ts setting are fine and its paths for seed and migrations are fine , i am using swc compiler for compailing nestjs code.
When i modified them, everything worked :man_shrugging_tone1:
my path for seeders and migrations are corrects thats why my code create table in migration and run all seeds on seed:run i just want to ask run seeder only for specific file not working ..
The reason for this is, that none of the seeds detected by the pattern 'dist/seeders/*.seeder{.ts,.js}' does match with the path src/seeders/user.seeder.ts


[CIVP-18337] send info-level logs to stdout

This PR fixes a bug that sent info-level CIVIS_PLATFORM_BACKEND logs to stderr rather than stdout.

Gonna try closing and reopening this branch in order to trigger a Travis build.

Closing in favor of #43, where I address the issues discussed above.


Update Minio and mc version

Add separate headless service for distributed mode

Fixes https://github.com/minio/minio/issues/7327

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[ ] Variables are documented in the README.md

cc @wlan0

/ok-to-test

/lgtm


threshold  wll affect accuracy?

Of course it will.
It's because of the type of the camera (image quality, resolution, iso) and the environment (light is to low, how pure the background is).


LAPACKE: fix xlascl family of functions

Applies fixes made by upstream in
https://github.com/Reference-LAPACK/lapack/commit/5eca362f3a654c74e1743fea0c9bbc9bb3d7370f
Our tests in https://github.com/gonum/lapack/pull/206 are failing because of this

It is a sub-set of #996, probably it is best to review that and say that that one would fix your test cases too.

@vladimir-ch , I just merged LAPACK 3.6.1. Does it already include this patch?

@xianyi The patch is already included in lapack 3.6.1 so this PR is already done by your merge of LAPACK 3.6.1


Change default spec version to latest

Is your feature request related to a problem? Please describe.
I'm always frustrated when I have to manually look up the newest version of the CycloneDX spec and specify it manually via the cli. Newer versions of the spec bring improvements and defaulting to old versions hinders adoption.
Describe the solution you'd like
It would be very nice, if the newest supported version would be the default, then one doesn't have to specify a spec version and nevertheless can use the latest and greatest version of CycloneDX.
Describe alternatives you've considered
Adopt a clear guideline on when to change the default to a new version, when not changing it directly, but rather e.g. 1 week/month/year after release of the new spec version.
Additional context
Version 1.4 (the current default) has been released on 12 January 2022, so it is over a two years old now and is the default for at least 1.5 years now.
Version 1.5 has been released on 26 June 2023, so is almost a year old now as well.
Version 1.6 has been released on 09 April 2024, so it is almost 2 weeks old now as well, but has been supported since over a month now.
Dependency Track works flawlessly with CycloneDX 1.6.

Newer versions of the spec bring improvements and defaulting to old versions hinders adoption.

Changing de default values has no benefit for nobody in this case - it would not affect the SBOM result in no way. So I do not see your point here.
I am well aware of adopting new standards. But it took the CycloneDX community several months to adopt spec 1.5 when it came to ingesting the data.

Version 1.5 has been released on 26 June 2023, so is almost a year old now as well.

See for example, DependencyTrack - CDX 1.5 support was introduced with on October 16, 2023.

Dependency Track works flawlessly with CycloneDX 1.6.

See https://docs.dependencytrack.org/changelog/
As of today, DT v4.10.1 is the "latest" version. It was built months before CycloneDX 1.6 was released...
So I would not count on that ;-)

It would be very nice, if the newest supported version would be the default, then one doesn't have to specify a spec version and nevertheless can use the latest and greatest version of CycloneDX.

From which none of its features is used in this tool, yet.

All in all, I see your request, and still I do not see any reason to change a default value to 1.6 yet.
-> I will close this issue as soon as the "latest" version became the default. No worries.


raidemulator: Convert CombatantTracker and subclasses to typescript

Converting CombatantTracker and related dependencies to typescript.
I had to touch a lot of the LineEvent classes to let pulling combatant info work after converting to concrete types, which caused this PR to be bigger than I would have liked.
There's a fair amount of code here that'll probably need updated once someone with some more deep knowledge of typescript reviews, there's a lot of ... as ... casting in Combatant.ts for example due to bounds checks that tsc doesn't seem to understand.

There are a lot of files here.  Are there smaller dependencies you can pick off first?

This could be split into two PRs such that:

LineEvent interfaces are added and those classes updated, with about half of the updates to CombatantTracker (CombatantTracker stays as javascript)

~300 additions, ~200 deletions, 13 files changed

Typescript conversion of CombatantTracker and subclasses

~300 addtions, ~275 deletions, 4 files changed

raidemulator would likely be broken between those two PRs getting merged, as well, depending on how much I can get CombatantTracker's changes in PR 1 to function.
If you don't mind raidemulator being broken for a while, I can break this down into as many small PRs as you'd like.

CombatantJobSearch
CombatantState
Combatant
LineEvent*
CombtantTracker

CombatantJobSearch.getJob's signature changed from abilityID: string to abilityID: number, so CombatantTracker's call would need to be updated, or the signature change would have to happen further down in another PR
CombatantState.partialClone's props argument was changed from allowing string | number props to requiring number props, so Combatant would need to be updated to reflect that change or it would have to be done further down in another PR
As stated in my previous comment, LineEvent* classes were normalized to follow the interface specs, so CombatantTracker would have to be updated to reflect this change, there's no splitting this into another PR further down

As such, the series of PRs you outlined would be:

CombatantJobSearch => ts + a single-line adjustment to CombatantTracker
CombatantState => ts + roughly half of the changes to Combatant
Combatant => ts + roughly 2/3ds of the changes to CombatantTracker
LineEvent* changes + changing about 1/3rd of the code that was previously adjusted in CombatantTracker to reference the normalized properties
CombatantTracker => ts

There's just not a clean division of code for these classes because they're all pretty tightly coupled. Part of the reason of changing LineEvent* classes is to reduce some of that tight coupling.

What you outline looks good, with maybe combining CombatantState + CombatantTracker.  I think it's hard to separate these things out afterwards rather than make smaller changes up front.

Deleted previous comment as the suggestion didn't make sense, sorry.
What about then:

CombatantJobSearch => ts + a single-line adjustment to CombatantTracker
LineEvent* changes + changing about 1/3rd of the code that was previously adjusted in CombatantTracker to reference the normalized properties
CombatantState + Combatant + CombatantTracker together if they're really all that entwined

It's just hard to review a 20 file change where there's logic and typescript and a bunch of changes, and it's even harder to try to separate it out afterwards.  I think it'd be easier all around if there were just smaller patches up front, or fewer logic changes when there are also TypeScript conversions?

Closing this PR in favor of other PRs that were split off from this.


Tests not working with pushMiddlewareToGroup

See https://github.com/laravel/framework/issues/15426

Applied temporary patch!


fix: use on-the-fly compression only for file objects

When we introduced the new GzipStream utility class which
handles the on-the-fly compression of the file like object.
We also started using it for in-memory objects, like bytes,
but it has introduced a minor incompatibility with the existing SDKs.
This commit contains the fix for that. From now the new utility class
will be used only for file-like objects and the rest of the types will
be handled as before (compressing in memory).

:tada: This PR is included in version 3.17.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Create script to get variable sets

Posible name - Get-OctopusLibraryVariable
Filters - Project
It should show which projects are using the variable sets.

This cmdlet was renamed to Get-OctopusVariableSet. This cmdlet will get both Project Variable sets, and Library Variable sets. It'll be used inside of Add-OctopusVariable (#38) to get the variable sets on which the variables will be added/updated


Fix notebook links in doc

Fixes #412

However, I ran the Getting Started notebook, and it is broken in MyBinder because the model wasn't downloaded. Can you check locally on your side?

Oups, I had not thought about that!
I have the same error as you. Since the model is not in the repo anymore, I don't think it can work with that notebook. Maybe we could try adding the download in another notebook specific to MyBinder? Or remove that option?
I suggest to remove it for now and open an issue dedicated to bring it back, what do you think?

However, I ran the Getting Started notebook, and it is broken in MyBinder because the model wasn't downloaded. Can you check locally on your side?

Oups, I had not thought about that!
I have the same error as you. Since the model is not in the repo anymore, I don't think it can work with that notebook. Maybe we could try adding the download in another notebook specific to MyBinder? Or remove that option?
I suggest to remove it for now and open an issue dedicated to bring it back, what do you think?

Let's see if it can be quickly fixed, and if it can't then we'll remove it from the doc.

@mariehbourget I found a solution, and resolved it in #415. We can go ahead and merge this PR as-is.

Thanks @mathieuboudreau!


Cannot use md-data-table in strict mode

Hi! I tried using the table. Our application uses the javascript 'strict mode' but because of that I get the following errors:
Error: [$injector:strictdi] mdDataTableDirective is not using explicit annotation and cannot be invoked in strict mode (...)
angular.js:12416 Error: [$injector:strictdi] mdDataTableHeaderRowDirective is not using explicit annotation and cannot be invoked in strict mode (...)
angular.js:12416 Error: [$injector:strictdi] mdDataTableColumnDirective is not using explicit annotation and cannot be invoked in strict mode (...)
angular.js:12416 Error: [$injector:strictdi] mdDataTableRowDirective is not using explicit annotation and cannot be invoked in strict mode (...)
angular.js:12416 Error: [$injector:strictdi] mdDataTableCellDirective is not using explicit annotation and cannot be invoked in strict mode (...)

Same here...any solution?

@Keepertje, @rabih-elzahr I've pushed my changes and now it should work.
Can u please one of you verify that my fix is solving your issue?

@iamisti  It seems to work now :), thank you !


Move Notifications out of Canary

What will this achieve?
Moving the Notifications component closer to release
How will success be measured?
When all review list items are complete for this component.

https://github.com/carbon-design-system/ibm-cloud-cognitive/pull/509

@matthewgallo notifications adds a console error on test. Please mock the console error as per packages/cloud-cognitive/src/__tests__/index.test.js

https://github.com/carbon-design-system/ibm-cloud-cognitive/pull/535

This is complete


Updates the integration test for parent support.

What type of PR is this?
/kind failing-test
What does does this PR do / why we need it:
It introduces a function HttpFileServer which is
used to serve devfile by creating a http file server.
Which issue(s) this PR fixes:
Fixes https://github.com/openshift/odo/issues/3638
PR acceptance criteria:

[ ] Unit test

[x] Integration test

[ ] Documentation

How to test changes / Special notes to the reviewer:
The integration test for parent support should pass.

$ make validate
./scripts/check-gofmt.sh
gofmt ERROR - These files are not formated by gofmt:
./tests/integration/devfile/utils/utils.go
Makefile:65: recipe for target 'gofmt' failed

@amitkrout Fixed


Exposed insert does not return generated id due to lack of autoIncrement() on column type

Unfortunately, we cannot add autoIncrement(), because it leads to """id""" in queries due to the bug in exposed impl:

Blocked by: https://github.com/JetBrains/Exposed/issues/1658

#1


Avoid sender-side copy in AMPI for large contiguous messages

Original issue: https://charm.cs.illinois.edu/redmine/issues/1111

AMPI should use the new onesided/RDMA interface of Charm++ to avoid a sender-side copy for large message sends.

Original date: 2016-06-22 15:44:34

One issue is that we want to avoid copying the RDMA sent buffer on the receiver side to an AmpiMsg. That can only be done if the recv has already been posted and we are waiting on that message (it is the next inorder message), but this is worth doing. I already did something similar for (i)(all)reduce and (i)(all)gather(v) here: ~~https://charm.cs.illinois.edu/gerrit/#/c/1276/~~ https://github.com/UIUC-PPL/charm/commit/8b8b8aaab007023f31c571f035204ad57939d59a

Original date: 2016-07-14 18:54:32

Initial implementation here: ~~https://charm.cs.illinois.edu/gerrit/#/c/1312/~~ https://github.com/UIUC-PPL/charm/commit/9349919e23899d482d611c84f8a6080176056c54
The biggest issue here is migration: what if a user calls AMPI_Migrate while one or more sends are pending? We can explicitly disallow calling AMPI_Migrate with pending send requests, though it needs to be documented in the AMPI manual.
TODO:

Use RDMA for MPI_SSend and MPI_Issend.
Use RDMA for MPI_Rput and MPI_Raccumulate.
-3. Document non-migratability of ranks with pending send requests.-

Original date: 2017-04-27 22:25:09

Related issues left to do:

Use RDMA for MPI-3's MPI_Rput and MPI_Raccumulate routines.
Use RDMA sends when using BigSIm on AMPI

These are minor points that can be addressed in the future.


War analysis require pom.xml?

Is pom.xml inside artifact required when I analyze a war/ear?
I have some artifacts built with Apache Ant (so without a pom inside) and cdxgen fail to analyze them:
root@5a1a49758fe7:/app# cdxgen --version
3.2.9
root@5a1a49758fe7:/app# cdxgen -t java myapplication.war
Cleaning up /tmp/war-deps-SjtRrX
Unable to produce BOM for myapplication.war
root@5a1a49758fe7:/app#

@mvanini The logic is here - https://github.com/AppThreat/cdxgen/blob/master/utils.js#L2261
I'm looking at MANIFEST.MF recursively. Would you be able to share a sample war file with me directly so that I can think about a different mode?
Email: prabhu @ appthreat .com


Link to Quick Start in README

Add link to Quick Start documentation in readme
https://github.com/8090-inc/xrx-core/issues/14

Thanks @sabbyanandan !


Kakadu Can't Process Compressed TIFFs

There are currently 69 derivatives jobs failing because Kakadu can't process compressed TIFFs.  It fails with the error:

The simple TIFF file reader in this demo application can only read uncompressed
TIFF files.  This has nothing to do with Kakadu itself, of course.  If you
would like to read compressed TIFF files, however, it should be sufficient to
re-compile this application with the symbol "KDU_INCLUDE_TIFF" defined, and
link it against the public-domain LIBTIFF library.

We should either replace the TIFFs with uncompressed TIFFs, or recompile Kakadu with better TIFF library support so it can handle compressed TIFFs.  Recompiling probably has licensing issues, but might improve performance.

The compressed TIFFs are:
/mnt/diglibdata/hydra_binaries/p0/g3/56/h3/p0g356h32q/00000001.tif
/mnt/diglibdata/hydra_binaries/p1/r6/6m/21/p1r66m2195/00000014.tif
/mnt/diglibdata/hydra_binaries/p2/b8/8s/c7/p2b88sc710/00000218.tif
/mnt/diglibdata/hydra_binaries/p3/48/51/j9/p34851j92f/00000234.tif
/mnt/diglibdata/hydra_binaries/p3/f4/64/69/p3f464690k/00000015.tif
/mnt/diglibdata/hydra_binaries/p4/t6/4j/p4/p4t64jp490/00000013.tif
/mnt/diglibdata/hydra_binaries/p4/x5/1k/n0/p4x51kn03n/00000226.tif
/mnt/diglibdata/hydra_binaries/p5/d8/6r/17/p5d86r174c/00000007.tif
/mnt/diglibdata/hydra_binaries/p5/t3/4v/m2/p5t34vm229/00000009.tif
/mnt/diglibdata/hydra_binaries/p5/x2/1w/h8/p5x21wh83d/00000010.tif
/mnt/diglibdata/hydra_binaries/p6/97/01/37/p69701376q/00000009.tif
/mnt/diglibdata/hydra_binaries/p6/97/01/38/p69701384c/00000016.tif
/mnt/diglibdata/hydra_binaries/p7/65/39/24/p765392450/00000239.tif
/mnt/diglibdata/hydra_binaries/p7/94/09/z9/p79409z940/00000001.tif
/mnt/diglibdata/hydra_binaries/p7/m0/1d/p5/p7m01dp572/00000002.tif
/mnt/diglibdata/hydra_binaries/p7/s7/5g/g5/p7s75gg548/00000002.tif
/mnt/diglibdata/hydra_binaries/p8/33/6k/51/p8336k5119/00000219.tif
/mnt/diglibdata/hydra_binaries/p8/k7/1q/j6/p8k71qj63x/00000006.tif
/mnt/diglibdata/hydra_binaries/p8/w3/2t/74/p8w32t748x/00000004.tif
/mnt/diglibdata/hydra_binaries/p9/s1/63/91/p9s1639117/00000004.tif
/mnt/diglibdata/hydra_binaries/p9/s1/63/91/p9s1639145/00000238.tif
/mnt/diglibdata/hydra_binaries/p9/w0/34/27/p9w034276m/00000006.tif
/mnt/diglibdata/hydra_binaries/pb/85/17/r8/pb8517r82n/00000001.tif
/mnt/diglibdata/hydra_binaries/pb/c3/88/n1/pbc388n16p/00000005.tif
/mnt/diglibdata/hydra_binaries/pd/50/4t/p4/pd504tp46g/00000237.tif
/mnt/diglibdata/hydra_binaries/pd/j5/2z/72/pdj52z722v/00000221.tif
/mnt/diglibdata/hydra_binaries/pf/18/83/n8/pf1883n80k/00000006.tif
/mnt/diglibdata/hydra_binaries/pf/18/83/n8/pf1883n882/00000002.tif
/mnt/diglibdata/hydra_binaries/pf/x7/1c/q0/pfx71cq09n/00000004.tif
/mnt/diglibdata/hydra_binaries/pf/x7/1c/q1/pfx71cq12r/00000230.tif
/mnt/diglibdata/hydra_binaries/pg/15/8d/j9/pg158dj96h/00000002.tif
/mnt/diglibdata/hydra_binaries/pg/h9/3k/22/pgh93k2224/00000011.tif
/mnt/diglibdata/hydra_binaries/ph/41/5r/98/ph415r988j/00000235.tif
/mnt/diglibdata/hydra_binaries/ph/q3/7x/q4/phq37xq41k/00000002.tif
/mnt/diglibdata/hydra_binaries/ph/q3/7x/q5/phq37xq56d/00000002.tif
/mnt/diglibdata/hydra_binaries/pj/d4/74/w8/pjd474w831/00000008.tif
/mnt/diglibdata/hydra_binaries/pj/h3/45/t5/pjh345t584/00000224.tif
/mnt/diglibdata/hydra_binaries/pj/s9/58/h1/pjs958h10t/00000005.tif
/mnt/diglibdata/hydra_binaries/pp/26/78/w7/pp2678w71g/00000227.tif
/mnt/diglibdata/hydra_binaries/pp/v6/3j/09/ppv63j099k/00000000.tif
/mnt/diglibdata/hydra_binaries/pq/23/7k/s8/pq237ks82v/00000002.tif
/mnt/diglibdata/hydra_binaries/pq/z2/0v/s5/pqz20vs57k/00000011.tif
/mnt/diglibdata/hydra_binaries/pr/49/4x/m7/pr494xm78w/00000000.tif
/mnt/diglibdata/hydra_binaries/pr/78/1z/j6/pr781zj60h/00000008.tif
/mnt/diglibdata/hydra_binaries/pr/j4/32/66/prj432668x/00000012.tif
/mnt/diglibdata/hydra_binaries/pr/v0/44/v2/prv044v20w/00000233.tif
/mnt/diglibdata/hydra_binaries/pr/x9/15/s4/prx915s403/00000228.tif
/mnt/diglibdata/hydra_binaries/ps/q8/7d/w3/psq87dw34k/00000225.tif
/mnt/diglibdata/hydra_binaries/ps/t7/4f/r7/pst74fr766/00000232.tif
/mnt/diglibdata/hydra_binaries/pt/43/5j/g0/pt435jg06k/00000236.tif
/mnt/diglibdata/hydra_binaries/pt/m7/0p/w4/ptm70pw410/00000240.tif
/mnt/diglibdata/hydra_binaries/pt/t4/4r/p2/ptt44rp22m/00000222.tif
/mnt/diglibdata/hydra_binaries/pv/h5/3z/z1/pvh53zz19s/00000002.tif
/mnt/diglibdata/hydra_binaries/pv/m4/10/t8/pvm410t819/00000002.tif
/mnt/diglibdata/hydra_binaries/pv/m4/10/v1/pvm410v109/00000217.tif
/mnt/diglibdata/hydra_binaries/pv/q2/81/p4/pvq281p467/00000001.tif
/mnt/diglibdata/hydra_binaries/pw/08/94/c1/pw0894c10w/00000003.tif
/mnt/diglibdata/hydra_binaries/pw/66/36/55/pw6636555t/00000007.tif
/mnt/diglibdata/hydra_binaries/pw/95/07/18/pw9507185j/00000003.tif
/mnt/diglibdata/hydra_binaries/pw/95/07/20/pw95072076/00000010.tif
/mnt/diglibdata/hydra_binaries/px/g9/4k/s5/pxg94ks567/00000231.tif
/mnt/diglibdata/hydra_binaries/pz/60/3s/z4/pz603sz48v/00000003.tif
/mnt/diglibdata/hydra_binaries/pz/60/3s/z5/pz603sz57t/00000223.tif
/mnt/diglibdata/hydra_binaries/pz/89/0t/x0/pz890tx005/00000004.tif
/mnt/diglibdata/hydra_binaries/pz/89/0t/x0/pz890tx07b/00000220.tif
/mnt/diglibdata/hydra_binaries/pz/g6/4w/p2/pzg64wp27h/00000229.tif
/mnt/diglibdata/hydra_binaries/pz/k5/1x/h2/pzk51xh219/00000001.tif
/mnt/diglibdata/hydra_binaries/pz/w1/31/65/pzw131659s/00000003.tif

Are these PUDL or PULFA images? If PUDL, @roelmoon would probably like to know.

The images in MC016 that couldn't be processed by Kakadu are a little more complicated than i thought.  I thought there were whole objects where all of the page images were compressed.  But now that most of MC016 is ingested, there are 235 images.  Some of the are compressed, but some are 256-color instead of grayscale.  And some objects have issues with all of their files, but some are just one file in an otherwise OK object.
I think we can still update the files (save the current versions in another location, and convert them to grayscale for ingest).  But do we want to review these at all?  Is this just sloppy digitization?

It was a Crowley project, so sloppiness is almost certainly the case. Were
any of the 256 color images actually color?
-Js
Sent via mobile. Please excuse typos, brevity, etc.
On Apr 7, 2017 3:16 PM, "Esmé Cowles"<EMAIL_ADDRESS>wrote:

The images in MC016 that couldn't be processed by Kakadu are a little more
complicated than i thought. I thought there were whole objects where all of
the page images were compressed. But now that most of MC016 is ingested,
there are 235 images. Some of the are compressed, but some are 256-color
instead of grayscale. And some objects have issues with all of their files,
but some are just one file in an otherwise OK object.
I think we can still update the files (save the current versions in
another location, and convert them to grayscale for ingest). But do we want
to review these at all? Is this just sloppy digitization?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pulibrary/plum/issues/1146#issuecomment-292627635,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA9BFaOvzDeseQG4O9Nx0vk6lwUH9312ks5rtouAgaJpZM4My6QU
.

I can run a batch routine to normalize the files. It would be ideal if the component IDs can be isolated.
I agree, Crowley hasn't generated much confidence lately.

@rmoon: When the MC016 ingest has completely finished, I'll make a complete list of the affected component IDs and files.  I haven't downloaded any of the TIFFs to see if they are actually color, or if there are any other issues with them.  But taking a quick look at some of them seems like a good idea to make sure there aren't bigger problems here.

@rmoon: the ingest jobs have finished and there are 238 files that fail to generate derivatives
files.txt
I downloaded 10 files at random, and all 26 of the files that are 256-color, and they were all grayscale.  Of the 256-color images, most were text, but some also had photos/illustrations, and a few were handwritten "Please omit" notes.

@rmoon Any status update on normalizing the files?

Done. fixed images merged with rest of MC016.

Thanks @rmoon — I'll reingest the impacted objects once the pudl0003 ingest has finished.


read a book about Qualitative Interviewing

I need to improve my skills in it, so I want to read "Learning From Strangers" by Robert Weiss

There is a $24 open bounty on this issue. Add to the bounty at Bountysource.

Hey, @DarkEclipse847 !
Let's make it about just a chapter or two, to keep the tasks small!
Godspeed digging into the book!

Hey, @DarkEclipse847 !
Let's make it about just a chapter or two, to keep the tasks small!
Godspeed digging into the book!

Hey, @DarkEclipse847 !
Let's make it about just a chapter or two, to keep the tasks small!
Godspeed digging into the book!

Kk

Hey, @DarkEclipse847 !
Let's make it about just a chapter or two, to keep the tasks small!
Godspeed digging into the book!

Kk

What do you mean while saying chapter two? :D
Second chapter in Part 1 or Second Pert of the book

What do you mean while saying chapter two? :D
Second chapter in Part 1 or Second Pert of the book

Had just the second chapter in mind!

Had just the second chapter in mind!

Introduction
If i will have any additional thoughts about it i will add them to github or personally to telegram
Firstly, In second chapter we can see some introduction part that we need to pay attention.
Qualitative Interviewing is not always about numerical data or something, it's about psychology and communication with colleagues/friends.
Collecting such information is great for understanding people.
Cool thing there is that we can adapt questions for different groups of people
Main Part
Sooo, I dont't know what to write about, so i will try to set out my thoughts
We can see in excerpt, that quialitative interviewing give us more information than simple sociological poll for example.
It's like an open form of quizes and interviews.
In this excerpt interviewer is too cagey, he could answer something more concrete cause interviewer is like a guide who conducts through topics.
"In the qualitative interview the respondent provides information while the interviewer, as a representative of the study, is responsible for directing the respondent to the topics that matter to the study."
He must act this way. In that case we see right communication with respondent, but I think that there is an other way to do qualitative interviews.
I think that we can improve our skills in it by representing interview like a conversation.
More warmer, more comfortable for respondent.

Introduction
If i will have any additional thoughts about it i will add them to github or personally to telegram
Firstly, In second chapter we can see some introduction part that we need to pay attention.
Qualitative Interviewing is not always about numerical data or something, it's about psychology and communication with colleagues/friends.
Collecting such information is great for understanding people.
Cool thing there is that we can adapt questions for different groups of people
Main Part
Sooo, I dont't know what to write about, so i will try to set out my thoughts
We can see in excerpt, that quialitative interviewing give us more information than simple sociological poll for example.
It's like an open form of quizes and interviews.
In this excerpt interviewer is too cagey, he could answer something more concrete cause interviewer is like a guide who conducts through topics.
"In the qualitative interview the respondent provides information while the interviewer, as a representative of the study, is responsible for directing the respondent to the topics that matter to the study."
He must act this way. In that case we see right communication with respondent, but I think that there is an other way to do qualitative interviews.
I think that we can improve our skills in it by representing interview like a conversation.
More warmer, more comfortable for respondent.

Thanks for doing this, @DarkEclipse847 !
Did you like the interviewing examples in the book?
What will you read next?

Thanks for doing this, @DarkEclipse847 !
Did you like the interviewing examples in the book?
What will you read next?

Yep, I like it, there's comfortble table in Part 4, check it out!
Thinking of reading some books about selfhypnosys
Btw if you have some good books please tell me their titles

Yep, I like it, there's comfortble table in Part 4, check it out!
Thinking of reading some books about selfhypnosys
Btw if you have some good books please tell me their titles

That's a tough one! There is no single book. On one hand, SH can be described sufficiently in one paragraph (as done by Ian Rowland). On another, one might want to reinvent the thing. I'll likely get back to you on Telegram.

That's a tough one! There is no single book. On one hand, SH can be described sufficiently in one paragraph (as done by Ian Rowland). On another, one might want to reinvent the thing. I'll likely get back to you on Telegram.


Thanks for the inspiration!

Describe the feature request
The simplicity of this library inspired me to write my own simple color library. Thank you!
Some things you might be interested in grabbing from gocolor:

codegen to add more colors from the ANSI colors supported on Windows (and they copied from Unix so they work there too)
Color.Func to let users easily create their own color funcs

Why do you personally want this feature to be implemented?
No response
How long have you been using this project?
No response
Additional information
No response

Awesome work! I'm always glad to see new libraries pop up, even if there are already existing options, as it helps prevent a monopoly from forming around a requirement.
Best of luck with your color library!


Remove dead settings from package.json

Fix https://github.com/microsoft/vscode-jupyter/issues/6972 and cleanup after removing the webview interactive window

Codecov Report

Merging #7087 (f876519) into main (e0dc6dd) will decrease coverage by 0%.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main   #7087   +/-   ##
=====================================
- Coverage     65%     64%   -1%
=====================================
  Files        362     362
  Lines      22897   22897
  Branches    3426    3426
=====================================
- Hits       14910   14877   -33
- Misses      6683    6708   +25
- Partials    1304    1312    +8

Impacted Files
Coverage Δ

...ence/notebook/noPythonKernelsNotebookController.ts
14% <0%> (-26%)
:arrow_down:

.../datascience/notebook/notebookControllerManager.ts
79% <0%> (-8%)
:arrow_down:

src/client/api/pythonApi.ts
59% <0%> (-5%)
:arrow_down:

src/client/common/cancellation.ts
72% <0%> (-4%)
:arrow_down:

...t/datascience/notebook/vscodeNotebookController.ts
78% <0%> (-2%)
:arrow_down:

...client/datascience/kernel-launcher/kernelDaemon.ts
53% <0%> (-2%)
:arrow_down:

src/client/datascience/serviceRegistry.ts
98% <0%> (-1%)
:arrow_down:

src/client/datascience/jupyter/jupyterNotebook.ts
51% <0%> (+1%)
:arrow_up:

src/client/datascience/raw-kernel/rawSocket.ts
83% <0%> (+1%)
:arrow_up:


Error Unauthorized

GET http://localhost:1337/cake/count 401 (Unauthorized)

Hi @aalshaikh can you allow your guest user to access this routes in the user & permissions tab?
When you click on the Guest role you should see all your content types and you can set permissions according to your need.
Let us know if it works!

I'm closing the issue


Limit number of picked days

Hello,
Is there some way to limit number of picked days?
Obviously i am using "multiple".
Let´s say i would like to limit it to 3 days, so when u pick 3 different days u can´t choose more but, when u deselect one of them u can again pick another date.
Any idea ?

Hi there,
I am also facing the same issue. Were you able to fix this? @Ansaen

The solution I found is to use the mapDays prop and pass the disabled prop when reaching the limit for other than selected days:
value={values}
mapDays={({ date }) => {
    const reachLimit = values.length === selectionLimit;
    const isNotSelectedDate = !values.some(val => moment(val).isSame(moment(date.toDate())));

    return reachLimit && isNotSelectedDate ? { disabled: true } : {};
}}


new summary text on top of pages displays unhelpful info on devtools homepage

See "Option 1" under "Tools"
I'd also like for this to say "Chrome DevTools" rather than "Tools". Maybe we can just re-org the YAML files?

i think this is fixed, if not, please re-open


feat(apps): add PowerToys CropAndLock

[x] I have formatted applications.yaml with komorebic fmt-asc.
[x] I added float identifier for PowerToys CropAndLock

Reparent mode doesn't play well with komorebi-managed windows:


Image format

Hi. I tried to print image but something is wrong, it prints too-many characters. Which image type I must pass to p.Image()?

The implementation of the image printing was never finished. I was porting it from a PHP library escpos lib, but never got around to printing it. The quality of the images that are printed (at least on the e-pos printers I have) aren't very good quality anyways.

Ok, thank you, I will try to do it

For what it's worth, I've learned a hell of a lot about Go since I wrote the original escpos code. It would be fairly easy to integrate the C code from this repository: png2pos. You might want to look into doing that. I would do it myself, but I have no immediate need to print images via an escpos printer/server at the moment.


Make gutter always visible

Disabling the gutter makes the breakpoints feature unusable. The preference for disabling the gutter should be modified to a preference to toggle line numbers only.

After all, we can keep an option to completely disable the gutter, in some cases it might be useful.


Fix lag

Was creating an error every time the blob moved and overloading the web browser

Thanks for the fix :D and welcome fellow spigot server owner :P

This idea for that code is to memorize stuff that gets removed for an extra 3 seconds. I'm surprised there's a problem with it.

Yea. I removed it on my local copy of the bot and it appears to work fine
On Wed, Aug 19, 2015, 11:35 AM Apostolique<EMAIL_ADDRESS>wrote:

This idea for that code is to memorize stuff that gets removed for an
extra 3 seconds. I'm surprised there's a problem with it.
—
Reply to this email directly or view it on GitHub
https://github.com/Apostolique/Agar.io-bot/pull/326#issuecomment-132640628
.

What's the error that you would see?

The variable you are calling it on is undefined
On Wed, Aug 19, 2015, 11:39 AM Apostolique<EMAIL_ADDRESS>wrote:

What's the error that you would see?
—
Reply to this email directly or view it on GitHub
https://github.com/Apostolique/Agar.io-bot/pull/326#issuecomment-132642108
.

If you add:
console.log("Time not updated: " + (window.getLastUpdate() - interNodes[element].getUptimeTime()));

It doesn't compute a valid number?

interNodes[element] is aparently undefined. All i know is i removed it on my side, it seems to be working and it isn't lagging.

interNodes is an object that contains elements. (cell IDs)
We loop through that object using:
Object.keys(interNodes).forEach(function(element, index) {
    ....
}

I don't think it's possible for interNodes[element] to be undefined.
It's the same as doing:
var someArray = [1, 2, 3, 4, 5];
for (var i = 0; i < someArray.length; i++) {
    console.log("The element at index: " + i + " is " + someArray[i]);
}

and saying that someArray[i] is undefined.

I had the same problem, thanks for the solution!

The problem with internodes is it has empty slots because it uses the delete command instead of splicing.

Oh.

Its not nessesarily a problem it just means you gotta use foreach instead of
for (var i = 0, var interlen = internodes.length; i < interlen; i++)

which is much much faster especially if you cache the length like I showed above


WIP/RFC: initial stab at TorchScript fallback

This patch adds a support for calling TorchScript. This can be used fallback for when torch operators are not yet implemented or if one wants to incorporate bespoke PyTorch custom ops into TVM with ease.
It adds

a new relay torchop that takes a variable number of inputs and executes a provided TorchScript (aka PyTorch JIT) function,
a backend module class that calls into LibTorch (aka the C++ bindings underlying PyTorch), executing a TorchScript function.

The key addition to the relay infrastructure is that while leaving num_inputs == -1 on operator registration is documented to indicate a variable number of inputs, the type inference pass is not prepared to deal with it and instead requires the number of arguments provided to match the number of arguments with add_argument on operator registration. The proposed change to the type inference is then to match the declared arguments but to allow additional arguments if num_inputs is -1.
The other detail is that it uses a string attribute in the call node's attributes to take the serialized TorchScript. This is a bit fishy as the serialized representation is binary. I used serialization to get TorchScript into TVM at the C++ level as it is tricky to interoperate between PyBind-wrapped objects (TorchScript in PyTorch) and the TVM FFI, but we might pass things around as handles later (but I'm not sure if that works well with attributes). I would be glad to have your advice on this.
Currently I only support one output tensor and FP32, but that is straightforward to make flexible, and I would do this in parallel to the more fundamental discussion e.g. around the things above.
Even though this is in draft state, I'm opening this PR to make discussions concrete in terms of code.
I also posted a discussion to the forum.

This is an interesting use case of byoc, cc @zhiics @comaniac

I'm curious how it integrates with PyTorch frontend. Do we convert every op not supported to relay.torchop, run BYOC flow to get TorchScript subgraphs, and send them to libtorch? Sounds interesting!

I'm curious how it integrates with PyTorch frontend. Do we convert every op not supported to relay.torchop, run BYOC flow to get TorchScript subgraphs, and send them to libtorch? Sounds interesting!

This is how I'd like it to work out. I've been thinking what the best "level" is and while the operator level might seem attractive, but I I'm not sure there is an easy way to run individual operators.
It won't help with out favourite inplace problems, though.

I have the same question as masahi. IIUC, after this PR, the PyTorch frontend has the capablility to convert all unsupported ops to torchop so that we can guarantee the flow would work. This is an interesting idea and this would be the first BYOC use case that could potentially incopreate two BYOC backends (Torch and others like TensorRT).
cc @zhiics

Yeah, the general idea is to use this as the fallback. I can add the fallback generation here in the PR if that is better.
Also I added a bit of a pro-con discussion regarding single op vs. program on the forum thread, if you have opinions, I'd be very grateful if you could chime in.

Hey @t-vi, the idea of a fallback for unsupported TorchScript Ops is great. I am currently pursuing a similar approach for unsupported (and custom) TFLite Ops.
I also stumbled over the issue that  num_inpust == -1 leads to problems in the type_infer step and "solved" it in a quite bad way by just packing all my inputs in a single Tuple and using num_inputs = 1. I would really appreciate getting at least your fix to solve this issue merged into upstream. Maybe in a separate PR at this is not really related to the TorchScript use case.

I would really appreciate getting at least your fix to solve this issue merged into upstream. Maybe in a separate PR at this is not really related to the TorchScript use case.

I'm all for it, but I wouldn't know how to add tests in lieu of something using it. If you or @masahi has any opinion on this, I'd be glad to split it out...

So I have been mulling over the best granularity / approach. Currently I'm taking TorchScript functions / graphs as the unit I'm working with. An alternative could be to move to the PyTorch operator level (so one aten::...-call) - which would seem to be more natural in Relay - but then one would either have separate TVM Modules for each operator or have to piece things back together into a graph.
What do you think?

I was going to suggest using MergeCompilerRegions, but I saw you already using it. So I like your current approach. Sending many small functions to torch sounds like a non-trivial overhead, and I think " piece things back together into a graph" is essentially what MergeCompilerRegions does at Relay level.

Just a quick note that when I tried to revive this back in the summer it got a bit stalled around pytorch/pytorch#65047 .

@t-vi Is this still WIP? I'm happy to merge whenever you think it is ready.

@masahi So I had hoped to get the dlpack header version in PyTorch bumped (see the linked bug) but Facebook has internal uses that let it insist on the old one.
I wonder if we could work around it by providing a "dlpack-compat" header that defines the names for the types / fields? Or I could try to somehow separate the two.

I wonder if we could work around it by providing a "dlpack-compat" header

Does this mean the same thing as https://github.com/pytorch/pytorch/pull/65047#issuecomment-972734912? (which sounds good to me). Anyway it seems we cannot count on the PyTorch-side to change, so I'd say anything that can unblock you from our side should be a good plan. cc @tqchen @junrushao1994

As commented by the author of PyTorchTVM, https://github.com/apache/tvm-rfcs/pull/25#issuecomment-908041324, many people are interested in this feature. Also people are actively talking about deeper integration of TVM into PyTorch workflow. So we should definitely land this.
As for me personally, I want to try out this feature to import models having fft-related ops (specifically, this cool model https://github.com/raoyongming/GFNet). FFT ops won't come to TVM any time soon.

So I thought, I could wait it out, but I'll look into working around the version discrepancy in the next few weeks.

Another interesting use case this fallback could enable is mmdetection https://github.com/open-mmlab/mmdetection. It has a lot of cool detection models but rely on many custom ops that cannot convert to relay.

M. Ruberry of the PyTorch team re-landed the update of the dlpack.h in PyTorch. If this still holds next week, it'll be exciting to bring this up to date. :)

Hi, so I rebased this finally and it all compiles and runs one test against a current PyTorch master, so I think I'm back in business with this PR (unless it has been obsoleted, but from what I understand, the bridge is in the other direction).

Great! Can you remove WIP from the title now?

Are you on the tvm discord or so to quickly discuss?

Thank you @masahi !

Hi, @t-vi. Thank you for the great contribution! This seems very interesting.
I'm trying to check out this feature but having some ABI link issue during setup - ld.lld: error: undefined symbol.
This seems like a known issue (link), so I followed their suggestion of adding -D_GLIBCXX_USE_CXX11_ABI=0".
However, this change makes TVM llvm codegen fails with ABI link.
Have you faced this issue before?
I'm wondering what will be the general guideline to fix such ABI conflicts.
Thank you!
For more detailed info, I installed pytorch package with conda and downloaded prebuilt binary by following the instruction.
Then, I modified my config.make to set(USE_LIBTORCH ~/libtorch).

The fundamental problem is that (pre-compiled) PyTorch python modules use the pre C++-11 string ABI to better blend into the Python ecosystem or so. TVM does not, so it needs to link to LibTorch with the "new" C++ string ABI.
But these two versions clash.
One option is to use self-compiled PyTorch (which is what I do and so got us into this mess...).
I've been trying on and off to look into linking LibTorch such that all symbols are hidden (and you can load PyTorch with whatever C++ ABI later), but I have not managed yet.

@sunggg This pr (https://github.com/apache/tvm/pull/12232) used a different way to solve the version mismatch of c++ ABI between TVM and pytorch, maybe worth looking into.


Run go test cases in a structured sequence and use test fixtures

https://github.com/intuit/katlas/pull/115

Done


 Most of the work for this ticket was in #6407, this is just catching a bug we hadn't noticed.
Reviewer Notes
Not sure if I should expand the tests to cover more details on the page. Currently we're checking headers and edit links, but maybe it should check all the data? It would be more tedious and verbose tests, but that way we could be sure we aren't missing info we expect? What do y'all think?
Setup

Run server
make server_run

Run client
make client_run

Log in as a service member.

Go to their profile page.

If they don't already have an alt phone, edit their profile to add one.

You should see the alt phone displayed

Code Review Verification Steps

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

Screenshots
Shows secondary number on profile page:

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

Messages

:book:
:link: MB-5883

  Generated by :no_entry_sign: dangerJS against 33085d8b71fc15b92d1dd9b094d12b9151265e8e

This PR itself looks good, but I'm seeing a server error when I try to edit the contact information after a move has been submitted.
e.g. submit a move, go to the profile page, attempt to edit the contact info -> bad request alert pops up


Start of executable program files and scripts

Hello Isaac,
thank you for your excellent software presencelight.
Is it possible for you to add the start of executable programs and scripts?
I would like to be reminded of various manual tasks at the start and end of a Teams meeting.
For this I would like to have a small message displayed using a script (VBS or PS1).
I would also like to have all Windows minimized when a Teams meeting/phone call starts.
This could also be done very easily with a script.
It would be ideal if a configuration page like the one in "Custom API" was available,
where you can store the path to executable EXE/script file and the parameters for each status.
Then you would have almost unlimited actions and execution options.
As a workaround, I thought about installing a web service that calls the scripts for me and whose URLs I then store in "Custom API".
I am not a developer and do not know exactly what information would be helpful.
If you are still missing information, please feel free to contact me.
Many thanks and kind regards
Simon

This is outside the scope of PresenceLight today. I recommend you fork the repo and add the functionality you are looking for. Feel free to email me if you run into issues with Entra configuration


reset data each reboot

Hi
this a very nice extension, nevertheless each time i reboot my linux, the counter starts again to 0 (amount of data used)
even if i choose "never"..and same for the other..
do you know why ?
thks !

Fixed in version > 15.

it seems working :-) (at least for keeping the data, we'll see reset for a month)
thks a lot !


add movetoroot method to dwindle layout

Describe your PR, what does it fix/add?
This pull request adds a movetoroot method to layoutmsg for dwindle layout. It "maximizes" the current window within the current workspace/layout by swapping it with the other sub-tree of the root node. In some sense, this is a replication of swapwithmaster method from the master layout.
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
This should not break anything. I used this pull request as a template, but I didn't bother adding another keybind seeing that layoutmsg, movetoroot just works.
Is it ready for merging, or does it need work?
It is ready for merging as is.
Though I could imagine to optimize the implementation a bit by limit recalcSizePosRescursive to the active window and swapped sub-tree, but I'm not familiar with the codebase enough to make the judgement. I also don't think average user will have a layout tree big enough for the optimization to make a difference.

also wiki mr needed or is there one already?


Different lock color when it's locked by current user

The lock will turn orange indicating that that specific page is locked by the current user.

NOTE: we can also add this line on the top "Golden locks are pages locked by YOU"
[^1]: Development Testing
[^2]: Browser and device support
[^3]: Accessibility Target

@zerolab please review my code

Also added a note that we should use a colour that already exists in the colour variables.
As for the note about what the 'gold' means - preference would be to avoid doing that.


[CI] Fix test skipping pytest attribute

The test previously had the incorrect pytest attribute skip_if, this replaces with the correct one

@tvm-bot rerun

nice catch, thanks!

I believe the Android issue is unrelated here so I merged this, thanks @neildhickey and @driazati!


Linux Port

Recently i have been developing a Linux-Compatible port of this program and a associated "installer" or "patch" that allows it to run like a normal .exe, here is a proof of concept build (seen below running on Kubuntu 19.04)

Pretty cool dude!

rad

Hey, any updates on this?


PHP: add deserialze as the argument for the interceptor

Hi @mehrdada
For the interceptor API, I feel like adding deserialize as the argument makes more sense, because it should be couple with the method. For example, If the user changes the method through the interceptor, the response type will change and the deserialize method should change too. In another case, if the user uses something else other than protobuf, they can use the interceptor to pick whatever deserializer they want.
In addition, use($channel) in the code is enough to hide what the user doesn't needs.
Since we are releasing the PHP interceptor by 1.13.0 and now it's 1.13.0-pre, if this change make sense, I think we can backport it to 1.13.0?

@ZhouyihaiDing this is an API change, so we should consider it carefully. The process to change it is necessarily heavy weight. However, since the API is not in yet, this is pretty much the last chance to change it.
So let's consider this change here before we decide if we want to re-visit the RFC.

Why will the response type change cause the deserializer change? Isnt the de-serializer fixed to how the response is encoded, rather than what the data is?
Do you have some use cases where we have encountered this?
Normally, interceptors contain cross cutting concerns rather than business logic, so they only mutate data rarely.. so how often do you see users get into a problem with the current API?
How do other languages do this, do they provide a deserializer?

I finished the implementation which only uses the interceptor to hook the GCP extension support with the change of this PR: https://github.com/ZhouyihaiDing/grpc/pull/22. The GCP interceptor is implemented with self maintained channel and call.
I chatted with @mehrdada offline and we could merge this to 1.13.0 by state "There is an experimental API and there is a proposal under the review".
Otherwise I need to add 2 more APIs as described before to de-couple the channel and call object from the stub, because currently there is no API allows me to hook the creating of any object with all information needed.
I am okay with adding CallInvoker and changing the interceptor, because CallInvoker will make the GCP extension looks more directly(and since the C# has it, maybe PHP can have it too), while the Inteceptor changes is doing the least changes to the grpc php code.

Hi, @chingor13 @michaelbausor
I am going to update the API for the interceptor to decouple the Call object and Channel object. Can you please give me some suggestions if there are some concerns relate to it? Thanks!

Sorry. I used the rebase in a wrong way and it was trying to commit tons of commits into the v1.13.x branch.

Known issues for the test:
Bazel Debug build for C/C++  - #15610
Bazel Opt build for C/C++ - #15610


Feature: break long lines

Note: transferred from rest.nvim repository as this has something to do with the parser grammar.


In-place update of resource dnacenter_configuration_template_version does not do anything with API

Prerequisites

[x] Have you tested the operation in the API directly?
[x] Do you have the latest Terraform provider version?
[x] Review the compatibility matrix before opening an issue.

Describe the bug
Creation and deletion of resource dnacenter_configuration_template_version works right, but once I need to change the resource no requests to DNAC API are being sent.
Output of terraform apply command:
data.dnacenter_configuration_template_project.all: Reading...
data.dnacenter_configuration_template_project.all: Read complete after 0s [id=1695747072]

Terraform used the selected providers to generate the following execution plan. Resource actions are indicated with the following symbols:
  ~ update in-place

Terraform will perform the following actions:

  # dnacenter_configuration_template_version.this will be updated in-place
  ~ resource "dnacenter_configuration_template_version" "this" {
        id   = "template_id:=3e02dfb5-d67d-4f49-a331-79fdcdc401a9"
        # (1 unchanged attribute hidden)

      ~ parameters {
          ~ comments    = "9003" -> "9004"
            # (1 unchanged attribute hidden)
        }
    }

Plan: 0 to add, 1 to change, 0 to destroy.
dnacenter_configuration_template_version.this: Modifying... [id=template_id:=3e02dfb5-d67d-4f49-a331-79fdcdc401a9]
dnacenter_configuration_template_version.this: Modifications complete after 0s [id=template_id:=3e02dfb5-d67d-4f49-a331-79fdcdc401a9]

Apply complete! Resources: 0 added, 1 changed, 0 destroyed.

Terraform debug log: dnac.sanitized.log
Expected behavior
The resource should send POST request with new payload.
Environment (please complete the following information):

Cisco DNA Center version and patch: <IP_ADDRESS>-70852
Terraform version: 1.5.6
Cisco DNA Center provider version: 1.1.17-beta
OS Version: Ubuntu 20.04

According to the documentation, there is no UPDATE method for this resource, we are working on providing an error message for these cases, in order to avoid confusion.

Ah error message was include to avoid confusions.


Crash / Ticking block entity on older version

Mods:
Forge: <IP_ADDRESS>54
CyclopsCore: 1.4.0
CommonCapabilities: 2.4.2
Integrated Dynamics: 1.0.15
Integrated Dynamics Compat: 1.0.0
Integrated NBT: 1.0
JEI: <IP_ADDRESS>3
Placed an NBT Extractor and a Variable Store next to it -> Instant crash
Tried after updating Cyclops Core, Common Capabilities and Integrated Dynamics to newest versions and it worked. So its obviously a problem with outdated versions.
I'd suggest to specify a minimum version somewhere (don't know if thats done in Curse or the Mod?), so people don't get confused with it.
Crashlog

Hello again. I am not able to access your crashlog. Can you re-upload it? I think if I can see what is causing the issue, I can avoid having to manually test all versions to determine the minimal requirements.

I was able to reproduce the issue myself. I picked a random version of Integrated Dynamics afterwards, which is v1.1.2, and it worked. So I am going to use that as the minimal requirements.
Anyways, thanks for letting me know. :)


Redot engine website needs to add additional paths to fix documentation links

Tested versions
redoteengine.org
System information
Windows 11
Issue description
Following links are missing
https://redotengine.org/license (\about\faq.rst, about/complying_with_licenses.rst)
https://redotengine.org/community (\introduction\community.rst)
https://redotengine.org/contact (\about\faq.rst)
Steps to reproduce
Either pages have to be created to cater the needs of documentation or documentation has to be updated
Minimal reproduction project (MRP)
N.A

Why was this submitted to the engine?

Migrated the issue to right repo


docs: Dynamically render latest image tag in docs

This change displays the latest image tag for the function in the
function doc page. This is intended as a convenience for users
to discover the full tag of the latest function image.

Thanks for the quick fix


stashcache: Replace reference to condor service with systemd timer

Bring docs into line with changes merged in opensciencegrid/StashCache-Daemon#12

@jthiltges - what's the status of this one?

I think we're good to review and merge.

If I have an old version installed that still used condor to do the reporting, what do I need to do when upgrading?

@matyasselmeci - I think stopping the HTCondor service & disabling on boot is sufficient.  I think there's minimal overlap between "normal" HTCondor installs and StashCache instances.
There's no harm in having both running.
Probably worth noting in the release notes but I'm reluctant to have any technical mechanisms.

We don't have to explain mechanisms, just required user actions for everything to keep working as expected.

Hasn't the relevant version of StashCache already been released? Can we just merge this change?

sure.


profiles.json "startingDirectory" : "%SYSTEMDRIVE%" points to system32

Environment
Microsoft Windows [Version 10.0.18362.175]
Windows Terminal version (if applicable): latest from the store
Steps to reproduce

Modify any profile in profiles.json such that
"startingDirectory" : "%SYSTEMDRIVE%",
Launch the profile

Expected behavior
Starting dir is the system drive, e.g. C
Actual behavior
Starting dir is %SYSTEMDRIVE%\system32
https://i.imgur.com/hyT2wi7.png
Note
%systemroot% works fine, didn't check other env vars

This is just a quirk of how paths on Windows work! Here's something neat. %SYSTEMDRIVE% expands to C:, but...
C: is a "relative" path, it's just relative to "which folder is currently active on the C drive"... which is a behavior that dates back to DOS 2.0.
As an example, if I set my "active directory" on my E drive to E:\src, I can do this:
dir E:openconsole

and I'll get the response:
 Volume in drive E is Thorium
 Volume Serial Number is 1AF0-C31B

 Directory of e:\src\openconsole  <-- here!

Anyway, the way around this is to use %SYSTEMDRIVE%\ (which you'll need to properly escape for JSON) which makes it an absolute path anchored at your system drive.
We, of course, cannot change any of this. It's old Windows and DOS arcana that people love to rely on. It lets you do things like COPY A:bigfile C:bigfile2 without specifying directories, which can get long and annoying.


Getting started guide: Angular path returns 404.

From this guide https://angular.io/docs/js/latest/guide/setup.html, it uses 2.0.0-alpha.21 which is currently missing from https://code.angularjs.org/.
so both https://code.angularjs.org/2.0.0-alpha.21/angular2.dev.js (TypeScript) and https://code.angularjs.org/2.0.0-alpha.21/angular2.sfx.dev.js (ES5) returns 404.
(I'm not sure how code.angularjs.org is populated but at the time of writing this, there are alpha 19, 20, and 22 for Angular 2.)

It has to be updated to 'alpha.22'


Component Type Inconsistency between Development & Release Builds

[+] I have reviewed the documentation
[+] I have searched existing issues
[+] I am using the latest React Native version

In a development build of a React Native application when iterating over the children of a component you can access the displayName of the child to determine the tag name of the component.
Here's an example of filtering a component's children by their displayName value.
const pluckChildrenByName = (children, name, extendedProps) => {
    return React.Children.map((child) => {
        if( child.type.displayName == name ) {
            return React.cloneElement(child, { ...extendedProps });
        }

        return null;
    });
}

This works perfectly fine in the Development build, but when a release build is created the value of child.type.displayName is undefined.
Environment

Environment:
  OS: macOS High Sierra 10.13.2
  Node: 9.7.1
  Yarn: 1.5.1
  npm: 5.6.0
  Watchman: 4.9.0
  Xcode: Xcode 9.3 Build version 9E145
  Android Studio: 3.0 AI-171.4443003

Packages: (wanted => installed)
  react: ^16.3.0-alpha.1 => 16.3.0
  react-native: 0.55.0 => 0.55.0

Steps to Reproduce

The following example demonstrates the use case:
<Field>
    <Text>Text is not permitted inside Field and will be omitted.</Text>

    <Label>Username</Label>
    <TextInput placeholder="Username" />
</Field>

// Field.js
render() {
    return (
        <View>
            <View>
                { pluckChildrenByName(this.props.children, 'Label', { foo: 'bar' }) }
            </View>
            <View>
                { pluckChildrenByName(this.props.children, 'TextInput', { hello: 'world' }) }
            </View>
        </View>
    )
}

The result is that Field is able to build a containing JSX structure and pluck the desired children, rendering them where they should be.
Expected Behavior

The ability to differentiate between children reflective of their displayName / component name in both development and release builds.
Actual Behavior

In a development build the ability to differentiate between children reflective of their displayName works as expected and subsequently the desired functionality is achieved.
In a release build displayName on any given child is undefined and therefore fails, breaking the functionality entirely.

@react-native-bot The issue is reproducible on the latest version 0.55.0.

Temporary Solution
The value of a Component's displayName property is completely lost when building for release, a temporary solution is to manually define it as a static property of the Component.
The following is the snippet used to filter a Component's children in this example:
getChildrenByName(name) {
    return React.Children.toArray(this.props.children).filter((child, index) => {
        return child.type.displayName === name;
    })
}

Without the static definition

With the static definition
class FilteredComponent extends Component {

    static displayName = 'FilteredComponent';

    ...

<View style={ styles.component_container }>
    { getChildrenByName('FilteredComponent') }
</View>


Prepare FLAnimatedImage for rendering asynchronously

Based on https://github.com/kean/Nuke-AnimatedImage-Plugin/issues/5
We can't create FLAnimatedImage on the main thread, but we want to keep all the benefits of the #5.
When the image is displayed Nuke should:

Display poster image first (synchronously)
Create instance of FLAnimatedImage asynchronously (it would calculate delay times, etc).
Display FLAnimatedImage on the main thread

The delay in playback is unnoticeable, especially when scrolling collections of images. Overall, I think it's a good starting point for most users.

Closed by 84f4ca90b6d57755516f12033dd24cffccf7a086


Color Picker HEX value in lower case without number sign

Microsoft PowerToys version
0.60.1
Running as admin

[X] Yes

Area(s) with issue?
ColorPicker
Steps to reproduce
When you hover your mouse cursor over the color you would like to copy, the HEX value of the color appears and then copies in lowercase without number sign(#), although according to documentation the color should be displayed and copied in uppercase with number sign (https://docs.microsoft.com/en-us/windows/powertoys/color-picker).
Would be good to, at least, have an option to choose between uppercase and lowercase.
✔ Expected Behavior
Copy HEX color in uppercase with number sign.
❌ Actual Behavior
Copy HEX color in lowercase without number sign.
Other Software
No response

This is totally intentional. In the past, the copied value did include the #. However, a number of apps/guis only accept 6 characters input.  Compared to that, typing a # by hand is not an effort.
Uppercase/lowercase has also been changed once, after requests/discussion. It's possible that docs just haven't changed with it. Is this really an issue for some apps or programming languages though?

About # I understood, no complains there.
For uppercase/lowercase - well, it's not an actual issue, just, for example, for native Android development it is preferred to define all apps colors in uppercase. So it would really nice to add some sort of settings, where user can select do he wants to get colors in uppercase or lowercase. Converting lowercase to uppercase manually wouldn't be a big effort, but getting already what you want would be a good UX.

There's also issue #8305, but until now, that's only a concept...


Index falls below the first H1

          Currently in Potato Indexer 1.1.1 and Obsidian 1.4.16 index falls below the first H1

To replicate -
Create new Document
Name Document
Create H1 Headings
Generate index for current note
Index falls below first H1
First
Content Index

First
Second
Third

some content
Second
Some Content
Third
Some Content
It may have something to do with how Obsidian handles Titles
Since it automatically inserts the first H1 as the title does not show the first H1 and the first actual H1 is interpreted as the title.
I'm not familiar enough with typescript to make a pull request unfortunately but I think it is here. You can just seek first H1 nd then add index above
indexSvc.ts
  const lines = content.split("\n");
  let index = "\n## Content Index\n\n";
  const indexPos: number[] = [];

and also .editorconfig
# top-most EditorConfig file
root = true

Originally posted by @synj00 in https://github.com/LoyalPotato/potato-indexer/issues/3#issuecomment-1765010167

@synj00 In the latest release I've added a setting to disable this.
If there's any issue with it feel free to add another issue or comment here to re open the current issue.


Replace simulation subscriptions with queries

Fixes database errors related to subscriptions for simulation data
Use top-level plan page reactive expressions to fetch simulation results with regular GQL queries
Remove top-level resourceTypes query since it's slow

See: https://github.com/NASA-AMMOS/aerie/issues/605

Workaround fix for:

https://github.com/NASA-AMMOS/aerie/issues/595
https://github.com/NASA-AMMOS/aerie/issues/574
Still unclear why the database is choking for these subscriptions, will require further investigation

Big thanks to @mattdailis for starting this effort!

To test:

Open a new plan
Run simulation
Make sure simulation results are still showing up
Test external profiles are still showing up

@jeffpamer Good point let's hold off on https://github.com/NASA-AMMOS/aerie-ui/issues/343 this sprint just because there is some uncertainty with how we will report this. It will require a little more refactoring than I anticipated. It will still need to happen tho, but we can maybe revisit next sprint.


Tags not written out correctly in Parser.String()

Similar to #14 in Unify.String(), the tags are also showing an issue with spaces in Parser.String().
Current:
import "pe"
import "cuckoo"

global rule payload :tag1
{
...

Desired:
import "pe"
import "cuckoo"

global rule payload : tag1
{
...

Fixed in dbbc41bfe674c92e9bc1b4de945940075797c5e4, although I mistakenly referred to it as 15.  Let me know how to proceed if this is going to be an issue.


Issue #529: NotepadTest (C#) sample update

This code is not mine. Kope13 posted the change here:
https://github.com/Microsoft/WinAppDriver/issues/529#issuecomment-482426016
but it doesn't appear that anyone has checked it in yet (as far as I can tell)
This change works with Appium.Webdriver <IP_ADDRESS>-beta and Selenium 3.141

 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.:x: CreaturePrime sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Hi @CreaturePrime,
I'm trying to understand the need for this. Looking at #529, I can see that the original issue were that samples were not compiling, but this was before we updated our samples with the release of v1.1.1 preview.
Is this still the case for you with the most up to date master branch?
If so, can you reply to me on that thread and (1) let me know what Windows version you are running (Start -> Run -> "WinVer"), and (2) what exact compile error you are running into.
This commit is not able to compile on my Windows 1903 machine without having to make changes, so I will have to most likely have to reject this commit.
Thanks


Tests / Examples need to be created

I'd like to mirror the YNAB tests for now that live in the Ruby/JS sdks for consistency.
https://github.com/ynab/ynab-sdk-js/tree/master/test
https://github.com/ynab/ynab-sdk-ruby/tree/master/spec

Probably going to need to address #1 before this can really be done properly. I've started writing a couple tests as examples but I'm going to hold off on more until I make the mock. Talking with YNAB's dev team they seem interested in creating some kind of global mock, but I'm not sure if/when that might happen so considering using VCR or something else for now.


github webhook.js is hard-wired to openwhisk.ng.bluemix.net

if one wants to set up openwhisk on some other endpoint, the git webhook will not work
  var endpoint = 'openwhisk.ng.bluemix.net';

for one, this affects the IBM Bluemix OpenWhisk staging services.

related https://github.com/openwhisk/openwhisk/issues/731

This is fix now https://github.com/openwhisk/openwhisk-catalog/blob/master/packages/github/webhook.js#L34
cc @bjustin-ibm


update rubocop to ver0.49

Pull Request Test Coverage Report for Build 28

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 26.786%

Totals

Change from base Build 27:
0.0%

Covered Lines:
15

Relevant Lines:
56

💛  - Coveralls


Set network state to implemented when no services

Description
This PR fixes an issue where isolated networks without services created in version before 4.17 and in state 'Setup' could not be used even though they require no further implementation

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

implemented in an upgraded environment from 4.15 to 4.17 with a network in setup state
before: a new VM could not be started in the network (as it was in state 'Setup'
after a vm could be started in the network

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6838 (SL-JID-2535)

@vladimirpetrov As this is a SQL only change, other than what travis does the only testing on this that makes sense is the actual upgrade of a network in state "Setup" as described in the description ;) Can you do the honours?
PS don´t be fooled by the first commit, it is already on 4.17 and main:
https://github.com/apache/cloudstack/pull/6824
https://github.com/apache/cloudstack/commit/79e5e7fe36efd01b88267e048ea49303a32c0177

@blueorangutan package

@vladimirpetrov a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 4518

@blueorangutan test

@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-5186)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 41166 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6838-t5186-kvm-centos7.zip
Smoke tests completed. 103 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_02_upgrade_kubernetes_cluster
Failure
506.07
test_kubernetes_clusters.py


Release 0.6.0

@rultor release it, tag is 0.6.0.

@rultor release it, tag is 0.6.0.

@rafamizes OK, I will release it now. Please check the progress here

@rultor release it, tag is 0.6.0.

@rafamizes Done! FYI, the full log is here (took me 2min)


SetHoldings/CalculateOrderQuantity bug fixes + new tests

Replaces #317

I'm tempted to just pull in the negative order fee fix directly for now, while we work through some of these other comments: f22224db5fc020bd25ab678c77c5bb67d11ab9dd

@mchandschuh About pulling in the order fee fix alone, I'm afraid the frequency of "Insufficient margin" errors would be increased. I think it should be merged along with the new CalculateOrderQuantity.


Docker compose doesn't work with mqtt

The following configuration results in a tcp socket error for mqtt.
find3server:
  container_name: find3server
  restart: unless-stopped
  image: schollz/find3
  environment:
    - MQTT_ADMIN=admin
    - MQTT_PASS=mypass
    - MQTT_SERVER='localhost:1883'
  volumes:
    - /home/USER/find3server:/data
  ports:
    - "11883:1883"
    - "8003:8003"

I have no idea why. It should work. I think its a docker-compose error because regular Docker works just fine, e.g.:
docker run -p 1884:1883 -p 8005:8003 \
	-v /home/$USER/FIND_DATA:/data \
    -e MQTT_ADMIN=ADMIN \
    -e MQTT_PASS=PASSWORD \
    -e MQTT_SERVER='localhost:1883' \
	-e MQTT_EXTERNAL='your public IP' \
	-e MQTT_PORT=1884 \
	--name find3server -d -t schollz/find3

will work no problem.

Hi,
Test with this:
`version: '3'
networks:
find3-network:
driver: bridge
services:
find3server:
container_name: find3server
image: schollz/find3
restart: always
volumes:
- db:/var/lib/mysql
environment:
- MQTT_ADMIN=admin
- MQTT_PASS=mypass
- MQTT_SERVER='localhost:1883'
- MQTT_EXTERNAL='Put here your public IP'
- MQTT_PORT=1884
volumes:
- /home/CHANGE_HERE_FOR_YOUR_USER/find3server:/data
ports:
- "11883:1883"
- "8005:8003"
networks:
- find3-network`

Ran into this problem as well
2023-01-17 21:14:37 [WARN] [PID-16] server.go Run:39 Network Error : dial tcp: lookup tcp/1883': Servname not supported for ai_socktype

but the following works for me to get it to work

stop container in docker (compose)
rm container from docker
start the container, new settings are actually picked up


Freebsd support

This PR updates your "freebsd-support" branch to resolve a couple of minor things.  The travis-ci build was failing for your PR, so this aims to resolve that.

Use the "before" metaparameter for the relationship between the package and localtime file.  We don't use the a package on FreeBSD, so you previously had a selector within the file resource to set require to "undef" if FreeBSD.  The selector within a resource is against the Puppet Labs style guide, so it caused puppet-lint to fail (causing the travis-ci build to fail).  This fixes all that by using "before" on the package.  If the package is in the catalog, it'll set the relationship correctly.

Evaluate the package variable correctly.  You had "if $package != undef", but "$package" wasn't a valid variable.  saz is making reference to the package variable in params, so this PR updates your logic to evaluate if THAT variable is present.  In the case of FreeBSD, it's not, so the package resource is ignored (as we want).

Thanks for the input. This definitely makes more sense.


Inference failed due to missing posporcessing.json file

Dear author,
I find your paper very interesting and I'm new in the medical imaging field. I am trying to reproduce your model results for version 1 and version 2.
In version 1, I couldn't find the HD95 code, therefore I copied it from  here. It gives the wrong HD95 during evaluation, can you please provide this version 1.
In version 2, when I run the inference code it gives the error shown in the figure. To resolve this issue I run consolidate_postprocessing_simple.py to compute the postprocessing.json file. But this says fold_0, fold_1, fold_2, etc are missing. I trained the model for fold=0 as described in the instructions. Could you please see this?

Hi Mustansar,
Thanks for your interest in our paper.
In version 1, we used another way to calculate hd95 (which is now deprecated), and I recommend you use version 2 as it's written based on nnUNet configurations and settings. You have to specify the fold number during inference and use the following command to generate predictions for each fold:
CUDA_VISIBLE_DEVICES=0 vtunet_predict -i imagesTs -o inferTs/vtunet_tumor -m 3d_fullres -t 3 -f 0 -chk model_best -tr vtunetTrainerV2_vtunet_tumor
If you want to get consolidated results, then you have to train the model for fold=1, fold=2, and fold=3 and run consolidate_postprocessing_simple.py
Kind regards,
Himashi


Date and time formats need to work in French

Discrepancy on https://staging.covid-hcportal.cdssandbox.xyz/fr/outbreaks/98b6369c-18fa-42fd-b789-90bac001edd5/datetime#time-fields
Figma: https://www.figma.com/file/znPqLTQQDcUkaNjdDcShiz/Exposure-Notification-Health-Portal?node-id=9614%3A67549

Date and time posted in English format in French screens
No AM / PM in French

I am closing this issue since the rework for this functionality to be implemented will be quite intensive.