James Harvey - Development Secretary

Name: James Harvey

Development Secretary
Manifesto

Hiiii investors!! ^~^
I'm devsec for this year, and want to continue being devsec for next year :3
I've done a mediocre job of rewriting HackSoc's infrastructure in niche programming languages that will probably be irrelevant in 5 years. If elected next year, I pledge to write at least one README.md for one of these new pieces of infrastructure.

👉 👈

me


Use annotations instead or with the current system to parse the function's signature

Python 3.x has the annotations feature.
If you can support it instead of comments for python 3.x it will make the code much more readable.

It can be usefull now that pythran supports Python 3

Note that with Transonic, it's already possible to use type annotations to use Pythran.
You can put anything in python type annotations, in particular, types defined with typing or numpy or even strings (so you can use something like "float[:, :] or int[:, : ,:]", which works with Transonic).
However, there is only one "signature" with type annotations, so to support the equivalent of multi-signatures in Pythran (which is really useful), we need a type system more powerful than what exists today in Python. I think we would need something like
from pythran import Array, Union

U = Union[float, Array[float, "3d"]]

def my_func(arg: U):
    return arg**2

But this is bad because then the Pythran file cannot be run without Pythran.
it is much better to use from transonic import Array, Union, because transonic is a much smaller (pure Python) run-time dependency.

See #2251 for python annotation support in local variables.


prep_for_subm attempts to use instance not available

Ran prep_for_subm.sh against a west1 cluster. The machinetype chosen was m5n.large which does not exist in that region
If the matter is security related, please disclose it privately to the
Submariner Owners: https://github.com/orgs/submariner-io/teams/submariner-core
-->
What happened: Improper machine type used
What you expected to happen: Machine type that is available in that region should be used
How to reproduce it (as minimally and precisely as possible): Run prep_for_subm in west1
Anything else we need to know?:
Environment:

Submariner version (use subctl version):
Kubernetes version (use kubectl version):
Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Network plugin and version (if this is a network-related bug):
Others:

