docker-arm drone pipeline is failing

https://drone.owncloud.com/owncloud/ocis/2694/75/4
+ /usr/local/bin/docker build --rm=true -f ocis/docker/Dockerfile.linux.arm -t 8343ce2caad4f93bd727ec1014d4439fe6393b2d ocis --pull=true --label org.label-schema.schema-version=1.0 --label org.label-schema.build-date=2021-01-16T09:28:46Z --label org.label-schema.vcs-ref=8343ce2caad4f93bd727ec1014d4439fe6393b2d --label org.label-schema.vcs-url=https://github.com/owncloud/ocis.git
Sending build context to Docker daemon  190.4MB

Step 1/10 : FROM arm32v6/alpine:latest
latest: Pulling from arm32v6/alpine
b550170d5d44: Pulling fs layer
b550170d5d44: Verifying Checksum
b550170d5d44: Download complete
b550170d5d44: Pull complete
Digest: sha256:a220fbceafab4a9af7dd7b574eea7ce646dc5ab57d0b7fcd1f349dcca3d66929
Status: Downloaded newer image for arm32v6/alpine:latest
 ---> 4a36e4045daf
Step 2/10 : RUN apk update && 	apk upgrade && 	apk add ca-certificates mailcap && 	rm -rf /var/cache/apk/* && 	echo 'hosts: files dns' >| /etc/nsswitch.conf
 ---> Running in 52350d4c37d1
/bin/sh: apk: Operation not permitted
The command '/bin/sh -c apk update && 	apk upgrade && 	apk add ca-certificates mailcap && 	rm -rf /var/cache/apk/* && 	echo 'hosts: files dns' >| /etc/nsswitch.conf' returned a non-zero code: 127
time="2021-01-16T09:28:52Z" level=fatal msg="exit status 127"

It started failing some time on Friday 2021-01-15.
The failing step uses plugins/docker:18.09 tag linux-arm. Nothing has changed for that docker image https://hub.docker.com/r/plugins/docker/tags?page=1&ordering=last_updated
That pulls arm32v6/alpine:latest and that has changed. https://hub.docker.com/r/arm32v6/alpine/tags?page=1&ordering=last_updated - latest was updated 1 day ago. That looks suspicious.

@phil-davis Fixed by #1219 by pinning Alpine to 2.12. 2.13 is the version causing the current trouble

@phil-davis Fixed by #1219 by pinning Alpine to 2.12. 2.13 is the version causing the current trouble


Update object reconciler to overwrite conflicting source

For source objects with a conflicting source in the ancestors, treat
them as propagated objects and overwrite them with the source in the
ancestor. See http://bit.ly/hnc-propagation-conflict
Tested by make test with a new integration test case.
Part of #1076

/hold
For the upcoming webhook change.

/lgtm
/approve
Why does this need to wait for the webhook?

Why does this need to wait for the webhook?

Ah right, the change in the reconciler can be merged without webhooks are in place on master.
/hold cancel


Dead link in README.md

Under Setting a default image the link for "Front Matter Defaults" (https://www.bridgetownrb.com/docs/configuration/front-matter-defaults/) is a 404 on the main Bridgetown Site.
Sorry this is not a pull request with a fix, but it's not apparent to me what the correct item in the bridgetown documentation should be.

Ah, yeah it should actually be: https://www.bridgetownrb.com/docs/content/front-matter-defaults


Move to own note

The side bar is not the ideal place for the gantt chart. Could you add the ability to assign it to its own page in the vault. I can move it but it returns to the side bar if there are any changes. The alternative is to add syntax to the code block so that we can restrict the folders it searches in.

thank you. I never think Mermaid is the good ideas, it cannot handle long data (above hundred of tasks). I using it from the beginning just because of simple . I don't planning much in that moments. I am migrating to more mature React library, which can handle thousand of task at once. And will try to find the way to render such big chart in the screen

However if you did make it movable to its own page it would still be very useful for me, if it is not that difficult?


What should I do when the Failed to load RTCore driver problem occurs?

Run with --debug and post the full output here please

[DEBUG] Program started
[DEBUG] Running with administrator privileges
[DEBUG] COE file: {}C:\Users\Administrator\Desktop\Pcie\save\I225V2\pcileech_cfgspace.coe
[DEBUG] Raw PCI Address input: {}7:00.0
Parsing COE file: C:\Users\Administrator\Desktop\Pcie\save\I225V2\pcileech_cfgspace.coe
COE file parsing complete. Lines read: 275
COE data size: 4096 bytes
Extracted Vendor ID: 0x8086, Device ID: 0x15f3
[DEBUG] COE data size: {} bytes4096
[DEBUG] Parsed PCI Address: {:X}:{:X}:{:X}700
[DEBUG] Attempting to extract driver to: {}C:\Users\Administrator\AppData\Local\Temp\RTCore64.sys
[DEBUG] Driver file extracted successfully to: {}C:\Users\Administrator\AppData\Local\Temp\RTCore64.sys
[DEBUG] Driver extracted successfully. Attempting to load...
[DEBUG] StartService failed. Error: {}183
[DEBUG] [ERROR] Failed to load RTCore driver.
C:\Users\Administrator\Desktop\cfgcomp>

This is happening because the same driver is loaded elsewhere, likely either MSI Afterburner or drvscan

Thanks, I'll try that, I think I have drvscan enabled.


Is it valid to make a partial capture?

Hi,
Is it legal to use in-application API to capture a small part of a frame? For example if I want to debug a specific draw call that is not issued every frame. In this case I can use API to surround this particular draw call in application code and make sure that when it is issued the capture is saved. Or maybe there are some caveats and it is better to capture the whole frame?

Hi there, your issue has been closed due to not correctly following the issue template. Please read the documentation about filing issues and open a new issue which does follow the issue template. Do not reply to this issue with more information or with a correctly formatted issue, as that will only lead to confusion.
I try to address all github issues and fix bugs quickly, but badly formatted issues or poorly described bug reports especially are a significant drain on project resources. Taking a few minutes to file a good issue makes a huge difference and will get it addressed faster - everybody wins!
In github when you first create a new issue the form is pre-populated with a template which you should read over and follow. As linked above there is documentation going into more detail about what makes a good issue.


SSL Problem with ez-letsencrypt

There is a problem with ez-letsencrypt.
When I tried to run ez-letsencrypt,
it runs like this with ssl files
ssl_certificate           /etc/letsencrypt/live/$le_hostname/fullchain.pem;
ssl_certificate_key       /etc/letsencrypt/live/$le_hostname/privkey.pem;
ssl_trusted_certificate   /etc/letsencrypt/live/$le_hostname/chain.pem;
However, this project has ssl files on ssl folder. That doesn't make work well
Please help me!

@tjinewpro - I can see how this might be confusing and I'll add some documentation to assist.
The local ./ssl directory contains self-signed certs for convenience. This directory is volume mounted from the host to the Nginx container in the provided example, but can be a mount to any directory that you genuine certificate reside in.
From the ez-letsencrypt example

The host SSL certs are stored at /root/certs so --certsdir /root/certs is the flag used when invoking that script

Internally this is resolved to a volume mount as --volume $le_certsdir:/etc/letsencrypt so the Nginx container will look to its local /etc/letsencrypt volume for the certs

This results in the default.conf stanza you've mentioned in your question
server {
    listen      443 ssl;
    listen [::]:443 ssl;
    server_name $le_hostname;
    ssl_certificate           /etc/letsencrypt/live/$le_hostname/fullchain.pem;
    ssl_certificate_key       /etc/letsencrypt/live/$le_hostname/privkey.pem;
    ssl_trusted_certificate   /etc/letsencrypt/live/$le_hostname/chain.pem;
    location / {
        root   /usr/share/nginx/html;
        index  index.html index.htm;
    }
}

Applying to WordPress
Lets say you have the following setup

Hostname my-wordpress-site.com
You want to store your certs at /root/mycerts, you'd run the ez-letsencrypt script with --certsdir /root/mycerts

You would make the following adjustments

Update the .env file value for NGINX_SSL_CERTS

# Nginx Settings
export NGINX_CONF=./nginx/default.conf
export NGINX_SSL_CERTS=/root/mycerts         # <-- set host directory to /root/mycerts
export NGINX_LOGS=./logs/nginx

Update the volume mount in docker-compose.yml

nginx:
    # default ports 80, 443 - expose mapping as needed to host
    image: nginx:1
    container_name: wp-nginx
    env_file:
      - .env
    restart: unless-stopped
    networks:
      - wordpress
    depends_on:
      - wordpress
    ports:
      - "8080:80"    # http
      - "8443:443"   # https
    volumes:
      - ${WORDPRESS_LOCAL_HOME}:/var/www/html
      - ${NGINX_CONF}:/etc/nginx/conf.d/default.conf
      - ${NGINX_SSL_CERTS}:/etc/letsencrypt:ro              # <-- set internal directory to /etc/letsencrypt
      - ${NGINX_LOGS}:/var/log/nginx

Update the nginx/default.conf file

    # update ssl files as required by your deployment
    # ssl_certificate     /etc/ssl/fullchain.pem;
    # ssl_certificate_key /etc/ssl/privkey.pem;
    # NEW Let's Encrypt Certificate
    ssl_certificate           /etc/letsencrypt/live/my-wordpress-site.com/fullchain.pem;
    ssl_certificate_key       /etc/letsencrypt/live/my-wordpress-site.com/privkey.pem;
    ssl_trusted_certificate   /etc/letsencrypt/live/my-wordpress-site.com/chain.pem;

At this point you should be able to run WordPress using your new certificates

Even though I changed all of files follow your direction, there was still an error.
`
Saving debug log to /var/log/letsencrypt/letsencrypt.log
Requesting a certificate for my-web.com
Certbot failed to authenticate some domains (authenticator: webroot). The Certificate Authority reported these problems:
Domain: my-web.com
Type:   connection
Detail: Fetching http://my-web.com/.well-known/acme-challenge/OdCWUbQGgEUQZGS2wA1ZFxN0HG8BcpaEqujWa9KFIBU: Timeout during connect (likely firewall problem)
Hint: The Certificate Authority failed to download the temporary challenge files created by Certbot. Ensure that the listed domains serve their content from the provided --webroot-path/-w and that files created there can be downloaded from the internet.
Some challenges have failed.
Ask for help or search for solutions at https://community.letsencrypt.org. See the logfile /var/log/letsencrypt/letsencrypt.log or re-run Certbot with -v for more details.
[INFO] Nginx ssl certificate configuration values (relative to nginx container: wp-nginx)

ssl_certificate           /etc/letsencrypt/live/my-web.com/fullchain.pem;
ssl_certificate_key       /etc/letsencrypt/live/my-web.com/privkey.pem;
ssl_trusted_certificate   /etc/letsencrypt/live/my-web.com/chain.pem;
`


Implement payment confirmation screens

This PR picks off from the https://github.com/GaloyMoney/galoy-pay/pull/210

adds payment confirmation screens (error and success)
adds new hook to get payment status
update to yarn lock file with peer dependencies
fix router/href implementations based on NextJs new release (12.2.3)

When this gets merged, the other PR https://github.com/GaloyMoney/galoy-pay/pull/210 will be outdated due to behind commits

I left one comment and also wanted to list some other things which I noticed which don't necessarily need fixed in this PR but will need to be fixed before rolling out:

There is no validation on the amount being entered for invoices.  I was able to enter an amount of 128.000000 for example.
The amount is included in the url of the page which displays the invoice QR code - but changing this amount has no effect after the page has been loaded.  I would expect to be able to pass an amount in a url and for it to be reflected in the invoice.
When I tap the "create invoice" button it makes 2 calls to the server lnInvoiceCreateOnBehalfOfRecipient mutation - one for 0.00 and one for the correct sat amount which has been entered.
When I tap "create new invoice" on the screen with the QR code it reloads the assets on the page (redownloads images etc).
Once an invoice has been generated and you are on the screen which shows QR code, if you go back to the digits entry screen and don't change the amount but hit "create invoice" it doesn't do anything (the call to the server is made with 0.00 as the amount as opposed to whichever amount is populated in the amount field.
There are not translations for text on display.

validation for amount has been fixed.
amount is now included in URL and triggers creating an invoice
Double calls to server has been fixed using the @galoyclient deps
re-download of assets does not occur on creating an invoice.
repeated calls have been fixed as stated above.
translations for text will be handled in the next PR.


fix: component names

Implemented the component names injection in the components plot. Add an option to choose custom component names. If not specified, Variable-# is written as a tittle of each component. Example of usage:
sm = SOMFactory().build(array, mapsize = [msz, msz], normalization = 'var', initialization='pca', component_names = vector_of_names)
sm.train(n_job=1, verbose="info")

I am going to wait until you merge the other pull request in order to adapt this branch for automatically merging it, cause they have changes that overlap. As soon as you merge the other one, I will adapt this one :D. Thanks

Conflicts solved. Now they can be merged into the master. I do not forget the doc... I am going to prepare a ipython notebook with a reproducible example covering all the functionality added
Thanks!


Fix main panel focus bug

Problem
After second (mouse) click on main panel the [, ] and esc stop working.
This fixes that.

good pickup!


Decimal -> binary

Linked with #3. We must choose number system. For now lets decide between binary in msb or lsb. Others may be added in the future.

Current decision is to use lsb system


Get output from integrated terminal "sendText"

Given this code:
const terminal = window.createTerminal(`Ext Terminal #${1}`);
terminal.sendText(command);

It will be nice to get the output of this command, something like:
const terminal = window.createTerminal(`Ext Terminal #${1}`);
terminal.sendText(command).
  then(stdout => console.log(stdout)).
  reject(e => console.log(e));

An example workaround for now is:
import * as cp from 'child_process';

cp.exec(command, {cwd: vscode.workspace.rootPath, env: process.env}, (e, stdout) => {
  if (e) {
     window.showErrorMessage(e.message);
  } else {
    window.showInformationMessage(stdout);
  }
});

Thanks!

Unfortunately this is difficult/impossible to do by design, we don't even know if sendText will run a command as all we're doing is sending some text to the shell (which would be running vim for example).

Ok. Thanks!


Can Unity run on a window computer while ROS run on the other ubuntun  without VM?

I have a feature request!
I want to do this:

Can Unity run on a window computer while ROS run on the other ubuntun  without VM?
For being able to do it, I wish ROS# had the following feature:

Yes.


Feature/tech 610 implement argocd

First draft of the logic based on the remote artifact caching pr since there is a lot of shared logic.

Closing in favor of a clean pr without all the rebasing issues fromt the caching pr.


IncludeOptional for conf.d ... is there a reason why this is not there?

hi,
we use the base httpd image sometimes to have a quick webserver. one thing that would really, really help (and make the base image much more flexible) would be something like
IncludeOptional /usr/local/apache2/conf/conf.d/*.conf
which, by default, would be empty of course. however, it would provide and easy hook for adding some custom configuration options. this would be much appreciated.
seeing that it is not there, i was wondering if this is by design? or just a plain oversight?
thanks
.rm

It's by design that we don't include this, but I can't speak for upstream:
https://github.com/docker-library/httpd/blob/385f0d1d8e09fd44c9b083fa390a8b1f09e1a33a/2.4/Dockerfile#L126-L132
This is the only modification we make to the default configuration supplied by httpd upstream -- namely, switching the default logging to use stdout/stderr (as is normal for container images) instead of files.

hi,
thank you for your answer. doing this include would be a nice addition though, which makes much more sense in a docker image than in a standalone binary - somewhat comparable to the change in logging. i just created a PR.
https://github.com/docker-library/httpd/pull/144
maybe this does make sense? if not, feel free to ignore :)
thanks
.rm

I would love to see this included by default.  With kubernetes I use configmaps to store my specific configurations and use volumes to mount them at run time. I do not always need to build an image however you are forcing me to aways extend your image just to add a few config files.

@tianon any opinion on this?

hi tianon - well ... upstream (i.e. vanilla install from tgz) it is super easy to add it manually. with a docker image, it's nice to have an optional hook (as greg said, and i would appreciate it for the same reason).

Closing as a duplicate of #105; see especially https://github.com/docker-library/httpd/pull/105#issuecomment-408918821 and https://github.com/docker-library/httpd/pull/105#issuecomment-434514664 for trivial ways to accomplish this both with a simple Dockerfile and without a Dockerfile (using the vanilla published image as-is).


Navigate from tab navigation to stack navigation

Hi, when I use a stack navigator inside a tab navigator an error occur( it is saying that homeroute should declare a screen). Here is my code:
const tabNav = TabNavigator(
  {
    Homeroute: {screen: homeStack },
    Profile: {screen: Profile},
   },
);
const mainStack = StackNavigator(
  {
    Splash: { screen: Splash },
    Home: { screen: tabNav },
  },
);

const homeStack = StackNavigator(
  {
     Home: { screen: Home},
    chat: { screen: chat},
  },
);

I'm new to react navigation, any help is appreciated.
| software         | version
| react-native     |  0.47

Is the Home component a valid React component?

Closing for inactivity. If this is still an issue, please reopen along with an expo snack repro.


WaitForCondition (mytouch4g) issue and fix

Hi,
First of all thanks for hosting this code, it is extremely helpful.
I have found one small issue that affects at least the mytouch4g.  If you switch between screens quickly (i.e browser and home screen), the wallpaper stops updating and logcat gets flooded with WaitForCondition.  If you put a short Thread.sleep (e.g Thread.sleep(10)) after mEglHelper.swap(), it fixes the issue.  Other than that, everything has worked for me so far...I've actually downloaded and modified min3d to work with your code for fun.
Thanks again,
Trevor

I tried out the implementation from JesusFreke, the issue still persists.
Tried with a NOOP-Renderer that just does nothing but it still occurs. Will have to investigate more.

I'm having this issue on HTC Desire S with Android 3.2. Sometimes I even can't open live wallpaper for preview without rebooting phone. Anyways wallpaper won't work any longer than a few minutes.
The less geometry I draw, the more stable it is. If I draw only a 3-5 objects, it is absolutely stable.  I've tried adding sleep(), it doesn't make it more stable, only causes framerate drops (obviously).
What is interesting, live wallpaper works like a charm on my Transformer w/ Honeycomb.

OK, so in my case I narrowed down this to problem with fragment shader, which misbehaved on Adreno GPU (oh boy I just love these differences between nVidia and Qualcomm's GLSL implementations :P ).
@Projekt2501 - so this issue really seems to be specific for every live wallpaper. In general, I believe, it is mainly caused by OpenGL used in non-optimal way.

I suppose you are right on this one.
2012/1/19 keaukraine <
<EMAIL_ADDRESS>

OK, so in my case I narrowed down this to problem with fragment shader,
which misbehaved on Adreno GPU (oh boy I just love these differences
between nVidia and Qualcomm's GLSL implementations :P ).
@Projekt2501 - so this issue really seems to be specific for every live
wallpaper. In general, I believe, it is mainly caused by OpenGL used in
non-optimal way.

Reply to this email directly or view it on GitHub:
https://github.com/markfguerra/GLWallpaperService/issues/11#issuecomment-3559149


Added onModAddDj, onModMoveDj, onModRemoveDj Events

I did your todo's :^)

Sweet, thanks, see comment for one last thing :)

:+1: Thank you!


Docker improvements

Hi @afh! Thank you for the PRs! Going to review them this week but so far really like the improvements you've made

Hi @miguelmota, thanks for your response here and the feedback that you so far really like the improvements I'm proposing. As I'm still largely unfamiliar with the code-base I'd appreciate any feedback on how some proposed changes could possibly be improved.
I've found cointop already very useful, so thanks for writing it!

@afh yes for sure, will let you know of any improvement suggestions and I could also use more feedback and suggestions from outside users and contributors : )

This PR looks good. Will merge.
Thanks!


Workspace create stopped at "Waiting for Agent"

Describe the bug
When i try to create workspace in azure, it creates vm without public ip but execution stopped at "Waiting for agent" then failed with timeout error after 10 minutes.
To Reproduce
Steps to reproduce the behavior:
Just added all variables & selecting repo from github
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop (please complete the following information):

OS: Linux Mint 21.3
Daytona Version: v0.48.0
Provider Version: 0.3.4

Additional context
I just changed VM specs & region

Need to start with root


Support for remote OSVR servers

I am attempting to use OSVR with a Samsung Gear mobile HMD. Our use case is a client running on the HMD, connecting to another machine that hosts all the devices the application may need - controllers, trackers, navigation devices, etc. There will be nothing connected directly to the HMD because it is impractical, thus there is no need for an OSVR server to run on the headset - reducing complexity and saving battery power.
Right now the high level API doesn't support this use case. The class osvr::clientkit::ClientContext doesn't allow to specify the host where the OSVR server is located. This can be worked around either by using the low level API (which does take a host argument) or by using an environment variable. Neither workaround is ideal - the low level API exposes Boost dependencies to the client application, making for a complicated compilation, especially on mobile platforms. The environment variable approach is not usable on Android.
Proposed enhancement
Add an additional constructor to the osvr::clientkit::ClientContext class,  the host argument accepting also the host:port format (important for servers running on non-standard ports):
inline ClientContext::ClientContext(const char applicationIdentifier[],
                                                      const char host[],
                                                      uint32_t flags);

Or, alternately, come up with a non-hacky alternative to the environment variable.
(I think host:port is already supported transparently with the APIs that accept a host)

Yes, it is - the low level API accepts that and it should even work as vrpnConnection is used underneath.

Updated the pull req for the latest master: https://github.com/OSVR/OSVR-Core/pull/331

Hello,
Is there a chance to get this trivial enhancement merged? I have encountered this again when updating our in-house OSVR copy today. The pull request is over 2 years old and doesn't conflict with anything. Right now we have to either use our in-house built copy or use a kludge with setting the environment variable from inside of the application, which is ridiculous.
If it isn't going to be merged, then I would at least like to know why so that I could accommodate that locally.
Thank you.


Added the sparse simulator sample.

The first-most link (to the sparse simulator doc) is not live yet. Will go live on Monday at 10am.
Other than that all the feedback is addressed and the PR looks ready to be merged.

I believe all the feedback has been addressed.
Please unblock/approve or provide more feedback.


Unable to mesh using hex

🔍 Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

🐞 Description of the bug
I am trying to mesh a structural model with a quadratic mesh.
maxSize = 100
minSize = 50
eshape = prime.VolumeFillType(n)
mesh_util.surface_mesh(min_size = minSize, max_size = maxSize ,generate_quads = True )
mesh_util.volume_mesh(volume_fill_type = eshape)

only n = 0 seems to work but that creates tets with the surface quad and 1,2,3 generate an error when saving to file.
If n = 1
structural_mesh = os.path.join(here, file + '_QUAD.cdb')
mesh_util.write(structural_mesh)

returns
_MultiThreadedRendezvous: <_MultiThreadedRendezvous of RPC that terminated with:
status = StatusCode.UNKNOWN
details = "Stream removed"
debug_error_string = "UNKNOWN:Error received from peer ipv4:<IP_ADDRESS>:50063 {grpc_message:"Stream removed", grpc_status:2, created_time:"2023-05-23T10:21:43.964883399+00:00"}"

📝 Steps to reproduce
See code above
💻 Which operating system are you using?
Windows
📀 Which ANSYS version are you using?
23r1
🐍 Which Python version are you using?
3.10
📦 Installed packages
xxx

@mcMunich Could you please share the input file that you are using?

I can send it but let me get confirmation from the customer and account manager.
One thing is weird I can get a plot of the mesh and even the quality, but it now fails when I try to write the cdb.
_MultiThreadedRendezvous: <_MultiThreadedRendezvous of RPC that terminated with:
	status = StatusCode.UNKNOWN
	details = "Stream removed"
	debug_error_string = "UNKNOWN:Error received from peer ipv4:<IP_ADDRESS>:50062 {created_time:"2023-05-24T13:19:55.329116117+00:00", grpc_status:2, grpc_message:"Stream removed"}"
>

@mcMunich cdb format does not support poly mesh type.  #205 will fix this.  The #volumefilltype you are using with the exception of #ansys.meshing.prime.VolumeFillType.TET all contain poly.
When you say quadratic I understand you to mean hex from the context of the discussion but if you wish to get linear quadratic tet elements you can use #ansys.meshing.prime.TetParams.quadratic.
We have only recently begun to expose hex meshing capabilities.  At 0.4.0 VolumeSweeper (Stacker) and ThinVolumeControl will be capable of hex meshing in certain scenarios.

@waltersma I was trying out the pcb example and the Volumesweeper seems to give an error:
sweeper = prime.VolumeSweeper(model)

AttributeError: module 'ansys.meshing.prime' has no attribute 'VolumeSweeper'
I think I have the most updated version;
"C:\spyderEnv\Scripts\pip" install ansys-meshing-prime[all]

Closing this issue as original question is answered.  For the second issue mentioned please open a new issue if this is reproducible after confirming this is not a client server compatibility issue (see https://prime.docs.pyansys.com/version/stable/getting_started/index.html for client server compatibility).
Thanks


How to work with the Eloquent Tenant model

I'm having trouble understanding how to work with the Eloquent Tenant model. In my usecase, the tenant is an hotel that have relations, getters, setters etc.. I am supposed to have a Hotel model that extends from Tenant? Should I extend something else?

The Tenant object is completely separate from your tenant app. The actual tenant app is not concerned with the tenant it belongs to, it's sort of unaware of tenancy.
Can you give me more information about why & how you'd need this?

I'm building a hotel management system.
The central database holds all the general data like users, permissions and hotels list (tenants).
Then for each hotel I create a separate database with data relevant only to that hotel, for example rooms, bookings, availability etc...
The reason the users are stored in the central database is that one user can manage multiple hotels. For example the user can be the hotel owner who owns multiple hotels, or even a worker that works for more than one hotel. Maybe I want to even create a user for an accauntant so he can look at financial information for multiple hotels at the same time, from a dashboard lets say.
So the user logs in to the central database, and he can switch between hotels with the same credentials. But core information about the hotel is stored in the central database, for example subscription plans, currency, address, etc...

I see. So you need a hotel model to define a relationship between a hotel/tenant and a user?

I guess I want the Tenant to behave like an Eloquent model, or at least be able to extend from it.
Like when I do Auth::user() I get the authenticated Eloquent User.
It would be nice to do Tenancy::tenant() and get the current Eloquent Tenant.

I can avoid using the tenants table completely, like this:

I wanted to 'merge' the hotels and tenants table as for me they are the same, but maybe for now this will work.

Right. You can rename the tenants table to hotels in the config, add your custom columns -- name, locale, country, timezone, currency_code ... -- to the config, and create a Hotel model in your application. If you want to use that central model in your tenant app, you can use the CentralConnection trait on it.
But you still have to use the Tenant class to create tenants and set things like timezone and currency_code for the tenant.

Now that creation of custom columns is fixed in v2.1.1, it is much easier to use your own Eloquent Model together with the Tenant model. The only thing is you need to manually update the created_at and updated_at models but I think this issue can be closed now.

your own Eloquent Model together with the Tenant model

Can you elaborate on this? The Tenant model used by the package is not meant to be used from user land.

I don't use TenantModel directly, I create my own Tenant model but they both use the same table. I just take care of not overriding the 'data' column and always use the Tenant facade for actual tenant manipulation.

Great, that's the right way. I'm thinking about removing TenantModel and DomainModel and using the DB facade instead, to make this less confusing.

That's great! I think those models are confusing because you naturally want to work with them directly and extend them. I would personally also remove the domains table entirely and use a json column in the tenants table instead. You can save an extra database query and reduce complexity at the same time! You are already doing a similar thing with the 'data' column so, why not? You can have everything you need to manage a tenants in a single table, much easier. Also for performance reasons it's not recommended to have a table with a varchar(255) as primary key.

domains table entirely and use a json column
Two things with that setup:

I'm not sure if you can find a tenant using a domain at reasonable performance
You don't have DB-level assurance that one domain can't belong to multiple tenants

Uhm, you have 2 good points there, I think you are right.

This may be help : https://infyom.com/blog/how-to-use-multi-tenant-with-single-db


"playCard" attribute for cardAction

Implement ability for a card to chain play another card to it if it's in player's hand or deck.

closed with 57f2c1893e044ffb9c3704fea9950a6110dd8a2c


Keyboard remapper doesn't work in Windows 11 emoji picker

Microsoft PowerToys version
0.49.1
Running as admin

[ ] Yes

Area(s) with issue?
Keyboard Manager
Steps to reproduce

Remap the "App" key and right control key to behave as left and right arrows.

Open Windows 11's emoji picker (Windows + . )
Attempt to navigate left and right using the remapped keys.

When using Windows 10, I could navigate in the emoji picker correctly using these remapped keys, however when using Windows 11's new emoji picker, I cannot. Regular arrow keys do allow navigation, which makes me think the problem is in PowerToys.
✔ Expected Behavior
That the arrow keys would be correctly recognized by Windows 11's emoji picker.
❌ Actual Behavior
The arrow keys are not recognized, I cannot navigate in the emoji picker using keyboard.
Other Software
Windows 11 Professional,
21H2
22000.282
Windows feature experience pack, 1000.22000.318.0

In Windows 11 the emoji picker might be picking up on these keys before Keyboard Manager does, or it has a higher elevation and we never get those keys.
Does running PowerToys as an admin fix this?

Good point, however I just tried running PowerToys as an admin and it didn't fix it.

If this worked in win10, win emoji may have changed something cause we did not.
milk try and look at this tomorrow

I came here looking for the same issue but I use AutoHotKey to remap my arrow keys (Capslock + I/J/K/L). So it's definitely something that emoji picker did that is breaking this functionality and not power toys.

I don't know if this would need a separate issue, but I have noticed that the same thing happens with the Run as administrator prompt. When the overlay pops up and asks for confirmation, I cannot use remapped keys to navigate.
Could this highlight an issue caused by a change higher up than win emoji itself, since it seems more parts of Windows 11 are affected?

Tried this on v0.61.1 and it works perfect here.
Is this still an issue?

I don't know if this would need a separate issue, but I have noticed that the same thing happens with the Run as administrator prompt. When the overlay pops up and asks for confirmation, I cannot use remapped keys to navigate.

@SeeringPhil I would say that is probably because the process doesn't seem to have acces/rights to that messagebox. Protection and such.

that could be one of the scenarios where how PowerToys does our keyboard work, it won't be able to inject keyboard hooks to prevent someone from auto accepting a UAC (run as admin prompt) for you for instance.

Cannot find your message where you're saying it works anymore but I'll try
answering the email and see if it works.
I do have an update as I just now have been able to use the computer that
had powertoys set up.
In fact, it seems to work better, but not quite like it should.
If I open up the emoji picker, I cannot use my remapped arrow keys to
navigate. However, if I press enter once (which selects the most
recently used emoji and inserts it in text) OR if I click with my mouse and
insert an emoji, I am then able to navigate using my reassigned keys. It is
really weird because I suspected that it could be that the window is not
currently in focus, but I validated this was not the case by using regular
arrow keys (well, Corsair iCue simulated ones, my keyboard doesn't have any
real ones) and with those "regular" arrow keys, I can navigate in the emoji
picker straight away without needing to insert an emoji, and also, you can
even start writing an emoji search term (so we're sure the focus is on the
window) and even then, the remapped keys won't work until you insert an
emoji.
TLDR: The remapping kinda works, but only after you press enter / insert an
emoji with your mouse or keyboard's ENTER.
I guess I'll just see if I can do the same remapping using Corsair's iCue
since this seems to work alright, but PowerToys working would be way more
versatile.
Le mer. 10 août 2022 à 07:58, Jay @.***> a écrit :

Tried this on v0.61.1 and it works perfect here.
Is this still an issue?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/14328#issuecomment-1210573316,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACFNAYOZX6AAICMRC36XH43VYOKPVANCNFSM5HS3K7TQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

This appears to be fixed! Going to close :) let us know if you still see this happening

I'm able to navigate through the emojis with:
Alt (left) + J -> Left arrow
Alt (right) + K -> Right arrow
What does your configuration look like? /needinfo

Ah yep, I can repro this! I'll reopen :)


Create New App using Yarn create nuxt-app works in dev mode but fail after being published to firebase hosting

Versions

nuxt: v 2.14.1

Reproduction
Github Repository
https://github.com/andyfensham/NuxtTest214

Template App created using yarn create nuxt-app and npx fails to run in production

Steps to reproduce
Please see all steps to reproduce.

The demo site created is : https://scadfirebase214.web.app/
What is Expected?
Site worked in Dev but does not work once uploaded
What is actually happening?
You can not navigate and the following error appears in the console.

I have managed to isolate the package causing the issue.  It is the "nuxt": "2.14.1", package.  Updating to 2.14.0 resolves the issue.  Not sure what the bug is though.

Thanks, I also got this problem yesterday after updating to 2.14.1, but only on surge.sh so I just filed an issue there. Good to know that it is not unique to surge.sh.
It works on all other servers I tried (netlify.com, commons.host and a couple of local servers).

Duude this bug almost drove me crazy , i've been trying to fight it for the past 3 days , opened a  TICKET then closed it because i thought it was coming from me ... i fixed it by copying old nuxt project node_modules folder in the new one but that was a temporary fix ..
i can confirm that setting the version of nuxt to  "nuxt": "2.14.0" does fix it

I suspect this might be related:
https://github.com/nuxt/nuxt.js/issues/7901

I suspect this might be related:
#7901

that's exactly it !


Fix build on FreeBSD

The FreeBSD port fails to install the latest version (0.6.2), with the following error:
*** Error compiling /usr/local/poudriere/ports/latest/misc/py-oterm/work-py311/stage/usr/local/lib/python3.11/site-packages/oterm/app/chat_edit.py...
File "/usr/local/lib/python3.11/site-packages/oterm/app/chat_edit.py", line 213
label=f"{tool_def["tool"][function][name]}",
^^^^
SyntaxError: f-string: unmatched [
*** Error compiling /usr/local/poudriere/ports/latest/misc/py-oterm/work-py311/stage/usr/local/lib/python3.11/site-packages/oterm/app/chat_edit.py...
File "/usr/local/lib/python3.11/site-packages/oterm/app/chat_edit.py", line 213
label=f"{tool_def["tool"][function][name]}",
^^^^
SyntaxError: f-string: unmatched [

Thank you!
I wonder how this went unnoticed...
Could you please add your change in the CHANGES.txt?

Thank you!


Update Run Plugin MDL2 Icons to Fluent Icons

Microsoft PowerToys version
0.64.1
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
Currently, the icons used for the various PT Run plugins are using the MDL2 font icons. It makes sense to once again update the icons used within plugins to be consistent with the rest of the Power Toys and Windows 11.
The previous upgrade to MDL2 icons was tracked in the issue #4040 and updated in the PR #4149
✔ Expected Behavior
I would expect to see fluent icons throughout all the plugin icons.
❌ Actual Behavior
MDL2 icons are present throughout the PT Run plugins.

Other Software
No response

@DarthAV https://learn.microsoft.com/en-us/windows/apps/design/style/segoe-fluent-icons-font#how-do-i-get-this-font
They aren't included on Windows 10 is why.  Seeing if this is allowed due to https://learn.microsoft.com/en-us/windows/apps/design/downloads/#fonts does have them.

@crutkas If it's not possible to use the fluent icons font, could we simply use transparent PNGs as used in #4149. These are still present in the current branch.
Question: how does the settings page look without the fluent icons on Windows 10? Aren't there many fluent icons throughout?

Question: how does the settings page look without the fluent icons on Windows 10? Aren't there many fluent icons throughout?

Seems to me that, in the settings page, it is as simple as just referencing the font and if the Fluent Icons are installed then it will use them, if not it will use the MDL2 icons.

Could we replace the current png files found in locations like Calculator Icon Image with either:

New transparent PNGs of the respective fluent icons
Unicode values for fluent font icons which have a fallback to MDL2 icons.

https://github.com/microsoft/fluentui-system-icons
we'd want to do a stripped down version of this TTF so we're not adding so much weight.

https://github.com/microsoft/fluentui-system-icons/tree/main/fonts is where the fonts are

I hate using png's in stead of font icons. We should use the variable theme symbol icon font. @niels9001 @htcfreek

we'd want to do a stripped down version of this TTF so we're not adding so much weight.

You know, for icons that do not exist in Fluent, we could also create a custom font with custom icons.

This was fixed in 0.76


electron version on mac sometimes fails to exit

i can hit cmd-Q and nothing happens, i have to force quit.
pretty easy to get into this state i don't have an exact repro.

based on our conversation this morning:

makeBundle and makeInstaller should depend on electronRealClean
upgrade from 36.5 to 36.12
fix the java leak and failure to ask about open notebooks bugs.


TEM-151: support for reindexing

adds support for reindexing in API via https://github.com/RTradeLtd/grpc/pull/9 (specifically https://github.com/RTradeLtd/grpc/pull/9/files#diff-a273fc40d35e6ceedd0c13e8c4d3ba05R12)
update behaviour of Store and related endpoints to accomodate Update behaviour
minor fixes:

refactoring for interfaces, cleanliness
tests for a few functions (notably Lens.Service::Update and Server.API::Index)
added logging to gRPC middleware

Codecov Report

Merging #14 into master will increase coverage by 7.59%.
The diff coverage is 54.08%.

@@            Coverage Diff             @@
##           master      #14      +/-   ##
==========================================
+ Coverage   42.53%   50.13%   +7.59%
==========================================
  Files          11       12       +1
  Lines         710      750      +40
==========================================
+ Hits          302      376      +74
+ Misses        355      308      -47
- Partials       53       66      +13

Impacted Files
Coverage Δ

search/search.go
32.96% <ø> (+2.23%)
:arrow_up:

lens.go
53.73% <28%> (+12.68%)
:arrow_up:

server/server.go
23.71% <41.02%> (+23.71%)
:arrow_up:

server/opts.go
88.23% <88.23%> (ø)

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 0cc3036...1614352. Read the comment docs.


Support for FAST pulse-with-hold, set one-shot bit on pulse commands

This PR makes improvements to the driver pulse() command for FAST hardware. Includes a test!
Pulse with hold
This PR adds two platform settings to FAST drivers: pulse_hold_power and pulse_hold_ms. When provided, these parameters are added to the hardware pulse command. FAST hardware supports a two-phase pulse action, with an initial pulse at PWM1 and a hold at PWM2. Using the hardware commands for this behavior is safer and more time-accurate than a pulse-and-enable followed by a software delay and disable.
One-shot bit on pulse
This PR changes the pulse command from 81 to 89. In FAST driver commands the 1 << 3 bit indicates to "one-shot" the driver after the configuration is written. This allows the configuration and pulse to be sent in a single command, which eliminates the need for a subsequent TN:NN,01 command (which adds network traffic and can sometimes be missed by the hardware).
As a result, the pulse method is updated to send either the pulse-and-one-shot command or the manual trigger command, not both.

Spike can do this as well. P/P3-Roc might also be able to do it. Do we have a general use-case for this? I have never seen this used in a commercial game. Where do we want it? Is this "timed enable". That might be something which we could use in a few places (i.e. eject for older troughs).

If multiple platforms can do it, I think it would definitely make sense to have it be general use.
My current use case is on drop targets, which require quite a lot of time (up to 200ms) to fully restore. I tried using just the pulse, but even at 200ms it took >80% PWM to successfully reset. With this change, I can pulse for 30-40ms and hold at 25% for 120-140ms and they reliably reset.
I had some similar behavior in Mass Effect, but I used software timers to enable and disable the coils. It was functional, but the timing wasn't as precise as I would have liked.
Happy to refactor to be more general use!

Sorry my late reply! I guess we could add a generic "enable_timed" command to our drivers. What do you think? We could default implement it using a delay and platforms which support this could use it. Would definitely be useful for drop and ball devices. Probably for more.

Closing this PR in favor of the abstract approach in https://github.com/missionpinball/mpf/pull/1613.


Object spread operator does not remove readonly when used with generics

Bug Report
Despite https://github.com/microsoft/TypeScript/issues/14058 works for non-generic types, it still remains readonly intact when generics are involved.
🔎 Search Terms
spread readonly generics
🕗 Version & Regression Information
All including beta
⏯ Playground Link
Playground link with relevant code
💻 Code
class MyClass<TProps extends { message: string }> {
   constructor(
      public props1: { readonly [K in keyof TProps]: TProps[K]; },
      public props2: { readonly message: string }
   ) {}

   func() {
      const event1 = { ...this.props1 };
      const event2 = { ...this.props2 };
      event1.message = "abc"; // error; why?
      event2.message = "abc"; // ok
   }
}

🙁 Actual behavior
Shows the error: "Cannot assign to 'message' because it is a read-only property".

🙂 Expected behavior
Should allow assignment since spread operator creates the object's copy.

Looks like duplicate of https://github.com/microsoft/TypeScript/issues/53917


Add info about reorganized le to fraud

As a NHS Admin
I want information about reorganizes legal_entities (employees.status_reason, ) were added to fraud
So that I can get analytics for those legal_entities

[ ] add reason to fraud declarations, employees #
[ ] change le reorganization, add status_reason to deactivated employees, set it as 'auto_merge' #

Pls also add info about reorganized le into Report DB.

@svetavedmed please ensure that erd-diagram has been updated

not actual anymore edenlabllc/ehealth.api#4498


compatibility issue with cuda 11.1

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): N/A
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows x64
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): 2.4.0 master
Python version: 3.8.6
Bazel version (if compiling from source): 3.5.1
GCC/Compiler version (if compiling from source):  visual studio 2019
CUDA/cuDNN version: 11.1/ 8.0.4
GPU model and memory: RTX2070 super

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
cudart library not loaded
Describe the expected behavior
cudart dll should be loaded properly
Standalone code to reproduce the issue
python
import tensorflow as tf

Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.
cudart64 library name for cuda 11.1 is cudart64_110.dll,  not cudart64_111.dll

also emit error in ubuntu

@alanpurple
Please share complete error log for us to analyse, also could you try with cuda 11.0 and not cuda 11.1 and let us know.Thank!

@nluehr has a fix for this which should land soon.

@sanjoy
thank you

Did the fix already land? Where can I find it?
Regards Geert

@geeheim It landed in the master branch on Oct 7th. https://github.com/tensorflow/tensorflow/commit/68a6fe0d984377b625b421d34f8c607d6ed73597


Update version to v0.12.7

Did I cause this build to time out? If so, by what?

You didn't, it was caused by something else.

Thanks! :+1:


Rearrange code.

Pipeline-specific functions were pushed down a package level and one reusable function was moved out of an obsolete 'grpc' package level.
@iliat please take a look at the new organization and let me know if any of the BAM stuff should be grouped differently.
https://github.com/deflaux/dataflow-java/tree/rearrange/src/main/java/com/google/cloud/genomics/dataflow/functions
https://github.com/deflaux/dataflow-java/tree/rearrange/src/main/java/com/google/cloud/genomics/dataflow/utils
https://github.com/deflaux/dataflow-java/tree/rearrange/src/main/java/com/google/cloud/genomics/dataflow/writers/bam

Coverage increased (+0.08%) to 20.343% when pulling efdf35f058e2b9baffe07ec3ac405bbde1573546 on deflaux:rearrange into 7d25de2f8be3428ff12dc8f57a967f000d804a15 on googlegenomics:master.

I would use more general terms such as analysis, validation or qc to group a larger set of  classes under.  It would be hard for folks to remember transmissionprob as package names, as in Java either they have the whole name in CamelCase, or is only one common word that is truncated, or has the whole package name as an acronym.
The Java class hierarchy provides a nice idea of how to structure an API:
https://docs.oracle.com/javase/8/docs/api/java/lang/package-tree.html
Hope it helps,
~p

LGTM for BAM, Thanks !

Coverage increased (+0.08%) to 20.343% when pulling 8b0661044b5feb116e5872a0cdb6f6212089525b on deflaux:rearrange into 7d25de2f8be3428ff12dc8f57a967f000d804a15 on googlegenomics:master.

Coverage increased (+0.08%) to 20.343% when pulling 1b92fb6754c705a3d4b602cf5434179de940855a on deflaux:rearrange into 7d25de2f8be3428ff12dc8f57a967f000d804a15 on googlegenomics:master.

The fixes look nice - thanks Nicole :)

Coverage increased (+0.08%) to 20.343% when pulling 0134a4fcd15c082b0929e9d0015f8dae0673fcfb on deflaux:rearrange into 7d25de2f8be3428ff12dc8f57a967f000d804a15 on googlegenomics:master.


rzz angle validation assumes fixed parameters

We recently added a check that rzz angle is in the range [0, pi/2] (#1949).
As noted by @ihincks, the check assumes that the angle is a number. The following code snippet fails:
param = Parameter("p")

circ = QuantumCircuit(2)
circ.rzz(param, 0, 1)

SamplerV2(backend).run(pubs=[(circ, [1])])

It produces the error:
TypeError: '<' not supported between instances of 'Parameter' and 'float'

In #1999 we only skip the validation if the angle is of type Parameter. One could expect that, if the angle is a Parameter, then we should verify that all values assigned to the parameter are in the permitted range. What do you think? (@ihincks, @jyu00)


PyPI package content

Hello.
Could you please include LICENSE, README.textile and test dir in PyPI package?
Thanks a lot and have a nice day.

This package is deprecated and not supported. Various component utilities of this package may be split out into more dedicated packages.


Implement metrics

See if we can add metrics to the Python integration.
To do
Feel free to spin any of these into their own tasks as needed.

[ ] Research how to implement metrics

[ ] OpenTelemetry metrics?

[ ] Add OpenTelemetry metrics support to the agent (include the OpenTelemetry metrics protobuf)

[ ] Agent extension

[ ] Add extension support to this project

[ ] Implement probe system? (probably necessary for the bullet points below)
[ ] Research useful Python VM metrics (add more bullet points here)

[ ] Proposal: Implement Python garbage collection metrics?

[ ] Research useful instrumentation metrics (add more bullet points here)

[ ] Proposal: Implement Celery task monitoring metrics?

Made some progress on this task, as mentioned on Slack.
First, a note: the OpenTelemetry docs say that there is no OTLP/HTTP exporter for metrics, but this isn't actually true. The exporter was implemented months ago and it's functional, but it's not documented.
Once that was figured out, I managed to get some metrics sent from the Python integration to the agent, and parsed on the agent’s side. Here’s the debug output from the agent: https://gist.github.com/unflxw/d34dd4c37a72269d5b1e1615a53b4411
Here’s the changes in the Python integration and in the Flask test setup, and here’s the (extremely work-in-progress!) changes in the agent.
The next step will be to make the agent turn these into AppSignal metrics. Some, such as counters, have a clear mapping to existing metrics. Others, such as histograms, not so much.
Finally, a caveat: in the current work-in-progress implementation, the agent's HTTP handler tries to parse the same bytes with different protobuf definitions, first as traces, then as metrics. What we're actually supposed to do, according to the OTLP/HTTP spec, is to differentiate them by the path that is requested (/v1/traces or /v1/metrics), which makes sense. We may or may not need to add these paths manually to the HTTP endpoint in the exporters -- it's unclear. Regardless, this will require some changes to the agent's HTTP handler.

Made some more progress on this. OpenTelemetry delta sum metrics are now reported to AppSignal as counters, and both OpenTelemetry cumulative sum metrics and OpenTelemetry gauges are now reported to AppSignal as gauges:

The agent branch linked in the above comment should contain the latest (still work in progress) changes.
Short term to do list

[ ] Use the HTTP path to determine whether to attempt parsing as traces or metrics (see above comment)

For compatibility, empty path (server root) should mean traces

[ ] Use data point attributes as metric tags

There will need to be a denylist for attributes, as OpenTelemetry metrics throw a lot of attributes of very little relevance.
We may want to always report a metric with and without the hostname
We may want to report combinatorials of the attributes, for small enough numbers of attributes? So users can filter the metrics by different attribute sets.

[ ] See if we can make magic dashboards out of this

There may not be much to report, using only counters/gauges, in the Python instrumentations we support. If so, do histograms first (see below)

[ ] Figure out a way to support histograms (see Basecamp)


Added a few HOWTOs to the ideas page

New addition to the Industrial Edge section for ideas.

@mbaldessari I fixed all the lint and spellcheck failures but I am still failing on html_proof step.  Can you take a look when you are back? Would love an explanation :)

Can you please rebase from latest git? I added a small change. It should now fail for you on a single addition and you can probably ignore that. Reason is that the "{:toc}" macro from kramdown is rendering menus as absolute URLs and since its not in git yet htmlproofer will fail. For now let's ignore it, I might look into it a bit more some other time (I spent some time on it but did not find a trivial fix)

Ack.  Rebased and pushed the additions.  I can see the failures but they are not marked as required checks.  I am making more changes to the verbiage today so let's hold off on the merge.


Migrate to Python 3.12, require it

I realize you support older, and may want to, I'm just thinking if it could be better to only support 3.12+ already.
What might be reasons and pros and cons? I see:

Porting to Python 3.12

Legacy Unicode APIs based on Py_UNICODE* representation has been removed. Please migrate to APIs based on UTF-8 or wchar_t*.

They want to move to UTF-8, I believe it's not yet the default encoding, but it's been an alternative redundant encoding for a long time.
When you call to or from Python, then I believe it's non-copying, you can reuse arrays in either direction, and that would be ideal for strings too and all else. What are the exceptions? Strings are UTF-8 in Julia and need to be converted (or not?), if calling Python, though probably if Julia is called with a Python string.
Also Dicts are unordered in Julia, so I'm not sure what happens. Possibly OrderedDefaultDict could be handled better, or if were Julia's default, but right now dicts can't be shared in either direction. Are there more exceptions?
Python 3.12 should work already, so you don't have to do anything per se. And PythonCall will install a Python for you (now a smaller download since a lot of deprecated/outdated stuff dropped), but if you require 3.12 or higher it will be 3.12 for now. I think you already get the latest anyway? Unless you ask for older directly or indirectly, i.e. some package forces it. So then it couldn't be used. Going forward will not be a problem.
The only point of restricting is if you add capability that only exists in 3.12, so that you can use it. Older versions of your package would still work with older Python and that could be documented, and even supported for a while.
So what do you think, is there anything new and interesting in 3.12 you want to take advantage of?
I haven't looked over everything, but there are intriguing changes (probably largest Python release in a while):
[Python already has a (limited) stable C API, that I suppose you use. Plus I now see this new unstable.]

New Features¶

PEP 697: Introduce the Unstable C API tier, intended for low-level tools like debuggers and JIT compilers. This API may change in each minor release of CPython without deprecation warnings. Its contents are marked by the PyUnstable_ prefix in names.

[..]

PEP 697: Add an API for extending types whose instance memory layout is opaque:
Add the new limited C API function PyType_FromMetaclass(), which generalizes the existing PyType_FromModuleAndSpec() using an additional metaclass argument. (Contributed by Wenzel Jakob in gh-93012.)

API for creating objects that can be called using the vectorcall protocol was added to the Limited API:
[..]
The Py_TPFLAGS_HAVE_VECTORCALL flag is now removed from a class when the class’s __call__() method is reassigned. This makes vectorcall safe to use with mutable types (i.e. heap types without the immutable flag, Py_TPFLAGS_IMMUTABLETYPE). Mutable types that do not override tp_call now inherit the Py_TPFLAGS_HAVE_VECTORCALL flag. (Contributed by Petr Viktorin in gh-93274.)
The Py_TPFLAGS_MANAGED_DICT and Py_TPFLAGS_MANAGED_WEAKREF flags have been added. This allows extensions classes to support object __dict__ and weakrefs with less bookkeeping, using less memory and with faster access.

[..]

The C API now permits registering callbacks via PyDict_AddWatcher(), PyDict_Watch() and related APIs to be called whenever a dictionary is modified. This is intended for use by optimizing interpreters, JIT compilers, or debuggers. (Contributed by Carl Meyer in gh-91052.)

[..]

PEP 683: Introduce Immortal Objects, which allows objects to bypass reference counts, and related changes to the C-API:

[..]

PEP 684: Add the new Py_NewInterpreterFromConfig() function and PyInterpreterConfig, which may be used to create sub-interpreters with their own GILs. (See PEP 684: A Per-Interpreter GIL for more info.) (Contributed by Eric Snow in gh-104110.)

In the limited C API version 3.12, Py_INCREF() and Py_DECREF() functions are now implemented as opaque function calls to hide implementation details. (Contributed by Victor Stinner in gh-105387.)

Just passing by to comment that as a maintainer of a Python-Julia downstream package (PySR), I would perhaps vote against this change, since some of my users have to use their institute cluster Python which is typically fixed to an older version.
I think it's important for tools at the upper levels of a dependency hierarchy, like PythonCall.jl, to impose as few subjective constraints as possible, as they have a cascading effect.
My suggestion would be to simply follow the "Security support" column of https://endoflife.date/python which is 3.8-3.12 currently. Older versions are not receiving security updates. 3.11 is still officially supported with security updates by the Python Software Foundation until October 24, 2027 – which would make that a good point at which to switch to 3.12 only. So my suggestion is to table this until then. Wdyt?

Older versions of Python would still work in older PythonCall.jl at least. I'm thinking for the future, to be able to rely on new features in Python, at least UTF-8 if/when it will be default. If there's nothing compelling to know we can depend on then it's a moot point.
Also I was thinking default/require 3.12, but possibly still in latest PythonCall.jl there could be a workaround to still allow older, or automatically support and then with reduced (i.e. current) functionality).
About that institute cluster Python, since it's older version than 3.12, would it not help if PythonCall can (and already does) download latest for you?

Maybe one could just wrap new features with a
import sys
if sys.version_info >= (3, 12, 0):
    ...

block? I know some Python packages that do stuff like this. I guess if you want to make structural changes then it becomes a bit bloated though, so not sure.
Re: cluster python, I agree that my personal preference is to just have my own python version. But I am aware of cases where sysadmin can get annoyed by this because it can blow up the file count on $HOME (e.g., some conda environments can have 100,000 files!), which itself slows down backups, compared to if it is a centrally managed python with common packages shared among all users. So some users are simply stuck with a particular python version and need to link it to any software they use. (This is obviously institute-dependent though!)

This intriguing UTF-8 bug on Windows was fixed, and backported, but only back to 1.10:
https://bugs.python.org/issue37871
I didn't confirm it could be triggered by Julia, but very plausibly, so at least recommend 1.10 or later?

Maybe this isn't written down anywhere and should be but I'm aiming to remain compatible with all supported versions of Julia and Python unless there are highly compelling reasons not to.
I'm not aware of any sufficiently compelling functionality in 3.12 (including what's pointed out above) so I'll close this issue.


Can't delete widget from dashboard

Hello!
I want to pay your attention, that when you create new dashboard, you can't delete from it widgets. In manageiq-botvinnik you can it do. It is very uncomfortable
Manageiq-capablanca(with issue)

Manageiq-botvinnik(without issue)

Best Regards.

Issue was fixed on upstream master in https://github.com/ManageIQ/manageiq/pull/5642. It seems this fix did not make it to latest capablanca build.


Update javadoc link for api-common-java.

@elharo, we have a disconnect between our google-cloud-clients/pom.xml use of versions for docs, and the version of the actual library that's defined in the bom.  Ideally, we only specify the version once.

@elharo @sduskis I can look into that as part of the javadoc cleanup issue I created last week: #4223


Fix: e2e test failes due to webhook not ready

Purpose of this PR
Proposed changes:

Wait for the webhooks to be ready in e2e test CI

Change Category
Indicate the type of change by marking the applicable boxes:

[x] Bugfix (non-breaking change which fixes an issue)
[ ] Feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that could affect existing functionality)
[ ] Documentation update

Rationale
The e2e CI occasionally failed due to webhook not ready.
Checklist
Before submitting your PR, please review the following:

[x] I have conducted a self-review of my own code.
[ ] I have updated documentation accordingly.
[x] I have added tests that prove my changes are effective or that my feature works.
[x] Existing unit tests pass locally with my changes.

Additional Notes

/assign @yuchaoran2011 @vara-bonthu @jacobsalway

Acking the review assignment for tomorrow and thanks for taking a look at this before me

Hmm I've run it a few times locally and was able to replicate the connection refused error once but not after that. I wonder if it's flakey depending on the exact poll timing vs when the bundle was updated and adding some buffer time might help?

After you create the webhook configuration, the system will take a few seconds to honor the new configuration.

https://kubernetes.io/docs/reference/access-authn-authz/extensible-admission-controllers/#configure-admission-webhooks-on-the-fly

I added another check to ensure that webhook service endpoints are not empty.

I did 10+ e2e tests locally, and all of them complete successfully.

Noting for posterity: we discussed this issue/change separately and think the flakiness is due to the MutatingWebhookConfiguration being created by Helm before the controller has generated the certs/the deployment or service objects are ready.
I'm happy to approve this PR as is/just add a time.Sleep buffer to fix the e2e flakiness if it's an issue, but we could also find a better fix

/lgtm

/approve


Adds support for Kong Workspaces

Hi @hbagdi,
Starting small here but wanted to keep the ball rolling on getting more EE support in go-kong per #17.
This adds basic Workspace CRUD capabilities as well as a helper function to manage testing CE vs. EE.
This does not include the management of Workspace Entities -- I'd like to keep PRs small if possible.
Thanks,
Tyler

Please rebase on latest master.

Tyler, can we please handle one endpoint at a time? Can you open a separate PR for workspace entity? That code requires more review it seems. The workspace code was almost ready to go but the entity work is blocking this PR.

There was a leftover WorkspaceEntity in kong/types.go. I removed that and merged in manually.
Thank you!
Let's move to the next one.


OLM: Subscription/InstallPlan for namespace scoped operator with required for cluster only scoped operator fails to install required operator

Bug Report
What did you do?

Create a Namespace scoped operator with a required api/csv that is part of a Cluster only scoped operator, such as serverless-operator.v1.0.0
Deploy the Namespace scoped operator using OLM Subscription, allowing OLM to emit an InstallPlan to install the required api/csv operator

What did you expect to see?

Both Operators are installed correctly

What did you see instead? Under which circumstances?

Required api/csv Operator fails to install with message

Failed: OwnNamespace InstallModeType not supported, cannot configure to watch own namespace
It appears the parent is passing down its InstallMode to the required operator in the InstallPlan, even though it is not supported. OLM could possibly figure out what InstallModes are supported, and choose another.
Environment

operator-lifecycle-manager version:

OLM 0.11.0 on OCP 4.2, also tried 0.12.0 with override

Kubernetes version information:

Kubernetes Version: v1.14.6+2e5ed54

Kubernetes cluster kind:

OCP 4.2
Possible Solution
Additional context

Hello @dacleyra,
Thank you for submitting an Issue.
Based on the problem you described, it appears that you've created an OperatorGroup where its spec.targetNamespace is to equal the metadata.namespace field. This configuration requires that any operators deployed within support the OwnNamespace installMode as defined in operatorgroups.md:

OwnNamespace: If supported, the operator can be a member of an OperatorGroup that selects its own namespace

Based on the logs you provided, it does not appear that the operator providing the required APIs supports the OwnNamespace InstallMode. The operator that owns the required APIs must be deployed in an OperatorGroup that matches an InstallMode that it supports.

Yes, by installing a namespace scoped operator, the OLM UI creates a Subscription and the OperatorGroup where its spec.targetNamespace is to equal the metadata.namespace field, as you say.
The problem is that that in OLM the Subscription automatically generates an InstallPlan with an unsupported InstallMode for the required apis / operators.

Unfortunately, the OLM resolver simply doesn't take into account the OperatorGroup settings (AKA InstallModes) during resolution. So it is unable to tell if this operator is not installable in this specific namespace. We currently don't have a plan to change the resolver to do so. However, we do have a plan to "descope" the operator so that they will become singleton operators. As a result, the InstallModes will be obsolete in the future. You can take a look at the issue #2437 for more information.


minimum redemption amount required to conversionRate UT Wei

for minimal precision of unstaking 1 ST Wei, we require minimum redemption amount to be required to be conversionRate UT Wei
fixes #39

@benjaminbollen, I struck out my discussion above w/r/t rounding because I just came to where you came to the same conclusion elsewhere.

Yes, @jasonklein current logic does not hold the conversionRate on the utilitychain side; this is a bit of an unfortunate UX; but we are still talking about negligible amounts; so negligible that it is hard to phantom how small;  I would rather put in a more reasonable minimum redemption amount related to the actual cost to unstake; so we can open an issue for that:
microtransactions in BT is not the same as unstaking micro-amounts; even unstaking O(1) BT will not be cost effective.

@benjaminbollen, to confirm: (1) the solution is to place a requirement on the value chain that is not mirrored on the utility chain and (2) that solution is preferred over a comment about rounding/floating point math?

Correct; also very explicitly lets discuss this as a new issue and move this forward; merge and move


NuGet packages do not include .js.map files

The bootstrap JavaScript files included in the NuGet packages include references to matching .map files at the end; however, the .map files are not included in the package. Chrome therefore tries to download the file from the site using Bootstrap, which will result in a 404.
I will submit a pull request with a fix to the .nuspec files shortly.

I think it was @supergibbs who added NuGet support originally—any chance you can confirm and review this?

Looks good to me! Is there a beta.3 coming out soon or should I update the beta.2 package?

I just shipped Beta 3 :D. But, we can get this in v4 stable—hoping for 1-2 weeks out on that.

Oh awesome! I'll update the nuget package now and hotfix this patch locally so the map files are included.

beta.3 tag but includes map files:
https://www.nuget.org/packages/bootstrap/4.0.0-beta3
https://www.nuget.org/packages/bootstrap.sass/4.0.0-beta3


update travis.yml to post to slack

reconf travis to post to slack
Description
Testing

[ ] I created new unit tests
[ ] All unit tests pass
[x] I have tested manually

Have you updated the README or other relevant documentation?

[ ] Yes
[ ] Not yet. I will do it next.
[x] Not relevant

I'll close this PR and create a new one so the checks would run again


(Build) Fail on dev branch.

This issue is created automatically by CI.
Linux Build State: failure
Windows Build State: success
Commit: f678952e9a9e811ee80b00bcf15e3a5936099851
Commit Branch: dev
Commit Author: Monika0000

Auto-closing as build in this branch is successful now. Commit: c1fa043bef67f16c3ed57ce2e68f73ba379e273b


Creation time is not available on this platform currently

Creating backup fails with the following error:
Error: Failed to truncate target directory

Caused by:
    creation time is not available on this platform currently

I really don't know why this happens.
Running on Linux 5.10.28-Unraid

I'll push out a fix for that soon

Alright, I pushed out a fix. Download the latest version on crates.io using cargo install cuprite.

Now it works, thanks!


Request: when NO_CDN=true, How to implement Code highlight Theme?

How to implement Code highlight Theme?
This function is disabled when the environment variable NO_CDN=true.
Could someone help me?
In Custom script session I can add some script from Code highlight Theme,
but I don't know how to apply it to my code block.
thanks

Sequece

Building NO_CDN image

https://github.com/weseek/growi/blob/v5.0.6/packages/app/package.json#L12
https://github.com/weseek/growi/blob/v5.0.6/packages/app/package.json#L51
https://github.com/weseek/growi/blob/v5.0.6/packages/app/bin/download-cdn-resources.ts#L26
https://github.com/weseek/growi/blob/v5.0.6/packages/app/src/services/cdn-resources-service.js#L53

CodeMirror themes in cdn-manifests.js will be downloaded into /static/styles/cdn

When an user select theme

https://github.com/weseek/growi/blob/v5.0.6/packages/app/src/components/PageEditor/CodeMirrorEditor.jsx#L416

Load CSS on demand from /static/styles/cdn

@sevenjay So currentry, GROWI with NO_CDN=true couldn't load theme styles from Custom script URL even if you added.

GROWI v6 is no longer supported to change the code highlight theme.


Add Copyright to generated Angular files

If you create files with ng generate they do not contain the copyright.

Done in V2


Typo and docstring

minor, noticed while reviewing other PRs

Changes Unknown when pulling 89cc6c6fc9eddeeedb7dc878782fb7d1a2df9ea0 on adamrp:typo_and_docstring into ** on biocore:master**.

Coverage remained the same at 78.4% when pulling 89cc6c6fc9eddeeedb7dc878782fb7d1a2df9ea0 on adamrp:typo_and_docstring into d5e9f280c59f893fc49b969f6f2ab1b2631f0d40 on biocore:master.

:+1:


[incubator/fluentd] Generalize chart to work with different backends

What

Made working with environment variables flexible
Made Fluentd be daemon set
Remove health check
Parametrise config mount dir
Mount host logs into the container

Why

Make chart be flexible enough to work with different fluentd containers (ex. fluentd-cloudwatch)
To store secrets private
DeamonSet allow collecting logs from all nodes
Not all fluentd containers support health checks
Different fluentd containers expect configs in a different paths
Allow collecting host logs with fluentd in_tail

/assign @prydonius

/assign @rendhalver

@monotek we will use https://kubernetes.io/docs/tasks/manage-daemon/update-daemon-set/

@osterman @mattfarina  @monotek @rendhalver
I addressed your comments and provide readme with examples how to use this chart with different backends.
Hope you would find chart flexible enough and ready to release

@goruha
I still think daemonset or deployment should be optional.
Maybe one don't wants to use it for kubernetes / docker logs but for sending app logs to the fluentd forward plugin on port 24224.
In this case 1 replica as deployment would be enough.

I tend to agree with @monotek here too.
We should have the option of running as a DaemonSet or a Deployment.

Imho we should also provide 2 configmaps for optional deployment / daemonset.
If deployment is used I'm pretty sure most of the current config is unwanted.

I think we try to mix different purpose charts.
My suggestion is to have 2 charts

fluentd - for fluentd as independent software (so leave this chart as is and close PR)
fluentd_kubernetes - run fluentd to collect kubernetes logs and forward it to different backends (create a new chart with separate PR)

What do you think about ?
@monotek @osterman @mattfarina @rendhalver @prydonius

Imho if differents fluentd images has to be used (output plugins) we should also use different charts.

We have separate charts for the different usages.
We currently have multiple charts for the logging that send to different backends.
Calling the logging chart fluentd_kubernetes isn't the best choice of name, fluentd-logging or fluentd-container-logging would be a better name and more explicitly explains what it does.


Create test123123123

Collaborator: Review for this pull request was created. Link to review

Collaborator cannot find remote user mapping for pull request creator, review created by system administrator


Python 3.2 support required

Python version 3.2 finally resolves the long-standing issue of Unicode support in WSGI headers (PEP 3333). Thus, it is now possible (and desirable) to port Werkzeug to Python 3 and embrace the future.
See also: http://docs.python.org/dev/whatsnew/3.2.html

It would be nice to clearly state that 3.2 wont ever be supported, in a front page somewhere.
I see http://werkzeug.pocoo.org/ says "Supports Python 2.6, 2.7 and 3.3." , which should be "Supports Python 2.6, 2.7 and 3.3+."
And maybe explicitly fail on cpython 3.2 during setup.py by using u'...' in setup.py .. as I believe that is the main problem with supporting 3.2 - all u'...' would need to be removed (and unicode_literals added), which is a lot of churn for little benefit.

@jayvdb This will be resolved in the upcoming rewrite of the websites for Werkzeug/Flask/Jinja.


Update readme.txt

add note on how to get the jar

This repository is a read-only sub tree split from the official IntelliJ community edition repository. To get any changes made to fernflower you should either file a bug report on Jetbrain's Youtrack and submit a patch file or you can make a pull request on the official IntelliJ community edition repository.

OK thanks.
On Thu, Jan 28, 2016 at 6:19 AM, Christopher Carpenter <
<EMAIL_ADDRESS>wrote:

This repository is a read-only sub tree split from the official IntelliJ
community edition repository. To get any changes made to fernflower you
should either file a bug report on Jetbrain's Youtrack and submit a patch
file or you can make a pull request on the official IntelliJ community
edition repository.
—
Reply to this email directly or view it on GitHub
https://github.com/fesh0r/fernflower/pull/9#issuecomment-176182941.


Exception in AbstractBladeBlock with ACF 5.8 final

Since upgrading to ACF 5.8, an Exception is thrown (5.8-RC2 was working fine)
Fatal error: Uncaught Symfony\Component\Debug\Exception\FatalThrowableError: Argument 1 passed to Itineris\AcfGutenblocks\Util::Itineris\AcfGutenblocks\{closure}() must be of the type string, null given in /var/www/html/wp-content/plugins/acf-gutenblocks/src/Util.php:16

Apparently className in AbstractBladeBlock::renderBlockCallback() is no longer an empty string but null, which screws the type declaration of the anonymous function in Util::sanitizeHtmlClasses().
The issue can be fixed by removing the type declaration and casting the variable to a string inside the anonymous method

Got this with ACF Pro 5.8
Undefined index: className in /acf-gutenblocks/src/AbstractBlock.php on line 38
Apparently, when empty $block['className'] is not set.

Got this with ACF Pro 5.8
Undefined index: className in /acf-gutenblocks/src/AbstractBlock.php on line 38
Apparently, when empty $block['className'] is not set.

Same here with ACF Pro 5.8.
I'm getting a similar Notice when trying to use "AbstractBladeBlock"
Notice: Undefined index: className in /acf-gutenblocks/src/AbstractBladeBlock.php on line 32
Then a Fatal Error
Fatal error: Uncaught TypeError: Argument 1 passed to Itineris\AcfGutenblocks\Util::Itineris\AcfGutenblocks{closure}() must be of the type string, null given in /acf-gutenblocks/src/Util.php on line 16

I have updated the pull request to fix these notices too, @nicksull @alfacommunication-alessandro could you test the changes and confirm that they fix the errors/notices?

@tbruckmaier
Fix confirmed!
Thanks

@tbruckmaier confirmed! Thank you!!


google_play_services_widget.dart not found

🐛 Bug Report

Expected behavior
Reproduction steps
Configuration
Version: 1.x
Platform:

[ ] :iphone: iOS
[ ] :robot: Android

google_play_services_widget.dart not found at pages

Hi,
This file has been removed since version 2.0.0.
please upgrade or pull the latest version.
You could find the latest version HERE if you need it.
please use the google api availability plugin, here.


Cookie import failed: JSON Query to graphql/query: HTTP error code 401.

When executing copokies.py
I get using cookies from 'right path'
and then Cookie import failed: JSON Query to graphql/query: HTTP error code 401.

Please provide a screenshot so I can better understand the problem you're facing and to take a look at the complete error message.


allow typeLength to come from opts.column when decoding FIXED_LEN_BYTE_ARRAY

Problem
typeLength is present in column options but decoding is throwing an error.
thrown: "missing option: typeLength (required for FIXED_LEN_BYTE_ARRAY)"
options object for reference:
    {
      type: 'FIXED_LEN_BYTE_ARRAY',
      rLevelMax: 0,
      dLevelMax: 1,
      compression: 'SNAPPY',
      column: {
        name: 'BLOCK_NUMBER',
        primitiveType: 'FIXED_LEN_BYTE_ARRAY',
        originalType: 'DECIMAL',
        path: [ 'BLOCK_NUMBER' ],
        repetitionType: 'OPTIONAL',
        encoding: 'PLAIN',
        statistics: undefined,
        compression: 'UNCOMPRESSED',
        precision: 38,
        scale: 0,
        typeLength: 16,
        rLevelMax: 0,
        dLevelMax: 1
      },
      num_values: { buffer: <Buffer 00 00 00 00 00 00 27 10>, offset: 0 }
    }

using parquet-tools schema here is the schema for this column:
optional fixed_len_byte_array(16) BLOCK_NUMBER (DECIMAL(38,0))

The parquet file is a direct export from snowflake and the data type of the column is NUMBER(38,0).
Solution
I traced through the code to find where the decode was erroring and added the ability to take the typeLength from column in the column options when it is not present at the top level.
Change summary:
see above
Steps to Verify:
decode a parquet file with this type of field.

Testing again with float16_nonzeros_and_nans.parquet and float16_zeros_and_nans.parquet and they do show this fixing things. I'm about to get up a pr that tests all the various reference tests, so we should just be able to uncomment those two and that shows this works. PR coming soon to fix that, then I think we can update and merge this one

apologies for the unresponsiveness - i'm away for a bit. thank you for moving this forward 🙏

No problem. Thanks for the PR! This was released with v1.5.0


Fix: dotenvx precommit hook should check subfolders

What does this PR include?
This PR includes a fix to #333 which includes more context on the overall issue.
I used the Ls class in src/lib/services/ls.js to find all the .env* files in the repository as part of the precommit hook. If any of these files are not encrypted the pre-commit hook will throw.
The fix was actually pretty easy to implement, I just had to fight with the tests for awhile 😅
Let me know if there's anything else I should clean up before we can merge.

thanks @MykalMachon ! just run npm run standard:fix to convert to standard formatting and this PR is ready.

actually, i'll take care of that. merging now. thank you! this will be huge for so many people!

released as 1.10.1

hmm @MykalMachon, maybe we went too aggresive:
<EMAIL_ADDRESS>tests/.env not encrypted (or not gitignored)

now i can't commit to dotenvx with its own precommit 🤣
because tests/**/.env*