Thanks for testing Submariner, @cooktheryan!
The script deploys an m5n.large EC2 instance type by default for the Submariner gateway node. Alternatively, you can customize the AWS instance type by modifying the GW_INSTANCE_TYPE environment variable. This is documented here: https://submariner.io/getting-started/quickstart/openshift/aws/#prepare-aws-clusters-for-submariner
FYI, we are looking to replace the prep_for_subm script with something more robust we can maintain and support in 0.9 (https://github.com/submariner-io/submariner/issues/873), so there is no appetite to enhance this script.

Thanks for testing Submariner, @cooktheryan!
The script deploys an m5n.large EC2 instance type by default for the Submariner gateway node. Alternatively, you can customize the AWS instance type by modifying the GW_INSTANCE_TYPE environment variable. This is documented here: https://submariner.io/getting-started/quickstart/openshift/aws/#prepare-aws-clusters-for-submariner
FYI, we are looking to replace the prep_for_subm script with something more robust we can maintain and support in 0.9 (https://github.com/submariner-io/submariner/issues/873), so there is no appetite to enhance this script.

:+1: excited to follow the path from here

:+1: excited to follow the path from here


build-snapshot: User Agent parser.

Created smart component to parse the User Agent and get the partial browser version for the following inputs:

IE
Chrome
Firefox
Opera
Safari
Edge

Created package component
Created Browser as entity
Created factory for Browsers (should be improved)
Added utils for parser component

closing as out-of-dated


Awesome_Books_with-ES6

-Create a module directory for the dependencies.
-Create an entry point which is index.js.
-Import all the dependencies to the main file.
-Export all the methods and classes in the dependency file.
-Refactor all the classes and functions in alliance with ES6.
-Import Luxon external library for time and date.

-The instant time is working on my side maybe I used a Luxon library and it's in my node_modules file. I wonder may be its not working on your side because I ignored the node_modules?.
-Also the navigation is working fine on my machine.


[Feature Request] Proposal: Separate Dashboard with One-Click Update Button for Seamless Release Management

Hi,
I have a suggestion.
Is it possible to run the dashboard separately in its own container and include an "Update to the next release" button inside the dashboard? In the settings, there would be a field to input a Docker Hub API key, and the update process would be triggered by pressing the button. This would be very convenient as it eliminates the need to access the server for urgent updates.
Thank you!

Yeah, that would be great, but people use too different environments...
Probably we could provide a UI where you can put few things and it'd do the update for you tho via ssh :thinking:
like what we'll need basically:

ssh server\username\password (or key)
docker key from portal.devlike.pro
Folder where your docker-compose.yaml lives

It'll work only with docker-compose installations, however
https://waha.devlike.pro/docs/how-to/install/
Let's see what we can do, but not this year 100%, we need to work on engines and new features

Yeah, that would be great, but people use too different environments...
Probably we could provide a UI where you can put few things and it'd do the update for you tho via ssh 🤔 like what we'll need basically:

ssh server\username\password (or key)
docker key from portal.devlike.pro
Folder where your docker-compose.yaml lives

It'll work only with docker-compose installations, however https://waha.devlike.pro/docs/how-to/install/
Let's see what we can do, but not this year 100%, we need to work on engines and new features

That's exactly what I meant. It would be really great, and I'm looking forward to it!

will work on that https://github.com/devlikeapro/waha/issues/419


add some fns to help us setup tracing for unit tests and run async tests

I found that tracing information is very important in our test cases to help us debug, and for unit tests, using nomos-log brings more complexity, so I added those three functions for testing.

Codecov Report
Patch has no changes to coverable lines.
:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.

Files Changed
Coverage

tests/src/lib.rs
0.00%

tests/src/tests/happy.rs
0.00%

tests/src/tests/unhappy.rs
0.00%

:loudspeaker: Thoughts on this report? Let us know!.


Update preparing and installing macOS section

-Removed parts that are covered in Apple's documentation
-Removed outdated info
-Removed manually checking the certificate because Apple silicon Macs do so automatically without needing user input
-update first boot section
-remove rambly essay about how bad Apple is bc it doesn't really add any useful information

What is wrong with the alternative installation steps? It seems like a lot of potentially useful information to remove, judging from previous contributions to that section.
I know we discussed keeping the guide updated for the latest Apple releases, but I'm not sure everyone is off Intel Macs to warrant removing all relevant parts just yet. What do others think?
The rambly essay can be removed but reference it as an additional resource at least. There are significant privacy implications with using secure hardware, and those certain trade-offs ought to be better discussed and explained.

Problems in order:

recovery mode instructions don't work on Apple silicon. linked to the apple documentation so it stays updated and offers the best coverage for different Macs
link to old version of macOS in the App Store. linked to the documentation instead of referencing specific versions so it doesn't go out of date
link to third party and very outdated instructions on making a bootable USB, Apple provides instructions on making a bootable USB so I linked to that instead
list of download links for older versions of macOS, this duplicates the official documentation needlessly so I deleted it
no need to verify the signature with codesign since the secure enclave handles that for you
instructions that duplicate Apple's documentation for making installation USB needlessly, linked to that instead
instructions for creating an install image that the guide itself claims doesn't work so deleted it
target disk mode replaced by revive/restore, could link to that but it's brought up later in the firmware part so I thought it was pointless to talk about it in two places
recovery partition isn't needed + you need an internet connection now if you reinstall anyway, so might as well just use the network recovery
macOS is most secure running on native Apple silicon, this extends to VMs as well so I replaced VMware Fusion with UTM which is free and doesn't require you to give them payment info, a guide about privacy should avoid software that requires personal info to use imo
the guide itself says that disconnecting the network before install is pointless since the 2016 Touch Bar Macs started requiring online activation so removed that part
intel nvram clearing irrelevant on Apple silicon
replaced the XKCD comic with EFF link that is more detailed and we link to later in the guide

The essay is just highly overdramatic about a simple hardware identifier. You don't have to log in to your Apple ID, you don't have to give apple your payment info, you can just buy it in cash if you want. If it's a new Mac you don't even need to do the activation process anyway afaik it's only if you try to reinstall. I personally don't see any value in linking to it, it's mostly just trying to scare the reader into avoiding macOS which, why have that in a guide about macOS? It's really a lot of drama over nothing. I will add that this guide recommends like 10 different pieces of software that all 1. require payment information and 2. can see all of your network connections, so it seems strange to hyper fixate on this hardware identifier when those are much worse imo. Just my thoughts tho

Added back the threat model mention.

Let's put VMware back. I understand some might not like it because it is for-profit software, which is a valid criticism, but it also has a proven track record of reliability. It at least deserves a mention after the UTM app is recommended, perhaps with even more alternative virtualization software listed. We can raise a separate issue to address non-free recommendations throughout the guide as well.
I insist on keeping at least a link to the "iOS, The Future Of macOS, Freedom, Security And Privacy In An Increasingly Hostile Global Environment" article. Yes, is it hyperbolic, but for obvious reasons. That other software requires payment or installs invasive components does not diminish its value. A single unique identifier in the hands of a very capable adversary can pose more risk than a dozen held by an incapable one. As you said, Apple silicon is the way forward for security, but it does require some privacy sacrifice.
The article's virtues and thesis should are best discussed in its respective comment section though. We should at least make it discoverable so readers can make up their own mind, and an opinionated take on this topic inspires critical thinking vs Apple's dry whitepaper. Threat modeling should be its own detailed section (or even guide) - feel free to open an issue to track this work. I would be glad to take the lead on it.
Thanks again for the lively discussion and all the modernization contributions you have been making. Keep the PRs coming!

Alright I re added VMware with a link to their documentation on installing macOS and put the article in the additional resources section.

Yeah it’s nice to see updates here. Thanks also from my side @kimg45
That's not supposed to sound negative, but are you planning to keep the guide up to date than it was before? After all, it seems to be the most well-known macOS guide  @drduh

It's already merged, but what's wrong with the tools from https://eclecticlight.co/virtualisation-on-apple-silicon/ and Parallels?

Eclectic light company on their downloads page list a lot of stuff that seems to be for very old versions of macOS? And a lot of their software seems to be like security snake oil kinda stuff. Idk can re add it I guess. Parallels is good, needs a subscription and stuff tho.

Fair enough 🍺


B5 task

baker-id: 979
node-id: 1a6b93a31f77af09
screenshot: gen_keys.jpg, register_baker.jpg
account address: 3H1jFeYPTaNh2Wb9x8tdENG5bPQNG7rMzo5ToehUdJUMGR9ZNY
Transaction used to register baker: 89183fcfc7e2543c6d40beb43f3cd673ab0bb698a925072b0ef324397affb6f3
blockhash of block you produced: 123687c13e33c914f59eaba71b297a941d39bb8b101f7032738453b5cf928493
transaction ID that changes the reward account: a29d412abe00137fa59f11cf9b0b44d579f3409b67a02fc1a9469e21e63b5936
transaction ID that updated baker's keys: 3H1jFeYPTaNh2Wb9x8tdENG5bPQNG7rMzo5ToehUdJUMGR9ZNY

observations: the GUI run not exactly: "In testnet stake delegation queue, waiting time 2h0m". The time loop 2h0m --> 2h1m --> 2h0m. meanwhile, page "https://dashboard.testnet.concordium.com/" load normally.

confirmed:B5
address:3H1jFeYPTaNh2Wb9x8tdENG5bPQNG7rMzo5ToehUdJUMGR9ZNY

confirmed:B5
address:3H1jFeYPTaNh2Wb9x8tdENG5bPQNG7rMzo5ToehUdJUMGR9ZNY

Form Submitted at 22:42 UTC+7


Segmentation Fault Core dump

After  installing the environment，I run the run_seq2seq.py. When I load the model,Segmentation Fault Core dump occurs. My environment is pytorch 1.1.0, cuda 10.1, torch-vision 0.3.0. so why the core dump occurs?

Hi @jaytsien ,
Could the docker (https://github.com/microsoft/unilm#docker) successfully run the program?

I have no authority to install docker, my colleague also meet this problem. Can you show the environment requirements?

https://hub.docker.com/layers/pytorch/pytorch/1.1.0-cuda10.0-cudnn7.5-devel/images/sha256-1be771bff6d18ede7d98c171c577ae8fcbaf6f53b59a3e5fe5cee9e36e188fbc


Add a migration page

This PR is providing a page that will help the user to migrate a Grafana dashboard to a perses one.
~I didn't find (yet) how to contain the JSONEditor it has an infinite height and in this context I would prefer it's not. I tried to wrap it in a Box with a limited height but it didn't work. I will find the way tomorrow.~
I took the opportunity to remove the folder sample_data, hope nobody minds about this cleanup.

~It also misses the way to add the dashboard to a project as well.~
I'm wondering if it would be interesting to have the preview of the dashboard also.

And thanks @AntoineThebaud for the fix around cue.
https://user-images.githubusercontent.com/4548045/202242821-342f2a30-3035-4e02-864a-e48c8382713f.mov

Yeah showing a preview of the converted dashboard would be neat.

Alright :) I think I will adresse that in another PR.
I added a way to save a migrated dashboard.
It looks like this now:

Looks good! The only thing I noticed is that the package-lock.json changed? I wouldn't expect there to be changes to that since none of the package.json files changed. 🤔

yeah I think I messed up this file when I added the dependency @mui/labs and then rollback. It's now fixed there is no change in the file package-lock.json.
Good catch thanks @LukeTillman


missing scope error

There may be a problem with the API
Traceback (most recent call last):
 [...]
  File "/usr/local/lib/python2.7/site-packages/slacker/__init__.py", line 191, in list
    return self.get('users.list', params={'presence': int(presence)})
  File "/usr/local/lib/python2.7/site-packages/slacker/__init__.py", line 117, in get
    api, **kwargs
  File "/usr/local/lib/python2.7/site-packages/slacker/__init__.py", line 99, in _request
    raise Error(response.error)
slacker.Error: missing_scope

instead of being so generic in providing auth accesses (weaker/loose security), we can be very specific in auth access needs and provide ONLY the accesses that are needed for the backups (this is much safer)
so all we need are to provide these accesses (easily done thru Slack API's OAuth's Scopes part of the page) :
channels:history
channels:read
groups:history
groups:read
im:history
im:read
incidentally, the "missing_scope" error msg's super short text's problem has it's roots in "__ init __.BaseAPI()", wherein currently on an error, only the error text is shown:
e.g. "slacker.Error: missing_scope"
which hides the detailed reasons for the error, 'coz of usage of just "error.text", instead of using "error.raw", in line # 112
this change has been implemented in these lines in my below forked repo:
https://github.com/fixbugsnow/slacker/blob/master/slacker/__init__.py#L102-L112
making this change will show the full error details, so the users knows all the reasons behind the error, just like Slack's API "Tester" shows the detailed results for errors:
e.g. slacker.Error: '{"ok": false,"error": "missing_scope","needed": "users:read","provided": "admin,identify"}
pgm flow happens (and still will happen) like so:
Traceback (most recent call last):
* * *
  File "c:\program files (x86)\python37\lib\site-packages\slacker\__init__.py", line 332, in list
    return self.get('users.list', params={'presence': int(presence)})
  File "c:\program files (x86)\python37\lib\site-packages\slacker\__init__.py", line 121, in get
    api, **kwargs
  File "c:\program files (x86)\python37\lib\site-packages\slacker\__init__.py", line 103, in _request
    raise Error(response.raw)
slacker.Error: {"ok":false,"error":"missing_scope","needed":"users:read","provided":"admin,identify"}

have requested for this change to be implemented in the original repo (https://github.com/os/slacker), for benefit of users of that repo and it's forks


Styles not applied when using TouchableHighlight from 'react-native-gesture-handler'

Hey there,
I'm using react-native-reanimated and react-native-gesture-handler to create some animations.
When using TouchableHighlight (not as string) from react-native-gesture-handler, the styles don't get applied, however when I use TouchableOpacity as a string, it works fine.
Try this:
import { styled } from '@styled/index';
import { View } from 'react-native';
import { TouchableHighlight } from 'react-native-gesture-handler';
const Container = styled('View', {
  flex: 1,
});
const Touchable = styled('TouchableHighlight', {
  flex: 1,
  backgroundColor: 'red',
});
// styles don't get applied to Touchable2
const Touchable2 = styled(TouchableHighlight, {
  // from react-native-gesture-handler
  flex: 1,
  backgroundColor: 'yellow',
});

export default function Test() {
  return (
    <Container>
      <Touchable>
        <View />
      </Touchable>
      <Touchable2>
        <View />
      </Touchable2>
    </Container>
  );
}

Hi @emroot 👋🏻
Unfortunately I'm not able to reproduce this issue.
Can you provide a Github repo with a reproducible example?

Closing this for not being able to reproduce the issue.


[Improvement] Set opacity to 0 on all elements with out classes when media is set to prefers-reduced-motion

The intention of "-Out" classes (fadeOut, zoomOut, slideOut, rollOut, and the like) is to stylize the disappearance of the element which uses this variation of "-Out" class. However, for browsers that have set prefers-reduced-motion, all these animations are not executed. Hence, during the instances that these elements need to disappear from the UI (opacity set to 0), they do not disappear as intended.
I found this issue while testing my website on Firefox v65 and Chrome Canary v74 on Windows 10 while I was checking for cross-browser compatibility.
To address this, I highly suggest addressing it like this:
@media (prefers-reduced-motion) {
  .fadeOut,
  .zoomOut,
  .slideOut,
  .rollOut {
    opacity: 0;
  }
}

If you have a better approach, just ignore mine, but kindly share in this thread too. Thank you.

Hello, @jadurani thank you for your suggestion, I will review this one together with @eltonmesquita and we'll get back to you... ;) what can you say about this @daneden

You clearly have a point. I'll take some time and review your request and its side-effects. I'll keep this open as a referral to the request.

@WarenGonzaga , would you mind to PR this into the dev branch?

@eltonmesquita just set all the animation with out classes to 0?

Yep.  I couldn't think in a scenario where this could be a problem, do you?

@eltonmesquita how about something like slideOutLeft is this included or just those classes with Out only?


Flutter web gives back a blank page

The doc on how to deploy to web is a bit unclear. I have built the project, deployed it in Firebase and when hitting the provided link, it opens a blank page (chrome and safari).
Here are the steps I followed:
1- Create web app in firebase and created as specified in the doc a firebase-config.js in the web directory with the output of the web project creation (without script tags).
2- flutter build web
3- firebase init providing as the location of the index file: build/web, selected also single page
firebase,json was created:
{
"hosting": {
"public": "build/web",
"ignore": [
"firebase.json",
"/.*",
"/node_modules/"
],
"rewrites": [
{
"source": "",
"destination": "/index.html"
}
]
}
}
4- firebase deploy
I can see 36 files deployed, I hit the URL: https://flyerchat-c8d43.web.app/ and it just displays a white page
Output of flutter doctor:
`Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.5.1, on macOS 11.6 20G165 darwin-x64, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.2)
[✓] VS Code (version 1.60.1)
[✓] Connected device (2 available)
• No issues found!`

Hey @appdevingenio

The doc on how to deploy to web is a bit unclear.

What doc are you referring to? We don't have any deployment documentation. Example is working in development mode and it is not supposed to be deployed. Anything else is on developer's and Flutter side, creating project, adding Firebase dependencies, set up, building, deploying it, not the part of this package. I believe you will need to check out Flutter and https://firebase.flutter.dev documentation to find out how to deploy a project correctly.

Hey @appdevingenio

The doc on how to deploy to web is a bit unclear.

What doc are you referring to? We don't have any deployment documentation. Example is working in development mode and it is not supposed to be deployed. Anything else is on developer's and Flutter side, creating project, adding Firebase dependencies, set up, building, deploying it, not the part of this package. I believe you will need to check out Flutter and https://firebase.flutter.dev documentation to find out how to deploy a project correctly.

I was a bit unclear. I have deployed Flutter web apps in the past in Firebase. Deployment works fine BUT the page that comes back is blank. There are instructions in the project for the config file, which I have also followed. Locally the config file did not work as expected throwing errors, I had to copy the code (firebase config) in the index file and I got the same exact result than deploying onto Firebase: a blank page.
So maybe the starting point is local deployment in dev mode which did not work following the instructions. I got the app to work fine on iOS with Firebase.


feat: add unit support of TreeResponse as one of the contents in MultiModalResponse

Fixes #6132

Description of what you did:

My PR is a:

[ ] 💥 Breaking change
[ ] 🐛 Bug fix
[ ] 💅 Enhancement
[x] 🚀 New feature

Please confirm that your PR fulfills these requirements

[x] Multiple commits are squashed into one commit.
[x] The commit message follows Conventional Commits, which allows us to autogenerate release notes; e.g. fix(plugins/plugin-k8s): fixed annoying bugs
[x] All npm dependencies are pinned.

To clarify for future reference: this PR adds core support for a TreeResponse model and corresponding view (currently using only PatternFly). Then, for test coverage, it introduces the command shown in the above screenshot. This command is owned by plugin-client-test.


Error in contrib.redirects import if the Excel file has lots of columns

Issue Summary
If you try to import the redirects from an excel file which has more columns than the number of import formats, and the relevant data is actually in one of these "surplus" columns, the form will be always invalid.
Technical details
The culprit is in https://github.com/wagtail/wagtail/blob/main/wagtail/contrib/redirects/views.py line 232:
form = ConfirmImportForm(
        DEFAULT_FORMATS, request.POST or None, request.FILES or None, **form_kwargs
)

The first parameter (DEFAULT_FORMATS) is the bogus one here since it should contain the dataset.headers, or at least a bit larger list of strings to be able to validate the values from request.POST.

fixed via https://github.com/wagtail/wagtail/pull/8814  & 649a8623a0449bad6e935cff73fae36a6a20c2e8


Possible TS 4.9 issue with changes to inference

Ref:

https://github.com/microsoft/TypeScript/pull/50831
https://github.com/microsoft/TypeScript/issues/50870

Resolved via #587 .


[Enhancement][DO NOT MERGE] Enhancement/optimize tablet mgr db lock

Why I'm doing:
What I'm doing:
Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[x] Yes, this PR will result in a change in behavior.
[ ] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.23 out of 24 committers have signed the CLA.:white_check_mark: meegoo:white_check_mark: luohaha:white_check_mark: dirtysalt:white_check_mark: decster:white_check_mark: zaorangyang:white_check_mark: amber-create:white_check_mark: sduzh:white_check_mark: evelynzhaojie:white_check_mark: Astralidea:white_check_mark: sevev:white_check_mark: packy92:white_check_mark: andyziye:white_check_mark: hellolilyliuyi:white_check_mark: mofeiatwork:white_check_mark: kevincai:white_check_mark: wyb:white_check_mark: DanRoscigno:white_check_mark: rickif:white_check_mark: stdpain:white_check_mark: letian-jiang:white_check_mark: stephen-shelby:white_check_mark: trueeyu:white_check_mark: GavinMar:x: zenoyangYou have signed the CLA already but the status is still pending? Let us recheck it.


[ssr] verify server rendering

My understanding is this should just work, but I haven't verified/explored it yet. Would like to launch v1.0.0 with the ssr story figured out.
https://twitter.com/poke707/status/845252764183683072
https://twitter.com/rauchg/status/845372211321982976

Forgive me if I'm wrong about this, but isn't vx-demo already running server side?
Doesn't Next already do server side rendering as just part of it's process?

but isn't vx-demo already running server side?

yup!

doesn't Next already do server side rendering as just part of it's process?

also yup!
there was a react error throwing (maybe it was from a previous version of Next) saying path attributes we're inconsistent. It doesn't look like it does it anymore, which is great.
I'd like to verify that it works with other SSR projects like https://github.com/airbnb/hypernova for example. I just haven't gotten around to it yet.

Newest Version of Next "next": "^2.3.1" throws following Warning:
warning.js?72da31c:36Warning: React attempted to reuse markup in a container but
the checksum was invalid.
This generally means that you are using server rendering and the markup generated
on the server was not what the client was expecting.
React injected new markup to compensate which works but you have lost many of the
benefits of server rendering.
Instead, figure out why the markup being generated is different on the client or server:
 (client) ="vx-bar" x="13" y="0" width="8" height=
 (server) ="vx-bar" x="13" y="27" width="8" height

So its not properly rendering serverside yet.

Yeah something is is up with 2.3.1. vx-demo.now.sh is using<EMAIL_ADDRESS>which appears to work without warning right now. I need to dive into the changelog and see what's up.

Hi, I would like to dig into this, is there a "must have" checklist to consider?

Seems like this is fine. No reports of problems so far. Going to close for now. Feel free to open a new issue if you run into problems with vx and ssr.


What does Halide generate when you have complex ops in your program   - e.g. e^x, sin, etc.

Complex operations exist for existing C functions, and are represented in HalideIR as a "Call."
For example, for Call* op, op->name is "sin_f32" when using sin(x).
Other complex ops available are (existing in IROperator.h):
sin, asin, sinh, asinh
cos, acos, cosh, acosh
tan, atan, atan2, tanh, atanh,
sqrt, hypot,
exp, pow, log, erf
fast_inverse, fast_inverse_sqrt
floor, ceil, round, trunc
lerp, popcount, count_leading_zeros, count_trailing_zeros


Missing string for translation of "your project needs a name"

The string for "your project needs a name" is missing.
(same in makecode.mindstorms.com)

fixed in beta


Feature/postgresql conf

Notes on consistent use of Ambari for configuring parameters.

Thanks, Jane - Can you squash these commits into a single one for the PR?  Not sure if there's a Jira associated with this, but if there is please mention it as the beginning of the commit message.


change default index for min/max to be a valid index

Address https://github.com/torch/cutorch/issues/331 , which is root cause of https://github.com/torch/cunn/issues/210

Note: verified behavior in base torch7, and it matches that in this PR.
Test on base torch7:
require 'torch'

a = torch.Tensor(10, 10):fill(-1/0)
m, i = a:max(1)
print('i[1][1]', i[1][1])

require 'nn'

net = nn.Max(1)

for it=1,3 do
  out = net:forward(a)
  net:backward(a, torch.Tensor(10):uniform())
end

Output:
i[1][1] 1

Thanks Hugh!


LoggingRow: InsertDate or UpdateDate for particular TimeZone

Is it possible to set value of LoggingRow: InsertDate or UpdateDate for particular TimeZone?
As Server may be in different country on which application is running.
Even after setting culture info in web.config, it is taking local server time only.
Similarly how to set value of DeleteDate for particular TimeZone, if entity is implementing
IIsActiveDeletedRow interface?
regards
shailesh

Add DateTimeKind(DateTimeKind.Utc). For different time zone, you need to write your own log behavior, and disable default one.

Ok, I will try.
Thanks for your Help.
regards
Shailesh


Cadl Validation update for renaming cadl to typeSpec

Fixed in https://dev.azure.com/devdiv/DevDiv/_git/openapi-alps/pullrequest/459647


style[next]: standardize error messages.

Description

add style guide to the coding guidelines
fix existing error messages in next
deal with ensuing qa errorrs

Requirements

[X] All fixes and/or new features come with corresponding tests.
[X] Important design decisions have been documented in the approriate ADR inside the docs/development/ADRs/ folder.

I only reviewed the changes in the coding conventions document and they look good to me (just have some style suggestions typo fixes).
As an additional comment (probably outside of the scope of this PR), I think we should look into refactor a bit our code to add more custom exceptions in the places where it makes sense and also to make sure we catch internal exceptions and wrapped them into informative DSLErrors in all places visible to users.

Out of scope for this PR but not for the project. Expect another PR along those lines.


Swarm/docker version conflict

Yet another issue with [Jenkins Docker plugin] (https://wiki.jenkins-ci.org/display/JENKINS/Docker+Plugin). The version number in /version is the swarm version number where the docker plugin expect it to be the docker version number. It is possible to set the swarm version to SwarmVersion and pass-through the docker version.
Docker plugin exception
Caused by: java.lang.NumberFormatException: For input string: "swarm/0"
at java.lang.NumberFormatException.forInputString(NumberFormatException.java:48)
at java.lang.Integer.parseInt(Integer.java:449)
at java.lang.Integer.parseInt(Integer.java:499)
at com.nirima.docker.client.model.Version.getVersionComponents(Version.java:45)
at com.nirima.jenkins.plugins.docker.DockerCloud$DescriptorImpl.doTestConnection(DockerCloud.java:353)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
at java.lang.reflect.Method.invoke(Method.java:597)
at org.kohsuke.stapler.Function$InstanceFunction.invoke(Function.java:298)
at org.kohsuke.stapler.Function.bindAndInvoke(Function.java:161)
at org.kohsuke.stapler.Function.bindAndInvokeAndServeResponse(Function.java:96)
at org.kohsuke.stapler.MetaClass$1.doDispatch(MetaClass.java:121)
at org.kohsuke.stapler.NameBasedDispatcher.dispatch(NameBasedDispatcher.java:53)
at org.kohsuke.stapler.Stapler.tryInvoke(Stapler.java:745)
... 56 more

I don't think it feasible to pass-through the docker version.
Firstly, the docker servers might not be running the same version.
Secondly, the swarm API is not exactly the same as the docker API. Some things are currently missing, and others are implemented differently (docker info responds with a list of nodes when run against swarm, docker ps prefixes the node names with the node ID).
Both these things might break a client if they receive something they weren't expecting.

I agree with @phemmer it's not really possible to put the docker version.
@alktlr does the plugin uses the version in anyway ?
It's crashing because the current version is swarm/0.0.1 (not a number)
Docker can have version 1.4.1-dev and the plugin isn't crashing.
What about we put 0.0.1 (swarm) as version ? I'm pretty sure it won't crash.
Can you test that ?

I have looked at the code and 1.4.1-dev would also fail. It creates an array split by dots and parses each element to an int. I use 1.4.1.
public int[] getVersionComponents() {
String[] v = StringUtils.split(getVersion(), ".");
int[] result = new int[v.length];
for(int i=0; i<v.length;i++) {
    result[i] = Integer.parseInt(v[i]);
}
return result;
}

The docker plugin will only accept version 1 or higher.
if( version.getVersionComponents()[0] < 1 )
                return FormValidation.error("Docker host is " + version.getVersion() + " which is not supported.");

The good news is that the version verification step can easily be bypassed by saving configuration without testing the connection. But then ran into a not implemented feature. (not to hijack this issue)
level=info msg="GET /images/json?filter=<docker image>&all=true"
level=error msg="Not supported in clustering mode."

ok, I believe we should fix the plugin itself, because if it doesn't work with 1.4.1-dev it means it only works with stable versions, not master.
Anyway, yes, swarm doesn't support docker images
We will add it, we just need to figure out how to do it. (I'll make a proposal soon)

@alktlr see #249

I'm closing this one. Feel free to comment in #249


Dropping OwnedByAnnotation from VMIs

What this PR does / why we need it:
Removing OwnedByAnnotation from VMIs.
Special notes for your reviewer:
This PR complements the alignments with Kubernetes that are introduced by PRs #1641 and #1665. This time we target  the OwnedByAnnotation  (see Roman's suggestion).
Release note:

None

@rmohr , can you please take a look and tell if we need to preserve handoverExpectations?

/cc @rmohr

@rmohr thanks, so I removed handoverExpectations and it all seems to work

Is this ready to be merged?

I believe it is, yes

ci test please

ci test please

This one should go in before #1201, since it was there first, so that this does not need another rebase. Let's wait for CI.

for reference: https://jenkins.ovirt.org/job/kubevirt_kubevirt_standard-check-pr/2607/artifact/ci_build_summary.html

ci test please


[Publication][Mobile view]: Unpublish request form for chat folder is closed in a few seconds

EPAM AI DIAL chat version
0.13.0
How to reproduce
Preconditions:
there is folder published inside Organization
Steps:

user open dial with mobile device
user click on 3 dots for published folder and select Unpublish option

Actual result
Unpublish modal form appears for a few seconds and then closes
https://github.com/epam/ai-dial-chat/assets/77381071/e6ea7623-52e9-43bd-abf9-710fb73ca43e
Expected result
Unpublish for should be displayed until user submit request , click on x- button to close or click to other area on the screen
Confidential information

[X] I confirm that do not share any confidential information

Verified on stg - fixed


fix: pass through report name, issue title, and search query when report is empty

Pull Request

Proposed Changes

Main goal: Ensure that report name, issue title, and search query the pass through properly when the search query results are empty.
Minor addition: I also improved the readability of the write_to_markdown function calls by using named parameters instead of positional parameters.
Example of markdown report before and after this code change:
Before:
#

no issues found for the given search criteria

_This report was generated with the [Issue Metrics Action](https://github.com/github/issue-metrics)_

After:
# Issue Metrics

no issues found for the given search criteria

_This report was generated with the [Issue Metrics Action](https://github.com/github/issue-metrics)_
Search query used to find these items: `repo:github/issue-metrics is:issue created:2024-10-01..2024-10-14`

Readiness Checklist
Author/Contributor

[x] If documentation is needed for this change, has that been included in this pull request
[x] run make lint and fix any issues that you have introduced
[x] run make test and ensure you have test coverage for the lines you are introducing
[x] If publishing new data to the public (scorecards, security scan results, code quality results, live dashboards, etc.), please request review from @jeffrey-luszcz

Reviewer

[x] Label as either fix, documentation, enhancement, infrastructure, maintenance, or breaking

Looking at testing now

No test updates as we don't test the main function for this app. Nothing inside the markdown writer changed so I wouldn't update those.


board: google_dragonclaw: add supported features

Add the list of supported features to the google_dragonclaw board definition.
It allows running more tests.

So there is no need to add google_dragonclaw board to the tests?

So there is no need to add google_dragonclaw board to the tests?

There is :-)


Testing feature

@fatmahussein  and @Armynerh implemented the following requirements:
We set up the React Testing Library.
We followed the instructions in the official documentation.
We wrote unit tests for the files operate.js and calculate.js using Jest.
We created unit tests for all React components:
We used Jest snapshots to test the components.
We used the React Testing Library to simulate user interaction.
We ran full tests suite using Jest. All tests pass.

Hi @medaminedev66
You gave a second opinion on our project for the math magician test. I would love it if you could please help us take a look at it and tell us if it’s okay to ask for a final review. the changes required have been implemented. Thank you

Hi @medaminedev66 You gave a second opinion on our project for the math magician test. I would love it if you could please help us take a look at it and tell us if it’s okay to ask for a final review. the changes required have been implemented. Thank you

Hello @fatmahussein,
I apologize for overlooking your previous mention. I'm sorry to inform you that there is files that still requires testing: for example fetchQuote.jsx. Could you please include test cases for this file as well? Your attention to this matter is greatly appreciated.


[flash_ctrl] Revert back to using per-bank, dedicated scrambling modules

An inspection of the DV issues arising from the change to share a single scrambling module between Flash banks revealed that there are indeed some corner cases that are not properly handled by the current design. Fixing those corner cases is tricky and may lead to further issues.
For this reason, this commit reverts the following commits for the time being:

[doc,flash_ctrl] Fix inconsistency and typo in documentation 3a35e26c7a1e6e26aa38f4ae899670274f69b1d1

[flash_ctrl] Use redundant arbiters in scrambling module a63c2d9a7abeef6411b9746f7540f444217aefa4

[flash_ctrl] Share flash_scrambling module among banks c95dc32e6235cd0dd3beaee313dc54e25cebd8b4

This is related to lowRISC/OpenTitan#22089.

Closing as we managed to fix the corner cases (an actual RTL bug).


Keybase not using User-Agent to scan website proofs; being blocked

According to one of my website hosts, Keybase does not send a User-Agent id when re-scanning for domain identities. See the server log below:
--------------------------------------------------------
<IP_ADDRESS> - - [25/Sep/2017:07:21:55 +0100] "GET /keybase.txt HTTP/1.1" 200 2941 "-" "keybase-proofs/2.1.17"
<IP_ADDRESS> - - [25/Sep/2017:08:45:42 +0100] "GET / HTTP/1.1" 404 - "-" "-"
<IP_ADDRESS> - - [25/Sep/2017:09:14:34 +0100] "GET / HTTP/1.1" 404 - "-" "-"
--------------------------------------------------------

As a result, this host - and possibly others - block the Keybase IP as potentially malicious. Should this be fixed?

keybase-proofs/2.1.17
On Mon, Sep 25, 2017 at 4:52 AM James Bridle<EMAIL_ADDRESS>wrote:

According to one of my website hosts, Keybase does not send a User-Agent
id when re-scanning for domain identities. See the server log below:

<IP_ADDRESS> - - [25/Sep/2017:07:21:55 +0100] "GET /keybase.txt HTTP/1.1" 200 2941 "-" "keybase-proofs/2.1.17"
<IP_ADDRESS> - - [25/Sep/2017:08:45:42 +0100] "GET / HTTP/1.1" 404 - "-" "-"
<IP_ADDRESS> - - [25/Sep/2017:09:14:34 +0100] "GET / HTTP/1.1" 404 - "-" "-"
As a result, this host - and possibly others - block the Keybase IP as
potentially malicious. Should this be fixed?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/8636, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA05_wCKuj07y1_gTrGPr2AwHMGnx58Gks5sl2m_gaJpZM4PiZW1
.

Yes, that User-Agent is sent the first time, but not on subsequent checks, so the block in initiated...

I am not sure what that other request is.
But this seems like a ineffective security policy.  Attackers will just lie
about their user agent.
On Mon, Sep 25, 2017 at 5:15 AM James Bridle<EMAIL_ADDRESS>wrote:

Yes, that User-Agent is sent the first time, but not on subsequent checks,
so the block in initiated...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/8636#issuecomment-331823853,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA05_6Q-JnfTrC2G2rvtRHNo1AtvhMMwks5sl29HgaJpZM4PiZW1
.

Not here to defend this as a security policy, merely relating that this is this host's policy, and as a result I can't authenticate my domain.

I have a guess as to where this might be coming from, but I'm not sure.  I'll push something later today, hopefully, but that won't affect people who have written down (silly, in my opinion) blacklisting rules for our IP. Thanks for your feedback.


feat: replace Docker references by generic OCI

Remove assumptions of the underlying container technology. The specific implementation should be part of the context or the SBOM

@ashsong-nv is there anything missing in this PR?
thanks

I compared accuracy between the original prompt and the requested changes and didn't see any significant changes in our test set.


[new feature] add shenyu can update local data

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #1971 (feb12fa) into master (786d7c9) will decrease coverage by 0.89%.
The diff coverage is 7.69%.

@@             Coverage Diff              @@
##             master    #1971      +/-   ##
============================================
- Coverage     63.18%   62.28%   -0.90%
+ Complexity     3962     3946      -16
============================================
  Files           578      580       +2
  Lines         16966    17080     +114
  Branches       1623     1638      +15
============================================
- Hits          10720    10639      -81
- Misses         5395     5587     +192
- Partials        851      854       +3

Impacted Files
Coverage Δ

.../shenyu/web/configuration/ShenyuConfiguration.java
0.00% <0.00%> (ø)

...pache/shenyu/web/filter/LocalDispatcherFilter.java
0.00% <0.00%> (ø)

...apache/shenyu/web/controller/PluginController.java
8.25% <8.25%> (ø)

...enyu/client/tars/TarsServiceBeanPostProcessor.java
26.47% <0.00%> (-63.24%)
:arrow_down:

...ache/shenyu/client/tars/common/dto/TarsRpcExt.java
0.00% <0.00%> (-30.77%)
:arrow_down:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
64.70% <0.00%> (-29.42%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
63.10% <0.00%> (-18.45%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
56.84% <0.00%> (-6.32%)
:arrow_down:

...henyu/plugin/httpclient/NettyHttpClientPlugin.java
58.00% <0.00%> (-2.00%)
:arrow_down:

... and 8 more

Continue to review full report at Codecov.

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


btDrops dosnt work with Vorkath

i been trying to generate loot tab form vorkath but unsucessfully the btcat works fine

Thanks I meant to look into this yesterday but forgot, Vorkath is
interesting as I am pretty sure that is one I specifically tested with
which means there may have been some change to the API.
On Wed, 6 Nov 2024, 14:18 shibaaa234, @.***> wrote:

i been trying to generate loot tab form vorkath but unsucessfully the
btcat works fine
—
Reply to this email directly, view it on GitHub
https://github.com/MitchBarnett/wiki-bank-tag-integration/issues/23, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABXIRZMY7XTZSYAG4B7JBBTZ7IQLRAVCNFSM6AAAAABRI6H7VSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGYZTQMRSGM3DGNY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I know i tested with Vorrath as i already have a tab for him

but its not working now


The list under "My APIs" is blank in the "msal-react-spa" when trying to add permissions

Issue

Please provide us with the following information:

This issue is for the sample

    - [ ] 1-1) Sign-in with Azure AD
    - [ ] 1-2) Sign-in with Azure AD B2C
    - [ ] 2-1) Acquire a Token and call Microsoft Graph
    - [X] 3-1) Protect and call a web API on Azure AD
    - [ ] 3-2) Protect and call a web API on Azure AD B2C
    - [ ] 4-1) Deploy to Azure Storage and App Service
    - [ ] 4-2) Deploy to Azure Static App Service
    - [ ] 5-1) Call a web API using App Roles
    - [ ] 5-2) Call a web API using Security Groups
    - [ ] 6-1) Call Microsoft Graph using on-behalf-of flow
    - [ ] 6-3) Call a web API using Conditional Access Auth Context
    - [ ] 6-4) Sign-in with Hybrid SPA flow

This issue is for a

    - [ ] bug report -> please search issues before submitting
    - [ ] question
    - [ ] feature request
    - [X] documentation issue or request

Minimal steps to reproduce

You need only follow the instructions.  Just don't use any existing registrations that you already created because you probably already own them.  When trying to add a permission in Step 7 of "Register the client app (msal-react-spa)" .  The list under "My APIs" is BLANK.

Any log messages given by the failure

n/a

Expected/desired behavior

When trying to add permissions to the "msal-react-spa", the "My APIs" list should contain "msal-node-api" application.

Library version

n/a

Browser and version

Chrome, Edge, Firefox, Safari? Chrome, but doesn't matter in this case.

Mention any other details that might be useful

The fix is to go to the "msal-node-api" application and add yourself as an owner of that api registration.  Afterwards, in the "msal-react-spa" the list of "My APIs" will contain the "msal-node-api" application.

You should find it under APIs by Your Organisation.


sugarorm can't work for android 6.0

When I use SugarOrm in android 6.0, it give a no such table exception report, android 4.x works well. If sugar can't create tables in android 6.0 automatically? And there is no way for check table is exist.

@JackLauu can you give more information about the error? Could you show me the entire stack trace?

Caused by: android.database.sqlite.SQLiteException: no such table: BOOKMARK (code 1): , while compiling: SELECT * FROM BOOKMARK WHERE BOOK_ID = ?
at android.database.sqlite.SQLiteConnection.nativePrepareStatement(Native Method)
at android.database.sqlite.SQLiteConnection.acquirePreparedStatement(SQLiteConnection.java:889)
at android.database.sqlite.SQLiteConnection.prepare(SQLiteConnection.java:500)
at android.database.sqlite.SQLiteSession.prepare(SQLiteSession.java:588)
at android.database.sqlite.SQLiteProgram.(SQLiteProgram.java:58)
at android.database.sqlite.SQLiteQuery.(SQLiteQuery.java:37)
at android.database.sqlite.SQLiteDirectCursorDriver.query(SQLiteDirectCursorDriver.java:44)
at android.database.sqlite.SQLiteDatabase.rawQueryWithFactory(SQLiteDatabase.java:1316)
at android.database.sqlite.SQLiteDatabase.queryWithFactory(SQLiteDatabase.java:1163)
at android.database.sqlite.SQLiteDatabase.query(SQLiteDatabase.java:1034)
at android.database.sqlite.SQLiteDatabase.query(SQLiteDatabase.java:1240)
at com.orm.SugarRecord.find(SugarRecord.java:201)
at com.orm.SugarRecord.find(SugarRecord.java:187)
at org.godfootsteps.thechurchofalmightygod.adapter.BookmarkRecycleAdapter.(BookmarkRecycleAdapter.java:37)
public BookmarkRecycleAdapter() {
this.bookmarks = Bookmark.find(Bookmark.class, "BOOK_ID = ?", bookId);
}
Thank you for you reply, stack trace is below.