Argh @motdotla , I don't know why I didn't think of this when testing.
I'm wondering if there's a way we could add a flag to dotenvx ext precommit --install that would allow you to set "paths to exclude" in the pre-commit hook?
For example:
dotenvx ext precommit --install --ignore-path "tests/**/*"

and that would simply set the pre-commit hook command to be
dotenvx ext precommit --ignore-path "tests/**/*"

This would ignore any .env files in your repository that match the ignore-path regex string?
I'm open to other possibilities here, any ideas?


Dashboard Stats History start time advances only for 10m storage duration option

When anything other than 10m storage duration is selected, the start time remains unchanged, the data keeps on accumulating and the graph gets squeezed.
When 10m storage option is selected, the start time advances, and the graph shows the last 10m's worth of data.
v19.46.

Additional info: the "Stats history duration" option actually has 2 "10 minutes" on the slider. One when the slider is to the leftmost, and one a single tick from the left (that is, the first two values are both "10 minutes"). When the second "10 minutes" is selected, it behaves as outlined above, too. The correct behaviour is observed only when the slider is in the leftmost position.

Okay, I finally figured this out. It's neat now! Give it a try with latest 19.47-RC3 (can be upgrade from theme configuration page).
Thanks for your feedback!

For anyone trying to find this configuration panel and was completely lost like I was the only place I could find to configure the theme was: Bell icon (top right) to open the right side panel, then the gear (bottom right), then theme configuration.
I couldn't find it in the webmin > webmin configuration > webmin themes > anywhere in there.