The tables are generated correctly when you deploy for first time your app? How did you put the configuration in the AndroidManifest.xml? All of your domain classes provides an Empty constructor?

Thank you JonatanSalas, I think my problem is the same as issue #575.

OK! I will close the issue, if you still have this trouble please re open it!


Question about z-wave USB stick

Hi there, thanks for the great script that saved me a lot of time setting up home assistant. Today i received my z-wave (Aeontec gen 5) usb stick and its connected at /dev/ttyACM0. But now i need to pass the --device /dev/ttyACM0 argument to the docker container but i have no idea how to do that. Can you help me out?
Thank you
Gr Rik

That is done automatic

Also when the stick is plugged in after installation?

It was working indeed out of the box, my fault :S


Create Document: 07 Add System Management Service

This document should demonstrate:

How to add the System Management Service to the existing demo
How to confirm that it is working
Discover the results if System Management is used to stop, start, or restart any services under Open Horizon management

In creating an entry in the service definition file that includes the SMA, registering with OH throws an error.  The SMA requires a bind to docker.sock, but OH appears to not allow writeable or read-only access to the socket.  I've reached out to @dabooz for guidance on how to proceed.


1029: Fix typo in Docker deployment docs

Hi folks:
This should fix #1029. Let me know how I can improve this!

thanks 👍


Backend Setup

🍰 Pullrequest

Backend Setup.
This PR is the first step for a nodejs based backend server. This PR can be merged as soon as the communication between client and server is completely covered by the new backend. Furthermore authentication via JWT would be a good feature to start with
TechStack

[x] typescript
[x] express
[x] express-graphql
[-] typeorm
[x] eslint
[x] prettier
[x] jest
[x] publish workflow
[x] lint workflow
[x] test workflow
[x] docker

Issues

relates #13
relates #17
relates #477

Todo

[x] Implement jest
[ ] Make Issue for jest and test workflow
[x] Implement docker
[x] Implement docker-compose setup
[x] Implement workflows
[x] Implement typeorm connection
[ ] Make Issue for typeorm implementation
[ ] Implement proper graphql schema
[ ] Issue: API Versioning?
[x] Config (enable graphiql, debug, which api version is supported, port)
[x] Implement API connection to Login Server
[ ] Implement JWT based on old session
[ ] Remove Book examples

Try Login User, but I get this error:
Cannot find module 'class-validator'\nRequire stack:\n- F:\\Gradido\\gradido_apollo\\backend\\node_modules\\type-graphql\\dist\\resolvers\\validate-arg.js\n- F:\\Gradido\\gradido_apollo\\backend\\node_modules\\type-graphql\\dist\\resolvers\\helpers.js\n- F:\\Gradido\\gradido_apollo\\backend\\node_modules\\type-graphql\\dist\\resolvers\\create.js\n- F:\\Gradido\\gradido_apollo\\backend\\node_modules\\type-graphql\\dist\\schema\\schema-generator.js\n- F:\\Gradido\\gradido_apollo\\backend\\node_modules\\type-graphql\\dist\\utils\\buildSchema.js\n- F:\\Gradido\\gradido_apollo\\backend\\node_modules\\type-graphql\\dist\\utils\\index.js\n- F:\\Gradido\\gradido_apollo\\backend\\node_modules\\type-graphql\\dist\\index.js\n- F:\\Gradido\\gradido_apollo\\backend\\src\\index.ts

Hi @ulfgebhardt, @Mogge, @einhornimmond,
like we discussed in the last meetings I worked on this issue so we can tunnel the Login & Community Server calls over the Apollo Server.
What @ulfgebhardt and @Mogge prepared was clear so I could understand the base structure and continue to implement the different calls.
Now I am a bit concerned about the datas we want to send.
For the moment we use the two Methods apiPost and apiGet that were writen for the frontend, the methods do interpret the response of the servers.
The different szenaries for apiPost are:
Error Cases:

The state is different from 200 the call send an ErrorObject that is given to the response
The data.state is warning send an ErrorObject that is also returned in the JSON
Response: error is the catched JavaScript object

{
success: false,
result: error
}

Success:
Response: result is the axio result object
{
success: false,
result: result
}

So my question do we need the whole axio response object and the whole error object when we go out of the apiPost Method? Does the frontend need this objects to interpret things?
Or can I withdraw the two objects and use a simplified result like:
ErrorJSON
{
  success: false,
  result: error.message
}

SuccessJSON
{
  success: true,
  result: result.data
}

No the whole object is not needed - feel free to structure the graphql response as you wish. Limit the information you send out to what the client needs.
In this case I would always throw an error whenever success != true and let graphql handle the error. The thrown error can contain data which could be used by the client if needed.
This way we have an uniform result:

Success case - the structured data
Failure case - throw an error and use graphql error propagation

Sounds good?

OK sounds great. So something like the following code would do it:
@Query(() => LoginResponse)
async login(@Args() { email, password }: UnsecureLoginArgs): Promise<LoginResponse> {
    email = email.trim().toLowerCase()
    const result = await apiPost(CONFIG.LOGIN_API_URL + 'unsecureLogin', { email, password })

    if (!result.success) {
      throw new Error(result.data)
    }

    return new LoginResponse(result.data)
  }


Sun Grid Engine Support

📝 Description of the feature
As I read documentation "The supported scheduler environments are Univa Grid Engine (UGE), Load Sharing Facility (LSF), Portable Batch System (PBS), and Slurm.". Sun Grid Engine is not included if there is a way to use it please can you share. Or is it possible to add Sun Grid Engine?
💡 Steps for implementing the feature
No response
🔗 Useful links and references
No response

Both Univa and Sun Grid Engine are supported in Fluent in the same way (https://ansyshelp.ansys.com/account/secured?returnurl=/Views/Secured/corp/v241/en/flu_lm/flu_sge_parallel_sge.html). So, the UGE documentation in PyFluent is applicable for SGE also.


category create Error

Error on creating category
There is no info  about  creating a database  with specific collation
Preconditions
create database  with utf8mb4_general_ci collation

[points....]

@maddrid
Check the solution here
https://github.com/bagisto/bagisto/issues/2901

I am closing this as this issue came many times and resolution also provided.


add to uniform printer the hospital gown

Description
There should be a way to get more hospital gowns. They can only be obtained at MediDrobe and you cannot buy restock in cargo

A wardrobe restock box can restock any department-specific clothing vendor, including the MediDrobe.

yep


WIP: Implement timeout

Closes: https://github.com/metarhia/metawatch/issues/5

[x] tests and linter show no problems (npm t)
[x] tests are added/updated for bug fixes and new features
[x] code is properly formatted (npm run fmt)
[ ] description of changes is added in CHANGELOG.md

Landed in https://github.com/metarhia/metawatch/commit/fd626b3f12a85da013a8a1e0c58d43bc634c4da0, https://github.com/metarhia/metawatch/commit/94c1ede69ac7f22335210639512b43a10ab7511e, https://github.com/metarhia/metawatch/commit/53d8b66c3ebe7662b3645e287ab2176f83b0e2d7, https://github.com/metarhia/metawatch/commit/223e320b8656e7677342bb63493f6eb527189826, and https://github.com/metarhia/metawatch/commit/a4f857589fc285df701346cc3aae9245b1fa4d16


add package for daemonize tool

Please, set "platforms.all" :)
And squash commits?

Really? Upstream homepage doesn't say it runs on openbsd, and I'm fairly certain it won't work on cygwin.

(or netbsd)

My fault :)
So just squash commit.

Damnit hold on

(Are we really allowed to put lists-of-lists in meta.platforms?)

Ok, also changed source hash to base32.


Check if port is in use

I was working on server while in background another instance was started listening on same port (UDP 53). This caused script to finish without showing me any info. It took me a while to discover what was the problem. We should check if port is in user before calling $loop->run() and throw exception.

Diving into React's code, it looks like it should be throwing an error if a port is in use, but it doesn't seem to. But in anycase, think this is something to put on whoever is implementing the project.


docker run woodpeckerci/woodpecker-cli setup tries to run xdg-open to spawn a web browser

Component
cli
Describe the bug
The docker version of the CLI wants to spawn a web browser via a launcher of some kind, which will always fail in the context of docker, because the image cannot include a browser.
Steps to reproduce
$ docker run --rm -it woodpeckerci/woodpecker-cli setup --server-url https://some.host
1:01AM FTL error running cli error="exec: \"xdg-open\": executable file not found in $PATH"

Expected behavior
Rather than raising an error for the not-found case, this logic:
https://github.com/woodpecker-ci/woodpecker/blob/964860434c7548b73778476bf64125e6713d6b0a/cli/setup/token_fetcher.go#L115-L124
could print the URL to stdout so that the user can click on it, or copy and paste it to open it that way.
That would also help in the "unsupported platform" case.
System Info
$ docker run --rm woodpeckerci/woodpecker-cli --version
woodpecker-cli version 2.5.0