Hello,
Thanks for the heads up!
The option to configure duration time was removed after switching the stats to use a standalone service accessible via WebSockets. This change was made primarily for optimization reasons. A duration longer than 1 hour can be problematic for performance on average systems, as storing history stats for 1 day can lead to inefficiency in reading and writing data. The file size can reach up to 7-9 MB.
However, I may reconsider this in the future and make the option available again, clearly pointing out the downsides. Currently, you can set this option to a maximum of 1 hour by running the following command in the browser's console:
settings_sysinfo_real_time_stored_duration = 3600;
theme.config.save();

If you're interested in running more tests with lifted sanity checks (i.e., 3600 seconds), let me know.

This feature has been implemented for the next release! There will be an option in theme settings to select the retention period for live stats storage.
https://github.com/user-attachments/assets/d907fd01-0ca8-4cfa-9aac-46cbaf7ebd40
It can already be tested using the nightly version of Webmin.


Question on FragmentViewPager

First off, great work! I was checking this library out and works pretty neat :)
From the readme your todo has plans for a FragmentViewPager (can't wait this!! :) )
I was curios on how you'd be implementing it given that the RecyclerViewPagerAdapter works with recycling RecyclerView.ViewHolder but what you'd want is a Fragment instead? Ideas?
Thanks a ton

I want to use a Fragment container  view in RecyclerView.Adpater which support replacing fragment  when position changed.

I have implement an FragmentStateViewPager, you can test it on dev branch now.


New version: OpenWhisperSystems.Signal version 6.43.1

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree


Management command does not work for Heroku

This is not a bug but a limitation of the current system.  That is heroku depoloyments cannot benefit from the approach being used with the management on|off command.
Any execution of heroku run manage.py will be run on a separate worker dyno, not the web one. Therefore the lockfile is set but on the wrong machine.
It would be nice to have a dynamic approach that works for Heroku.  It seems like the most obvious thing is to maintain the flag in a DB table instead of in the filesystem.
Another possible workaround is providing a a staff-authenticated URL route which togged the maintenance status by adding/removing the file.

+1 for this, or at least mention in the README that it isn't Heroku-friendly.

Another possible workaround is providing a a staff-authenticated URL route which togged the maintenance status by adding/removing the file.

Here is how it can be done:
from maintenancemode.utils import get_maintenance_mode, set_maintenance_mode

set_maintenance_mode(True)

if get_maintenance_mode():
    set_maintenance_mode(False)

Sorry guys, it's my fault, he just made copy and paste from my lib: https://github.com/fabiocaccamo/django-maintenance-mode/


fix(style): Clean up custom CSS, revert to default bootstrap

Removed a lot of unnecessary custom CSS in HUD. Just double check that everything still looks normal, mostly in the navbar.

@hoganmaps

poked around in ie9 and latest chrome, looks good to me :+1:

:+1:


feat(plugin-esbuild): move webpack into devDependencies

PR Details
Move Webpack Into DevDependencies to reduce package install size.
Description

update import xxx from "webpack" to import type xxx from "webpack"
update package.json (which moves webpack in devDependencies)
create several utils function  instead of use webpack file matcher.

Related Issue
https://github.com/modern-js-dev/modern.js/issues/485
Motivation and Context
Reduce install size. Improve stabilization.

PTAL @leeight

PTAL @leeight

Good job, thanks for the 1st PR.

PTAL @leeight

Please update the lock file.

PTAL @leeight

Please update the lock file.

Done.

🎉


Two Identity objects (core and propertybundle)

In v0.1.0 of the ontology there is a core "Identity" object and a propertybundle "Identity" object.
These should be renamed. The glossary terms of the ontology should be unique to make supporting tools work effectively and avoid confusion for mappers.

In v0.2.0 there is just one Identity


Open-generic implementation may be wrongly considered for service with more type args

class A<X> : I<X, B> {}
interface I<X, Y> {}

In the example above A<> is considered for any I<,> but should be only for I<,B>

fixed


chore(main): release 0.4.0

:robot: I have created a release beep boop
0.4.0 (2023-05-11)
Features

Updates supported resource types (4) (#134) (2e466f2)

Bug Fixes

removes remains of DeletionPolicy we are required to set during import (#126) (02843c5)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/udondan/cfn-teleport/releases/tag/v0.4.0 :sunflower:


Add more chefspec tests

Add some missing spec tests to cover more chef resources,
but it reports some same untouched resources repeatedly
for many times.
Fixes issue #190

I think this is in reference to something like this: https://github.com/jjasghar/rabbitmq/pull/168/files#diff-55bf87238c9b8af164c5133a60721f12R169

@wenchma I tried fixing these errors, couldnt fix it :(.
Can you rip out those tests, so we can get this merged in?

updated the PR for spec tests

~/rabbitmq$ rspec spec/
....................................
Finished in 1 minute 28.43 seconds (files took 4.51 seconds to load)
36 examples, 0 failures
ChefSpec Coverage report generated...
 Have a fantastic day!

@jjasghar  success to run rspec on my local env, but the travis CI build failed,  it confused me.

 I'm not and i'm seeing errors.
I'm gonna close the issue #190 so we have the conversation in one place.

@wenchma can you rebase this guy and see if travis'll pass now? I think i've fixed things upstream.

@jjasghar Ok. I will rebase it and update the PR

have updated the PR, and the coverage result:
Finished in 1 minute 28.65 seconds (files took 4.43 seconds to load)
38 examples, 0 failures
ChefSpec Coverage report generated...
 Have a fantastic day!


Take into account special characters

GPT2 has only 1 special character for EOS, but other models like Bloom may have BOS too. This, together with the fact that the 0 index is masked in the loss, may result in degenerate model behaviours

bloom's padding token is 3, not 0. we now take this into account.


View logs from the app container by default, not the init container

View logs from the app container by default, not the init container:

The data returned in the API  projects/{projectID}/environments/{environmentID}/resources/{resourceID}/components  requires the value of resource.name.

Environment

Walrus version: main 1a7081b174c9781533116ce301abd5bc46d4043a
UI version: dev-71a5388

Test Result: pass


[3.5] draw_rulers on mac OSX causes error Invalid polygon data, triangulation failed

Godot version
3.5
System information
Mac osx
Issue description
In Godot 3.5, on a mac OSX, opening the editor can cause an endless print of Invalid polygon data, triangulation failed. which causes a massive performance degradation.
Project Settings -> HiDPI is enabled. No external monitor. Grid is enabled in the 2d editor. Initial scene is 2d.
Error is printed from this stack trace:
VisualServerCanvas::canvas_item_add_polygon()
VisualServerCanvas::canvas_item_add_line()
CanvasItemEditor::_draw_rulers()

Example points that cause this triangulation failure are:
Vector2(22.5, 4473933.5),
Vector2(22.5, 4473933.5),
Vector2(30.0, 4473931.5),
Vector2(30.0, 4473933.5)

You can see the endless printing in https://github.com/godotengine/godot/issues/64337
Steps to reproduce
Open editor with hiDPI enabled, grid enabled, view the 2D canvas grid.
Minimal reproduction project
No response

Our speculation here as to why this is failing is due to some precision change, (I believe at some point there are float/double changes in a lot of geometry areas) This is a quad that produces failure:
Vector2(22.5, 4473933.5),
Vector2(22.5, 4473933.5),
Vector2(30.0, 4473931.5),
Vector2(30.0, 4473933.5)

Steps to reproduce would be have the engine try to triangulate the above quad, in which it will fail.
These are some of the quads that the editor's _draw_rulers ends up trying to draw.

@jasonwinterpixel @jordo Can you reproduce this in any of the 3.5 betas and RCs to determine when the regression started?


windows 10/11 not working.

neovim-0.9.0 startup clean way.
but it is not working.

Your setup is very peculiar, you should look into how to create a proper init.lua or init.vim.
:set statuscolumn doesn't do anything. You should source the script 1.lua (in your setup) to initialize the plugin.

I sourced 1.lua， and setup from:

but i can't get what i want.

i want to click statuscolumn to fold indent.

OK, it looks like you have properly configured the plugin if you :set statuscolumn returns %!v:lua.StatusCol(). What exactly doesn't work? Is your 'foldcolumn' set? Do you have any folds in the buffer? Do the foldmarkers show? Does only clicking not work? Please be elaborate when opening issues. I can't gather from those pictures what isn't working.

thanks, it is because i haven't set foldcolumn.

like this.


Is this message in verbose output significant?

I am seeing this in output
[LightGBM] [Info] Finished loading parameters
[LightGBM] [Info] Finished loading 1001 models
[LightGBM] [Info] Finished initializing prediction
[LightGBM] [Info] Data file /Users/.../dev/snexample/models/9ogh6b9bl344q13keqvq3k7bco.test doesn't contain a label column
[LightGBM] [Info] Finished prediction

Is the missing lable column significant?

No worries about this .This says that the lightgbm expects this to be a prediction dataset as it does not contain a target variable. in the beginning


Add GetUserMedia test case for verifying media capture APIs could access

from localhost.
Impacted tests(approved): new 1, update 0, delete 0
Unit test platform: Crosswalk Project for Windows 20.49.525.0
Unit test result summary: pass 1, fail 0, block 0
BUG=CTS-1714

@Honry PTAL

Who is the author of this sample? Does he come from Intel?

It's from webrtc team and Google.

LGTM


ORC-1632: Add test for count command

What changes were proposed in this pull request?
Add unit test for count command of orc-tools.
Why are the changes needed?
Improve code test coverage and avoid regressions.
How was this patch tested?
Add UT.
Was this patch authored or co-authored using generative AI tooling?
No

I merged #1825 .  Please rebase once more, @cxzl25 .

Merged to main/2.0. Thank you, @cxzl25 .


Create an admin interface to add new posts

Use @oscargodson's epic editor for this.

Linkage: https://github.com/OscarGodson/EpicEditor

Yes, fork it an implement it ;)

"EpicEditor UI" was a project i was thinking about making that would come with a file browser, form setup, etc. Basically a drop in UI for admin interfaces

@OscarGodson How would you implement a file browser? Would you expect an API of some kind to list pages?

EE currently stores all "files" in a localStorage JSON string. (this is how you can do ee.open('random-file')):
{
  "files": {
    "example-1":"#EpicEditor\nThis is some default content. Go ahead, _change me_."
  }
}
```

It'd just read from that. If you're saving to a DB you could just overwrite this LS object too by just saving this blob or saving and converting to this format.

I'd prefer an easy alternative. Define a .save() method on the text, and define a .load() method on the editor. That way we can just load in a markdown file, edit it, and save it.

@Techwraith You should take a look at my API ;) http://oscargodson.github.com/EpicEditor/#api-save
Its specifically for that. This is a developer preview and i haven't spent much time on the API yet, just the drop-in-and-go part, but save does work (change the value of the editor via some random AJAX call and do save())

So, @Techwraith to make it easier to visualize i made a sample GUI to your app:
Demo: http://jsbin.com/awupiw/
Code: http://jsbin.com/awupiw/edit#javascript,html
Features:

"Default Page" ready for editing on first load
Add new pages
Remove pages
Edit pages

Thats epic. Now all I need to do is have it use a (as yet un-made) api to load, create, and save pages. I'm working on turning Scotch into a Geddy app soon, so I'll be sure to build this in.