Additional context
No response
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Checked that the bug isn't fixed in the next version already [https://woodpecker-ci.org/faq#which-version-of-woodpecker-should-i-use]

If I bypass that by running setup with --token, the next thing I run into is:

https://github.com/woodpecker-ci/woodpecker/issues/3739

I realize that I probably also need to map a volume to capture whatever state setup is setting up, but it's a bit of a moot point at the moment! :-)

I had to go back to v2.3.0 to be able to use the CLI:
$ docker run --rm -it woodpeckerci/woodpecker-cli:v2.3.0-alpine --server $WOODPECKER_SERVER --token $WOODPECKER_TOKEN user ls

all subsequent versions are unusable due to external dependencies.


Updating Current View Coordinates

Description: Explore a method to update the current view coords when the viewport changes and affects the previous views.
Options:

Implement an Observer on the CSS variable values of the previous views to automatically update the current view.
Use an observer tied to getClientBoundingRect() of the previous views to recalculate the current view coordinates in real-time.
Split the zoomIn function into zoomIn and updateCoords to better manage updates.

Objective: Ensure the coordinates are dynamically updated based on changes in the viewport.

We made some improvements to allow coordinates recalculation when zumly canvas changes its size. Those changes will be merged when smae level navigation will be ready


Merge alpha into master

Description
Releases all alpha changes to master which mainly includes the PolyxTransaction entity, 6.x.x supported events and new migration handlers to support incremental migration support
Breaking Changes
NA
JIRA Link
NA
Checklist

[ ] Updated the Readme.md (if required) ?

@VictorVicente Can you help ignore the code duplication from the SonarQube?

lgtm. you should be able to merge in using push on the command line to override SonarCloud check


Odd error message when attempting to pass full path to --editor

I get the following (found while looking into #76), running from cmd.exe.
>renamer -p -e "%LOCALAPPDATA%\Programs\Microsoft VS Code\code.exe" *
Error: Failed to execute editor command: 'C:UsersOliverAppDataLocalProgramsMicrosoft VS Codecode.exe'

Caused by:
    program not found

The same can be got via pwsh.exe:
$ & renamer -p -e "$env:LOCALAPPDATA\Programs\Microsoft VS Code\code.exe" *
Error: Failed to execute editor command: 'C:UsersOliverAppDataLocalProgramsMicrosoft VS Codecode.exe'

Caused by:
    program not found

It appears that either the output of the path interpolates the backslashes somehow or the path gets interpolated by renamer (1.6.5) removing all backslashes.
@mtimkovich My current working theory is that shell_words::split() and/or shell_words::join() are coming back to haunt us in this case. Mind you, I introduced those 😑. It may be sufficient to make sure that all backslashes are doubled on Windows. I'll have a look.

@mtimkovich my hunch seems to have been right. It can be addressed using this:
diff --git a/src/main.rs b/src/main.rs
index 583ee64..615b6fc 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -253,7 +253,12 @@ fn open_editor(
         write!(tmpfile, "{}", input_files.join("\n"))?;
     }

-    let editor_parsed = shell_words::split(editor_string)
+    let editor_path = Path::new(editor_string);
+    let editor_path_exists = editor_path.exists();
+    let editor_quoted_path = shell_words::quote(&editor_string).to_string();
+    let editor_string = if editor_path_exists { editor_quoted_path } else { editor_string.to_string() };
+
+    let editor_parsed = shell_words::split(&editor_string)
         .expect("failed to parse command line flags in EDITOR command");
     tmpfile.seek(SeekFrom::Start(0))?;
     let child = Command::new(&editor_parsed[0])

What do you think? I'm sure you have a much deeper understanding of Rust than I have, so changes are improvements can be made.
This is still far from ideal, because it will also give the same weird output for the EDITOR command with the backslashes "eaten up". But using shell_word::quote() unconditionally doesn't seem like the right thing to do. Does it?
PS: tested with .\renamer -p -e "%LOCALAPPDATA%\Programs\Microsoft VS Code\code.exe" * (debug config).

Thinking about this some further, I'm more and more convinced that renamer should either do this only on Unix-like systems or, provided we don't want to exclude Cygwin, MSYS2 et. al., on Windows needs to check for the SHELL variable to detect if it is non-empty and contains some known value pointing to a Unix-y shell ...
Any input, @mtimkovich?

I'm doing some research into how other tools handle this issue and I'll report back

This person ran into the same problem as us. What do you think about using their package if the user is on Windows?
https://github.com/chipsenkbeil/winsplit-rs

I also found that crate, would that be a valid solution to you? Personally I am fairly conservative when it comes to adding new dependencies. That's why I ask.

Hi @assarbad! I have a draft PR I'd like you to have a look at: 5d52a59. I decided to use the winsplit library since the library itself is small and I didn't want to reinvent the wheel.
I also encountered a related issue with shell splitting that also affects Linux:
$ ./target/debug/renamer -e "/mnt/c/Program Files/Neovim/bin/nvim-qt.exe" *
["/mnt/c/Program", "Files/Neovim/bin/nvim-qt.exe"]
Error: Failed to execute editor command: '/mnt/c/Program Files/Neovim/bin/nvim-qt.exe'

Caused by:
    No such file or directory (os error 2)

I solved that by manually wrapping the editor string in quotes before parsing it.


Switch to internal value type with P3 support?

Thanks for the awesome library. Having a very pragmatic approach combined with a nice to use API: congrats!
Unfortunately there is one issue I experienced when working with "deeper" colors e.g.
oklch(76.28% 0.18 82.0399)
This is outside of the sRGB spectrum. The library seems fine for a starting point for doing calculations on sRGB values but I wish there would be an option to keep an extended color format internally and only fallback to sRGB if required e.g. by calling the methods on the from return object.
This is a nice color picker showing how it might be done better: https://oklch.com/#76.28,0.18,82.0399,100 - doing all processing on the extended color space and showing a fallback value where needed.

This is a great suggestion, and something I’ve been thinking about doing. This library was written before OKLAB/OKLCH were even added to the CSS Color Module 4 spec, and before browsers had experimental support for expanded color spaces (other than Safari’s P3 support). So sRGB was fine at the time, but it’s exciting to see the rapid change in web support for expanded color gamuts (which Evil Martians, creators of oklch.com, have had a huge part in advocating for).
This library keeps track of colors as sRGB currently, so swapping to another colorspace (e.g. XYZ) would require a rewrite. But it wouldn’t be a substantial change as it would basically just mean delaying calculations until sRGB is specifically requested. And because the web is rapidly moving past the sRGB gamut, this change makes sense long-term. I think I originally didn’t start that way because a) sRGB was sufficient a couple years ago, and b) I wanted to keep this footprint tiny / I didn’t want this to balloon into a “do-everything” color library of which there are several good ones already.

The question than is also what the best internal value type would be... e.g. what type supports all human-noticable accents to allow dealing with P3, Rec2020 and others in the future.

The question than is also what the best internal value type would be... e.g. what type supports all human-noticable accents to allow dealing with P3, Rec2020 and others in the future.

XYZ is the defacto “universal” colorspace because it was created to encompass all human-perceivable color, beyond what RGB can reproduce. So that’s probably the answer. Most color science libraries use XYZ space under-the-hood.
Again, maybe a mistake on my part not doing that in the beginning; it just seemed like overkill. But now, it feels like the right move.


Example with KV VM Extension in a full template

It would be great if you could document the caveats or tips on how this can be added to a full template in one shot, which would be common in actual usage.
For example, is this valid syntax to get a secret uri? Most examples on w/ search point to secretUriWithVersion, and it's also hard to find where these properties live.
"observedCertificates": [
"[reference(resourceId('Microsoft.KeyVault/vaults/secrets',variables('keyVaultName'),variables('SSLCertificateSecretName'))).secretUri]"
]

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

ID: a52b0724-6b3b-0daf-7749-d18104d6a40b
Version Independent ID: 56e5c7c4-f456-fc2e-99da-2771eb162947
Content: Azure Key Vault VM Extension for Windows
Content Source: articles/virtual-machines/extensions/key-vault-windows.md
Service: virtual-machines-windows
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@juliusl
Thanks for your feedback! We will investigate and update as appropriate.

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

We are currently cleaning up old issues and closing out items that are greater than 90 days old. If an issue is still present, please open a new feedback item on the document so we can prioritize correctly.
#please-close


[Actualites] Likes a post not works

Wrong design  see pics: send me a website reference how to design topic item.
And Recent post should place on left see police page
Repo steps:

Open the app on browser
Press actualise to display all topics
Press like on  first top:
Expected: Only the first  topic should increase the like --->  But all topics increase it ... LOL

This vfeature Not works and should be in Details file


Fix table schema location

The canonical place to find the schema in all cases is Table.schema, as it will inherit the schema name from its owning MetaData object if no explicit schema is set. See https://docs.sqlalchemy.org/en/13/core/metadata.html#sqlalchemy.schema.Table.params.schema for details.

Done


chore(*): Improve @clerk/backend DX [Part 3 - Internal subpath exports]

Description
Introduce @clerk/backend/internal and move some exports from top-level to the subpath. This change causes the removal of those exports from all the packages re-exporting from @clerk/backend (eg @clerk/nextjs).
Check changeset for examples and more information.
Checklist

[x] npm test runs as expected.
[x] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[ ] 🌟 New feature
[x] 🔨 Breaking change
[x] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[ ] @clerk/backend
[ ] @clerk/chrome-extension
[ ] @clerk/clerk-js
[ ] @clerk/clerk-expo
[ ] @clerk/fastify
[ ] gatsby-plugin-clerk
[ ] @clerk/localizations
[ ] @clerk/nextjs
[ ] @clerk/clerk-react
[ ] @clerk/remix
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/themes
[ ] @clerk/types
[ ] build/tooling/chore

@dimkl Shall we add the
/**
* @internal
*

jsdoc directive to these as well?

/**

@internal

i have added the @internal JSDoc


Kotools Assert 2.0

Description
Migrate Kotools Assert from 1.1 to 2.0.
Checklist

[x] Migrate.
[x] Test.
[x] Refactor.

Done.


How to solve PA2 level 6?

How do I implement the reduce method to simplify fractions? I am assuming a loop is required using the mul method in Num class but I'm not sure how to implement. Any kind soul please advise!

I think you can start by writing a helper function to find the GCD of 2 numbers, then another division function to reduce the fraction. Hope this helps!

I solved it by creating a mod() method for the Num class. Then solved for the gcd of the numerator and denominator using the following algorithm

I created a method to generate the greatest common denominator, as well as a division method for natural numbers. In my gcd method, I started the divisor with two and kept increasing the prime factorisation by one until the divisor exceeds a condition. (I used a while loop for this)

Hi @Jonacs13, you might want to create a .division() and .greatestCommonFactor() class, inside your Num
This will make it easier after you access the Num in  .reduce() found in Fraction.class to reduce to simplest fraction.

create a gcd method to find the gcd and div method in num to do any "divide" functions u may need

Thanks guys!

@gordengorden hi how do you create a mod() method without using the '%' operator as stated in the qn?

@gordengorden hi how do you create a mod() method without using the '%' operator as stated in the qn?

hi @giffordlee, what I did was to call sub() until the next sub() would result in a NaN output

hi @giffordlee, what I did was to call sub() until the next sub() would result in a NaN output

i see, thank you!

Hi @Jonacs13! I agree with @junbin-tan and @gordengorden on creating a divide() and highestCommonFactor() function. Personally, for me, I created an additional remainder() function to replace the % operator used in the highestCommonFactor() function mentioned by @gordengorden. Hope this helps!


Update TDM/Blitz scoreboard, Fixes #698

Change scores on the scoreboard.
Tested code

I'm pretty sure there are more adjustments to be made. If I'm not wrong, the heading for objectives used for maps that aren't Blitz is now the same as the Blitz entry.

Has what I've done need to be changed or just improved ?

maestro, if there is a max, to remove the "- - - - MAX - - - -" entry in the scoreboard, and set the score for each team as score/max blue team (for example 0/10 Blue Team)

Also, the team with the biggest score should go above

And, this should be in one commit.

@maestro34 Do you plan on adding the changes that @Pablete1234 mentioned?

@maestro34 This PR should be rebased

@iEli2tyree011 Sure, I will check that as quick as possible.

@thomblr this can be updated on #861 , you maybe should close this

Implemented https://github.com/twizmwazin/CardinalPGM/commit/ed7dff33ffd165c3414960b0add6673ce6dc641d

@iEli2tyree011 @twizmwazin @thomblr this can be closed


Deep ....rest operator

How about something like this?
array := [1, [2, 3], 4]
console.log [....array] # [1, 2, 3, 4]

unpack := (....args) => console.log unpack
unpack(1, [2, 3], 4) # [1, 2, 3, 4]

a := a: { b: 2 }
b := a: { c: 3 }
c := a: 1
console.log { ....a, ....b, ...c } # { a: 1, b: 2, c: 3 }

Perhaps by compiling to something using .flat()…
Also thinking maybe .... would flatten one level, while ..... could be fully recursive?

I could see ....array as shorthand for ...array.flat(). There are several examples of this in the Civet parser. For example:
  TopLevelSingleLineStatements:first NestedTopLevelStatements*:rest ->
    return [
      ...first.map(s => ["", ...s]),
      ...rest.flat(),
    ]

Is this clearer with ....rest instead? I'm not totally sure. It might be easily misread. ......rest (like two ...s) is maybe more intuitive to me, though we might be getting a little silly notation-wise. What do others think?
I don't think it makes sense to flatten other objects.

Four consecutive dots already has a meaning: https://civet.dev/playground?code=ZiAuLi4u


Split table search string by runs of white space

[ ] Changes have been thoroughly tested to not break existing functionality.
[ ] New functionality has been documented or existing documentation has been updated to reflect changes.
[ ] Visual changes are explained in the PR description using a screenshot/recording of before and after.

Discussion on this at https://github.com/filamentphp/filament/discussions/8325 in particular https://github.com/filamentphp/filament/discussions/8325#discussioncomment-6957655

Thanks. If I don't get the changes sorted tonight, I'll push them tomorrow (Monday).

My original thinking was, the query string is what it is - spaces, characters, null (at the start), mixed letter case. That doesn't get changed in the process, since any party of that could be important to someone's search rules.
The two functions that do SQL searching then needs a list of words. The extractTableSearchWords() function would return an array of words from the raw search text. It would handle all the parsing - trimming, white space, etc. The methods using extractTableSearchWords() would not need to care what that raw string looks like, whether it has white space around it or not. That's why I had the trim() in there.
The search string does not need to be trimmed before checking if it is filled() because filled() will treat a string of zero or more whitespace characters as unfilled.

Ok, for now I've removed the tests about untrimmed whitespace, since that will never be passed to extractWords() in the first place 👍
Thanks

This word splitting is really nice for small applications and tables. But it does come with a downside from an indexing perspective.
Is there room for having both, defaulting to the nice word splitting, but having the option to disable it easily on the Page level?
I put a bit more info in this discussion item: https://github.com/filamentphp/filament/discussions/13466


Update devtest-lab-guidance-prescriptive-adoption.md

Added a section for Intended audience with a link to case studies for DTL.

@tanmayeekamath : Thanks for your contribution! The author, @spelluru, has been notified to review your proposed change.

#sign-off

@ktoliver : Thanks for your contribution! The author, @spelluru, has been notified to review your proposed change.

@ktoliver : Thanks for your contribution! The author, @spelluru, has been notified to review your proposed change.


CSS property not exported from react-print

I wanted to add a google font to my template . Hence I tried to follow the docs :

But on running , I get the error "'CSS' is not exported from '@onedoc/react-print' (imported as 'CSS')."

Another way as mentioned in the docs: using the import url in a css file and then using it is not working either.
It would be great if this issue could be fixed.

Hey @career-tokens, thanks for reporting this!
This may be because your client SDK is not up to date, as the export has only been recently added (see https://github.com/OnedocLabs/react-print-pdf/blob/main/src/index.ts#L22)
You should be able to get rid of that error using npm i -s @onedoc/react-print@latest
Let me know if this works!

Okay sure I will try that and let you know.
On Fri, 16 Feb 2024, 08:57 Titouan Launay, @.***> wrote:

Hey @career-tokens https://github.com/career-tokens, thanks for
reporting this!
This may be because your client SDK is not up to date, as the export has
only been recently added (see
https://github.com/OnedocLabs/react-print-pdf/blob/main/src/index.ts#L22)
You should be able to get rid of that error using npm i -s
@@.***
Let me know if this works!
—
Reply to this email directly, view it on GitHub
https://github.com/OnedocLabs/react-print-pdf/issues/1#issuecomment-1947700457,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BAD5CLTU627OE75VULS6MOTYT3GZ5AVCNFSM6AAAAABDIX52C6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBXG4YDANBVG4
.
You are receiving this because you were mentioned.Message ID:
@.***>


Strange artifacts/rendering bug when using Spine in 3.51

Version

Phaser Version: ^3.51.0
Operating system: Windows 10, PRO
Browser: All Browsers

Description
Note: This also happens in 3.50 (not sure about older versions).
Using the SpinePlugin located on GitHub (here) when loading a spine character there are several artifacts and missing pieces of the texture that do not occur in other Spine renderers. Per the documentation I have made sure that the version of Spine we've exported the character from matches the version of the Spine Runtimes used, in this case, 3.8.95.
Example Test Code
Demonstration link: https://christiantucker.github.io/phaser3-spine-bug/
Minimum Viable Reproduction: https://github.com/ChristianTucker/phaser3-spine-bug
Additional Information
I will provide two examples, the first of which is a rendering from Phaser and the second of which is a rendering using PixiJS (both using the same spine assets) and pixi-spine.
Phaser

PixiJS

This is a pre-multiplied alpha issue. You need to set the textures preMultipliedAlpha flag in the load call to match how it's exported from Spine.

This is a pre-multiplied alpha issue. You need to set the textures preMultipliedAlpha flag in the load call to match how it's exported from Spine.

Hey @photonstorm thanks for reaching out, you were right this absolutely was the issue. I must have missed it in the documentation, however, perhaps I'm blind but I can't seem to find the documentation about that option when calling .load in found here.
However, I can find the documentation in the code stating that you can use a configuration object, which isn't mentioned anywhere in the SpinePlugin documentation available on the website. The only place I can find a config object mentioned is in the documentation for make(config, [, addToScene]) but this doesn't show initialization of preMultiplyAlpha
I may just be totally missing it, but where is the documentation located that goes over the initialization parameters? I see things like this in the documentation
this.load.spine('stretchyman', 'stretchyman-pro.json', [ 'stretchyman-pma.atlas' ], true);

and judging on the config-object I found in the code earlier I can only assume that true here is for preMultipliedAlpha but  as shown in the example that doesn't work while the following config does.
this.load.spine({
    key: 'test',
    jsonURL: 'character.json',
    atlasURL: 'character.atlas',
    premultipliedAlpha: true
});

Hey @photonstorm thanks for reaching out, you were right this absolutely was the issue. I must have missed it in the documentation, however, perhaps I'm blind but I can't seem to find the documentation about that option when calling .load in found here.
However, I can find the documentation in the code stating that you can use a configuration object, which isn't mentioned anywhere in the SpinePlugin documentation available on the website. The only place I can find a config object mentioned is in the documentation for make(config, [, addToScene]) but this doesn't show initialization of preMultiplyAlpha
I may just be totally missing it, but where is the documentation located that goes over the initialization parameters? I see things like this in the documentation
this.load.spine('stretchyman', 'stretchyman-pro.json', [ 'stretchyman-pma.atlas' ], true);

and judging on the config-object I found in the code earlier I can only assume that true here is for preMultipliedAlpha but  as shown in the example that doesn't work while the following config does.
this.load.spine({
    key: 'test',
    jsonURL: 'character.json',
    atlasURL: 'character.atlas',
    premultipliedAlpha: true
});

It's in the docs here: https://photonstorm.github.io/phaser3-docs/Phaser.Loader.LoaderPlugin.html#spine__anchor (which uses the SpineFile object here https://photonstorm.github.io/phaser3-docs/Phaser.Loader.FileTypes.SpineFile.html)

It's in the docs here: https://photonstorm.github.io/phaser3-docs/Phaser.Loader.LoaderPlugin.html#spine__anchor (which uses the SpineFile object here https://photonstorm.github.io/phaser3-docs/Phaser.Loader.FileTypes.SpineFile.html)

@photonstorm Interesting, I wasn't expecting information like that to be scattered throughout other places, it feels a little strange to me that documentation relating to loading a spine object as defined in the SpinePlugin is documented in the LoaderPlugin. I would have never thought to look there.
Thanks for all the help! Checking LoaderPlugin and FileTypes documentation will sure save me some headache in the future.

@photonstorm Interesting, I wasn't expecting information like that to be scattered throughout other places, it feels a little strange to me that documentation relating to loading a spine object as defined in the SpinePlugin is documented in the LoaderPlugin. I would have never thought to look there.
Thanks for all the help! Checking LoaderPlugin and FileTypes documentation will sure save me some headache in the future.


It's not possible to overwrite the Label text with an empty string

Hey there! Long time WTForms user, first time ticketer.
I want to generate an empty label for a checkbox (so that I can create a custom styling), but I noticed that I can not overwrite the label text using
field.label("")

because of the text = escape(text or self.text) here: https://github.com/wtforms/wtforms/blob/master/src/wtforms/fields/core.py#L461
Of course, I can work around it by doing
field.label(" ")

which doesn't really make a difference to the generated HTML, so I'd be okay just doing that.
Is this on purpose, or would you accept a PR that checks if it's a string first?

Can you explain what you mean by this? If you're trying to render your own custom html label why do you want to use the form's?

I thought that way I wouldn't have to reimplement what was essentially the exact same label that wtforms already provides, just with empty text. But I guess it doesn't really matter.

I see what you're trying to do, but I'd advise still rendering the label then hiding it with css, as that is much nicer for assistive technology like screen readers.

@davidism Yes, sure! I was indeed rendering two labels: One as the actual label, and one as a custom checkbox, so I think it all worked out in the end.


Blender: fix import workfile all families

Brief description
Having this feature related to workfile available for any family is absurd.

EDITED: maybe the "workfile" also needs somewhere to load it right.

EDIT: Maybe you need to register in somewhere to make the import blend and append blend loader to show up, or use the original code.

I'm sorry, I don't get your review. Could you please explain a bit more?

EDIT: Maybe you need to register in somewhere to make the import blend and append blend loader to show up, or use the original code.

I'm sorry, I don't get your review. Could you please explain a bit more?
when I was using your code with workfile added into families, the loader doesn't show the import option
But if I run the original code, it will be there.

EDIT: Maybe you need to register in somewhere to make the import blend and append blend loader to show up, or use the original code.

I'm sorry, I don't get your review. Could you please explain a bit more?

when I was using your code with workfile added into families, the loader doesn't show the import option But if I run the original code, it will be there.

Tell me if I'm wrong, but it is actually the point. You mustn't see the loader for families other than workfile. And in your example you seem to test exactly the case where it must be disabled, confirming the change works.

EDIT: Maybe you need to register in somewhere to make the import blend and append blend loader to show up, or use the original code.

I'm sorry, I don't get your review. Could you please explain a bit more?

when I was using your code with workfile added into families, the loader doesn't show the import option But if I run the original code, it will be there.

Tell me if I'm wrong, but it is actually the point. You mustn't see the loader for families other than workfile. And in your example you seem to test exactly the case where it must be disabled, confirming the change works.

ah i see. I misunderstand the intention of this PR and I resume my previous comment. If it is the case, i think you are right. It shouldn't load the workfile, instead there should be a family for raw blend scene included into the loader and the extractor for exporting the blend scene.


Modernize test coreRfeature

This modernizes test_coreRfeature as follows:

Everything is wrapped in test_that
All try(test_that(…)) idioms removed.
Remaining try for compileNimble uses silent=TRUE and does lead to a managed use of expect_equal to check if compiler failure or success was as expected.
Uses in test-coreRfeature, test-allocation and test-numericTypes` work on my machine.

LGTM
BTW Have you checked with both test_package/test_file and Rscript?

That expect_error example would be cleaner but equivalent in behavior.  I'll try it.  Thanks for the reminder to try Rscript mode as well.   I ran via test_that locally and I guess they are running via Rscript on travis.

@perrydv It looks like this merge conflicted with my travis-osx merge, possibly due to different behavior of expect_identical on linux and OS X. The current devel is now failing OS X but passing Linux. As you suggested, I think this should be an easy fix.


New lines not adding to any messages!

i tried to message contain new lines. but it doesn't work properley.

Hi @horapusa-lk
Can you describe what command you tried?
This works for me:
npx mudslide send me 'hello\nworld'

thanks it works. sorry for the disturb sir!.

No problem 🙂 I've added your question to the Readme 👍


Add CORS support to blackd

See #622. This is my first time contributing to a project that uses pipenv, so please let me know if there's something I did wrong.
Additionally, I'm not sure how to add a test for this. I'd expect it to test that a request sent from a client on a different host works correctly, but I read through the docs for the aiohttp TestClient and couldn't come up with anything. Setting the host param in the constructor for TestClient async with TestClient(TestServer(app), host=some_string) gives a TypeError about the host parameter being invalid.

Pull Request Test Coverage Report for Build 819

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-4.4%) to 91.702%

Totals

Change from base Build 814:
-4.4%

Covered Lines:
2774

Relevant Lines:
3025

💛  - Coveralls

@zsol, when you get a chance could you give this a look?

Thanks!


build: fix typo in build system

This fixes a typo in the build system which is important because it
fixes the dependency tracking and the output location for the swift doc
that is generated for the swift SDK overlay.

CC: @das @MadCoder @gottesmm @llvm-beanz

moodule_directory => module_directory. I think this is fine to merge.

thanks!


NAIP mosaic dataset references invalid data paths

Data\FoundationOverlays\OrthophotoBase\NAIP\NAIPClip.gdb\NAIP mosaic dataset references the following paths which do not exist (in addition paths should reference UNC path):
E:\Data\Esri\NewGeography\NAIPClip.gdb
E:\Data\Esri\NewGeography\NAIPClip.Overviews\MosaicDataset.Overviews
I'm correcting the path on the UC machines.

@NatalieCampos can you take a look at this issue and fix if still required. .

@Ben @lorraine
I am in the portal training today but I can look at it at the end of the day.  But I think that might be past the cutoff.
Sent from my iPhone
On Sep 30, 2015, at 8:43 AM, Ben Conklin <notifications<EMAIL_ADDRESS>wrote:
@NatalieCamposhttps://github.com/NatalieCampos can you take a look at this issue and fix if still required. .
—
Reply to this email directly or view it on GitHubhttps://github.com/Esri/ops-server-config/issues/896#issuecomment-144451794.

I have repaired the item paths from E:\Data\Esri\NewGeography\ to Data\FoundationOverlays\OrthophotoBase\NAIP.
@nfeuerstein  @ScottCecilio
I am confused about which NAIP imagery is being released the clipped in the Mosaic dataset or the orignial Mr.Sid files in the NAIP folder.  Should the original Mr.SID images be removed in favor of the clipped  NAIP images?  Also I noticed that the clipped images are stored in the NAIP.gdb.  It is not best practice to store raster imagery inside the gdb as it results in slower read times.

@NatalieCampos can you look and see which images are used in released image services and remove  any that aren't used (you can tell if it is released if the owner is not an individual person).

I think I might be copied on this exchange by mistake.
Lorraine Reinsch
Please excuse my phone's auto-correct.

On Oct 1, 2015, at 9:09 AM, Ben Conklin<EMAIL_ADDRESS>wrote:
@NatalieCampos can you look and see which images are used in released image services and remove any that aren't used (you can tell if it is released if the owner is not an individual person).
—
Reply to this email directly or view it on GitHub.

@NatalieCampos I removed your incorrect mentions from the comment you made. Be more careful with your mentions and make sure you reference the correct person.

@elinz The original issue of incorrect data paths has been fixed.  Please close this issue after the data is reviewed.
@conklinbd None of the NAIP is currently being released. Currently, @nfeuerstein owns NAIP image service and several web maps associated with it.  No data will be removed at this time.

@NatalieCampos Looks like something is wrong with this mosaic dataset. If you navigate to the NAIPClip.gdb in ArcMap Catalog tree you will see AMD_MosaicDataset_ANA and AMD_MosaicDataset_STS. Datasets that start with "AMD" are supposed to be hidden tables that uses can't interact with.

@nfeuerstein Can you address the AMD_MosaicDataset_ANA and AMD_MosaicDataset_STS issues?

Yeah, I still shouldn't be on this chain...
Lorraine Reinsch
Please excuse my phone's auto-correct.

On Oct 2, 2015, at 12:22 PM, Natalie Campos<EMAIL_ADDRESS>wrote:
@nfeuerstein Can you address the AMD_MosaicDataset_ANA and AMD_MosaicDataset_STS issues?
—
Reply to this email directly or view it on GitHub.

@Lorraine, my apologies, we're not able to remove you from this issue.  if you click on "Unsubscribe" on the right hand panel you will stop getting notifications.  Sorry!

@NatalieCampos can you please look in to this issue.

@NatalieCampos Please tell me what you have done to fix this issue as the mosaic dataset in question Data\FoundationOverlays\OrthophotoBase\NAIP\NAIPClip.gdb no longer exists. Did you delete and create a new mosaic dataset \[FILESERVER]\data\FoundationOverlays\OrthophotoBase\NAIP\NAIP.gdb\NAIP ?

@elinz that is what was done.

Well the NaipClip.gdb has been renamed to NAIP.gdb.  the NAIPClip Mosaic dataset was deleted and a new NAIP mosaic dataset was created.  The reason the new NAIP mosaic dataset was created was because I was unable to reset the path for the overviews.  The overviews were stuck being generated in some other location and I had no ability to correct the path.

Verified mosaic dataset are referencing UNC paths and AMD_MosaicDataset_ANA and AMD_MosaicDataset_STS "hidden" tables are no longer being listed in Catalog.


Rancher Secrets custom path

I have image that is taking config file from exact path that can't be changed.
How to specify custom destination path of secrets, so my config gets mounted to that path?

you might be able to sym link it after the container starts to the desired path... as secrets are mounted in /run/secrets.

Is there any method to avoid modifying entrypoint/cmd?

No, it's a memory-backed volume mounted onto the container.  You can't put individual secrets in arbitrary places.

So "new feature request"?

it's a memory-backed volume mounted onto the container
There must be optional "mount path" when you put secret in container.

On RedHat systems (Atomic and RHEL) /run/secrets/etc-pki-entitlement, /run/secrets/rhel7.repo, and /run/secrets/rhsm are mounted to propagate RedHat subscription info to all constituent containers on the system, regardless of base OS in the container. It would be incredibly useful if we could specify an alternate mount path for the secrets service container, like /run/rancher/secrets/. Or simply mount just the specific secret to /run/secrets/ instead of the whole directory. As-is this feature is completely unusable in a RHEL environment.

Making this configurable might be an option. I'll change the ticket to feature request.

If you have to use RHEL's Docker but don't need to mount for subscription info you can configure --enable-secrets=false inside the OPTIONS parameter in /etc/sysconfig/docker to disable the mount on /run/secrets. Don't forget to restart the docker service to enable the change.
If you have to use the subscription info, this issue remains valid.

Thanks for your report!
With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.
If you think we should keep this issue open, please let me know.


How the selection box sets the properties of the display

Demo URL：https://anu-vue.netlify.app/guide/components/select.html#states
Through the case on the website, I want to achieve a custom display effect.
However, I found that I could not set the displayed elements manually
For example, in the above case, when I selected VueJS, I wanted to display VueJS instead of vue in the selection box