4.3

https://github.com/hibernate/hibernate-orm/blob/b943525c80b411c9fa1f66b44f8a5b14f928bf34/hibernate-core/src/main/java/org/hibernate/internal/util/xml/DTDEntityResolver.java
In ths  DTDEntityResolver.java
if systemId(line number 64)  is not startsWith   HIBERNATE_NAMESPACE   and  not startsWith   OLD_HIBERNATE_NAMESPACE   and   not startsWith   USER_NAMESPACE .then  the code
should Display prompt  in  last  else  。like：
else{
log.error( "your hbm dtd HIBERNATE_NAMESPACE  is not the same as   dtd file in  package   org/hibernate/, read the Hibernate Migration Guide!" );
}
//The first time I used GitHub, and my English is  very poor。   It is just what I want to say  Thank you

There are an awful lot of commits here for something so minor.  Did you start with master and then try to target 4.3?

Closing this as I am pretty positive it is simply a poorly targeted Pull Request.  My guess is that you based your work on master, but are now targeting 4.3.  That will not work.


added documentation for functions

Fixes #11592
Added function documentation for /etc/hosts
Signed-off-by: Jason Smith<EMAIL_ADDRESS>

Due to the pkg/networkfs rework from another PR, this needs a rebase.  Should be straightforward as the file didn't change, just its location!

@jasonrichardsmith were you able to get this rebased?  do you need any help?

Thanks @jasonrichardsmith! I'm going to carry this for the rebase (which means you'll still get credit for the original change).

Closed by #12428.


This issue is for API mutation from graphQL

This is from Web Interface
:1st_place_medal:

Testing the comment

And Again

There it is !!

Mutation

Issue is Open, Closing it thorugh API

Query !!

Viola :trophy: :1st_place_medal: Issue Closed


Question: Unmount component

I have a problem with states inside react-modal. I have couple of components inside react-modal. After I close this modal and reopen again, states of internal components still the same (for instance, input values do not reset after modal toggling). Is there such a possibility to reset all state inside react-modal after close? I know it should be done inside componentWillUnmount but as far as I understood react-modal hides instead of unmounting.
My current solution is - do not render modal until open.
state = {
 isModalOpen: false
}

// somewhere in component
{ isModalOpen && <Modal isOpen={ isModalOpen }  /> }

It is possible that maybe Modal.js#L113 and/or Modal.js#L129 are skipping some updates.

@ajfuller any thoughts on this one?

@enheit is this with the latest 2.0.6 release? Just to make sure we're reproducing the issue correctly, could you create a minimal reproduction in a repo or on https://codesandbox.io/? Thanks!
@diasbruno it should only be preventing render if the state is different, so closing and reopening should trigger the renderPortal method.
We do have a blindspot in our tests for state changes of modals, so maybe there is something we're missing there 🤷♂  (working on a PR that I should have out shortly)

@enheit I've created the following codesandbox: https://codesandbox.io/s/X4KQq22A
I'm not seeing the behavior as described with the latest published version. Are you rendering anything differently from my demo?

@ajfuller I use 1.7.1 version of react-modal. So, I need to update lib and review it again.
Does 2.0.6 release has incompatible changes with 1.7.1?

@enheit It should be backward-compatible. The major version is just the modernization of the js syntax used.

@diasbruno thanks for reply. Here is my codesandbox example: https://codesandbox.io/s/oQjMMQjMX
It looks weird. I select 1.7.1 version of react-modal and 'ControlledInput' invokes componentWillUnmoun.

This behavior can happen when removePortal is called or when the ModalPortal is ready to close which will return null or div (this can also trigger unmount).

Anyway, I will check it tomorrow ( will update to latest release) and provide you response.

@diasbruno, @ajfuller I have updated react-modal to 2.0.6 but state inside modal still the same. Is it depends on my libs? For example, I use redux state manager and redux-form. And this (redux-form) doesn't change after toggling window. Any idea?

You can check if redux-form don't clear the state after unmount. Since it store the information on the redux state, it can recover the previous information. Probably, you will need to dispatch and action to clean the state.

@enheit I'm stuck with the same problem (using redux-form too). Did you find a solution finally?

@Neurostep unfortunately no :( If I will find solution it will be explained in this post ASAP.

@diasbruno & @ajfuller I've reproduced the same behavior as in my project. The input value still the same after open -> editing -> close. On other hand, this state still the same because (I suppose) the modal doesn't know anything about child element state and don't know how to clean it. So, any idea? Should I look for workaround or there is a better solution?

It only knows when to build the ModalPortal which contains the children, therefore the element is constructed and destructed when toggle the modal isOpen state.
In your example, you are accumulating state using handleInputChange and this is state is not internal to the modal. You'll need to use onRequestClose to flush it.
onRequestCloseCleanup = () => {
  // this is the opportunity to do a proper clean up.
  this.setState({ isOpen: !this.state.isOpen, inputValue: ''  <--- });
}

Examples:
<Modal ...>
  // state is inside the modal, so when modal is closed
  // MyComponentWithState is destructed and so its state.
  <MyComponentWithState />
</Modal>

<Modal ...>
  // state is outside the modal, so it does not manage it.
  <MyComponentReceivesState state={outerState} />
</Modal>

I finally found a solution how to fix this.
So, if you try to use your any elements right inside modal, this will not clear your values at all. But if you move your elements in separate component and just pass this component (which contains your elements) to the react-modal it will clear everything. If you didn't understand, take a look at example below:
// ModalComponent
<Modal ...>
   <form>
     <Field ... /> // It will not update your fields after close
   </form>
</Modal>

// ModalComponent
<Modal ...>
  <SomeForm />
</Modal>

// SomeForm
<form>
  <Field ... /> // It field will be reset after close
</form>

...or you can get more readable solution using this link: https://codesandbox.io/s/kRRp1ZQwY

Ah, got it. Sorry for the bad answer.
In redux-form, the default behavior is to destroy when unmount. redux-form@master:/src/createReduxForm.js#L89.
One probable reason is because when toggle isOpen attribute on Modal, it's never calls to unmount on ModalPortal, therefore unmount on the form is not called. See src/components/Modal.js#L129
There is room for further inspection on this issue.

I'm going to open a new discussion about this issue.
Thank you, @enheit.


Novel4up error : cannot convert null value to object

When I go to novel4up extension especially Apotheosist novel error appears says cannot convert null value to object

Can't seem to find the novel in the source website. They might have removed it.


[Question] How to extract client certificate from the TLS handshake at the istio gateway level using envoy filter

Title: Extract client certificate from TLS handshake using envoy filter
Description:
I am trying to mimic Nginx' behaviour where it extracts the client certificate and saves it as ssl_verify_client. The saved value of certificate can further be injected as a custom header before the request is sent to the target service.
I tried writing  envoy filters following examples elsewhere which did not achieve what I need as I was unable to extract the certificate from TLS handshake.
Could someone here help me with this ?
Thanks

cc @lizan @ggreenway as TLS experts.
I'm assuming this can be added to the TLS extension, but don't know what it entails.

The cert details are stored in the header x-forwarded-client-cert. Is that what you're looking for?


views missing config for CIRCULATION_LOAN_LINKS_FACTORY

Here https://github.com/inveniosoftware/invenio-circulation/blob/master/invenio_circulation/views.py#L90 the loan links factory is retrieved from the config CIRCULATION_LOAN_LINKS_FACTORY  but is never set to a default. Also should be changed to be able to be overridden  by an external module like invenio-app-ils.

The bug has been fixed in https://github.com/inveniosoftware/invenio-circulation/commit/8349241adedcc9e1e0b926fa8b355d62b38d73a4

#84 completes this issue by making loan_links_factory extendable from external module.


DSM 7 (Intel Xeon D-1527 broadwell) Docker stops if run with VNC enabled

Hi, I have tried several possible configurations, but if I enable VNC, the docker stops after few seconds.
These are the logs I get. I at 2022-05-05:14:02 it restarts
2022-05-05 12:14:02 | stdout |
-- | -- | --
2022-05-05 12:14:02 | stdout | Available platform plugins are: eglfs, linuxfb, minimal, minimalegl, offscreen, vnc, xcb.
2022-05-05 12:14:02 | stdout |
2022-05-05 12:14:02 | stdout | This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.
2022-05-05 12:14:02 | stdout | qt.qpa.plugin: Could not load the Qt platform plugin "xcb" in "" even though it was found.
2022-05-05 12:14:02 | stdout | qt.qpa.xcb: could not connect to display :5
2022-05-05 12:14:01 | stdout | [deconzcommunity/deconz] NOVNC port: 6085
2022-05-05 12:14:01 | stdout | - Backgrounding (daemon)
2022-05-05 12:14:01 | stdout | - SSL/TLS support
2022-05-05 12:14:01 | stdout | - Web server. Web root: /usr/share/novnc
2022-05-05 12:14:01 | stdout | - Listen on :6085
2022-05-05 12:14:01 | stdout | WebSocket server settings:
2022-05-05 12:13:59 | stdout | -----
2022-05-05 12:13:59 | stdout | writing new private key to '/opt/deCONZ/vnc/novnc.pem'
2022-05-05 12:13:59 | stdout | ........................+++++
2022-05-05 12:13:59 | stdout | ...............+++++
2022-05-05 12:13:59 | stdout | Generating a RSA private key
2022-05-05 12:13:59 | stdout | tigervncserver: Could not acquire fully qualified host name of this machine.
2022-05-05 12:13:59 | stdout | hostname: Name or service not known
2022-05-05 12:13:59 | stdout | tigervncserver: Could not acquire fully qualified host name of this machine.
2022-05-05 12:13:59 | stdout | hostname: Name or service not known
2022-05-05 12:13:59 | stdout | tigervncserver: Could not acquire fully qualified host name of this machine.
2022-05-05 12:13:59 | stdout | hostname: Name or service not known
2022-05-05 12:13:58 | stdout | [deconzcommunity/deconz] VNC port: 5905
2022-05-05 12:13:58 | stdout | [deconzcommunity/deconz] Checking device group ID
2022-05-05 12:13:57 | stdout | [deconzcommunity/deconz] Modifying user and group ID
2022-05-05 12:13:57 | stdout | [deconzcommunity/deconz] Using options --auto-connect=1         --appdata=/opt/deCONZ         --dbg-info=1         --dbg-aps=0         --dbg-zcl=0         --dbg-ddf=0         --dbg-dev=0         --dbg-zdp=0         --dbg-ota=0  --dbg-error=0         --http-port=85         --ws-port=448
2022-05-05 12:13:57 | stdout | [deconzcommunity/deconz] Websockets port: 448
2022-05-05 12:13:57 | stdout | [deconzcommunity/deconz] Web UI port: 85
2022-05-05 12:13:57 | stdout | [deconzcommunity/deconz] Current deCONZ version: 2.15.03
2022-05-05 12:13:57 | stdout | [deconzcommunity/deconz] Starting deCONZ...

Any idea on how to solve this.

hostname: Name or service not known
Try to start your container without setting a hostname from outside.

Hi, I don't know how to do that. I use the Docker app from DSM 7.0 in synology. I use the host network setting. With this setting I share the name and network from the DSM.

I run bash inside the docker and when I type hostname is returens deconz, but /etc/hosts the only names defined are localhost and nas (which is the network name of my nas). I don't know how the hostname was set to deconz or how to change it in the Docker app in DMS7 in Synology. Any idea?

I have downloaded the docker code to update start.sh by adding in the second line of the script the line
echo "<IP_ADDRESS> deconz ">> /etc/hosts
which I thik it would solve the issue, but I get an error.
I use the comand
 sudo docker build --build-arg VERSION="2.15.03" --build-arg CHANNEL="stable" -t "deconz:local" ./docker/
After a long while of doing things I get this error:
+ DECONZ_VERSION=2.15.03
+ CHANNEL=stable
+ PLATFORM=
+ echo ''
+ grep -qE arm64
+ echo ''
+ grep -qE amd64
+ echo ''
+ grep -qE v7
+ curl -vv '' -o /deconz.deb
* Closing connection -1
curl: (3) URL using bad/illegal format or missing URL
The command '/bin/sh -c chmod +x /download-deconz.sh && /download-deconz.sh ${VERSION} ${CHANNEL} ${TARGETPLATFORM}' returned a non-zero code: 3
I have to say that the instructions on how to build are wrong as I had to replace the ./Docker/ in the end with ./docker/
Any ideas?

Actually I also don't know how the hosts file is filled - but yes it gets the name of your container. And if the container is started

co-running on your host it will take that hostname
with --hostname option the hostname will get this

As I mentioned not to set the hostname - this was my guess of things going wrong. In your case you could have use the option the set the hostname corresponding to the entry in /etc/hosts ...
I really think the start.sh should get some line to sort out the needed entry in /etc/hosts!
Im am running on DSM7 too - but in my case I use a mac-vlan (or likely) so the the container uses a different ip. Imho this is needed on synology NAS as this hosts is using some needed ports by itself.
As you found out on your own: your build call lacked of the platform information.

I have been using it for some days and it works perfectly.
It is amazing how much lighter than a VM the docker is.
It is an amazing solution. Thanks for doing it.

Hi guys, I've exactly the same problem but I'm not so familiar with Linux, so I'm not getting how you solved it.
Someone could help me ?
I'm running DSM 7 (last update availalable)
Deconzcommunity/deconz:latest docker image
Deconz work fine w/o VNC enabled

I recreated the image in Docker with same name of my synology device and then renamed in Docker.
In this way I was able to start VNC service....it's a bug in my opinion or something to improve at least.
thx to all


Researching Build Breakage

Dear developers of the naver/pinpoint project,
We are MSc students from the Delft University of Technology doing research on why builds break in open source projects. To do this, we are analyzing Travis build statistics and build history of pinpoint.
By analyzing the build history of pinpoint, we found the following:

The project has 16% failing builds in the history
The project has a low amount of build failures compared to other similar projects

When looking at pinpoint’s build metrics, we could not find clear factors that are highly correlated with this result.
To identify the reasons behind build breakage, we would like to collect developer insights into the reasons why build breakage occurs. If you have five minutes, please answer the survey we have created. Your response will be very useful in our study.
Thank you.

Ok I serveyed. Good luck.


Indication of the file size (before download!)

Only after the download is complete does the user know how much storage space the video file requires.
For better planning, it would be an advantage if the file size were indicated in addition to the quality level.
By the way, and just for the sake of interest, does Zapp check before downloading whether sufficient memory is available?
Compare also #108.

Sorry. Zapp uses the mediathekview api at https://mediathekviewweb.de/api/
It returns only one file size but one to three video files. Maybe this issue would be fixed in mediathekview v2 - but I doubt it will be released any time soon. I don't know if the mediathekview scraper can provide this information.

MediathekViewWeb actually has this data. But at the moment it's only available via websockets (it is lazy loaded and only fetched from the channels on demand). I could provide a HTTP api for it.
The list from the scrapers don't provide this information.

@bagbag I don't know if its worth it when other Zapp features & fixes are more pressing. But I'll leave this issue open for future reference.

@bagbag I don't know if its worth it when other Zapp features & fixes are more pressing. But I'll leave this issue open for future reference.

I agree.


Add arm64 etcd

Description
Add the etcd-arm64.yaml files for supporting arm64 platform.
The etcd on arm64 platform is different from x86 platform. So etcd-arm64.yaml is added on each versions.
The difference between the arm64's version and x86's version are listed as follows:
-          image: quay.io/coreos/etcd:v3.3.9
+          image: quay.io/coreos/etcd:v3.3.9-arm64

+            - name: ETCD_UNSUPPORTED_ARCH
+              value: "arm64"

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

@fasaxc Could you please review this code for me ?

Change to another way for adding this file


resume waiting for the container to complete

This backports the resuming of tailing logs introduced in https://github.com/drone/drone-exec/pull/26.

 You should sign our Contributor License Agreement in order to get your pull request merged.

@macb while working on the agent I ended up altering the Wait function as follows:
func Wait(client dockerclient.Client, name string) (*dockerclient.ContainerInfo, error) {
	<-client.Wait(name)
	return client.InspectContainer(info.Id)

When I originally wrote this Wait function the Docker client didn't expose the client.Wait endpoint so I had to simulate by reading the logs and writing to ioutil.Discard. So this might simplify things even further!

That would definitely be simpler than the discard logic and might even be more reliable. I can switch this around to use that and maintain the retry logic if that seems reasonable?

Switched it around to use the Wait method. Left in the retries since it seemed possible to hit an intermittent failure due to a network hiccup https://github.com/samalba/dockerclient/blob/master/dockerclient.go#L453-L469.

looks good


mint-dft error claiming 0<mint_amount≤100,000,000 but mint_amount=100,000,000 doesn't work

I've deployed a token and I've set the amount per mint to 100,000,000. It was successfully deployed and verified. My issue is that now that I want to mint it, I get this message :

Ticker : void
Atomical ID : 1409f02645e15b815655396f416b789658062a673af068cc04ad90e45bd0ba02i0

Anyone have an idea on this issue ? @atomicals

Hello, anyone could check this ticket ?


contact janodvarko and add to the adopters list

https://github.com/ahmadnassri/har-spec seems to be the new way forward?


Scrolling Object Collection - Adding new example scene

Overview
Adding a draft example scene for Scrolling Object Collection.
Adding @cre8ivepark to the branch for further polish
App : HL2 Build

This is much improved compared to the previous experimental state of scroll object collections! Couple of feedbacks after trying the appx:

The overscroll is not have any damping applied to it. Try scrolling on the shell pins panel you'll see that the farther you move the more resistance the scroll has.

Scroll via touch: When the finger leaves the touchable area I hear a touch up sound even though I'm still engaged / touching the scroll view. When I enter the scroll area again, I hear a sound.

Far scroll - when the button gets unpressed due to a scroll, I hear a button up sound. I would not expect to hear a button up sound if a button gets unpressed because I am scrolling, only when it's unpressed due to a click.

Touch scrolling -- the scroll amount is not 1:1. When I move my finger down 5 cm, the resulting scroll content moves quite a bit more than 5 cm, as if the scroll is amplified. Try scrolling on your phone -- notice that when you touch the phone and scroll, if you move your finger 1 cm, content moves 1 cm. If you try this on the start menu in shell you'll see the same thing. I expect the scroll amount to be 1:1

minor: the lunar lander in "scroll view no pagination" is upside down

The overscroll is not have any damping applied to it. Try scrolling on the shell pins panel you'll see that the farther you move the more resistance the scroll has.

Scroll via touch: When the finger leaves the touchable area I hear a touch up sound even though I'm still engaged / touching the scroll view. When I enter the scroll area again, I hear a sound.
Far scroll - when the button gets unpressed due to a scroll, I hear a button up sound. I would not expect to hear a button up sound if a button gets unpressed because I am scrolling, only when it's unpressed due to a click.
Touch scrolling -- the scroll amount is not 1:1. When I move my finger down 5 cm, the resulting scroll content moves quite a bit more than 5 cm, as if the scroll is amplified. Try scrolling on your phone -- notice that when you touch the phone and scroll, if you move your finger 1 cm, content moves 1 cm. If you try this on the start menu in shell you'll see the same thing. I expect the scroll amount to be 1:1
Will address all these for stabilization.

Pushed some design polish items:

Updated with new description panel
Updated scroll view examples in the center and right: color, layout, card design polish.

/azp run


TrayIcon/NativeMenuItem Header as DataBinding  does not update on binded data change

Describe the bug
I have a NativeMenuItem with binding as so
when i change the value of swearFilterHeader, programmatically, it does not update the header in the menu,
To Reproduce
Steps to reproduce the behavior:
i have the following TrayIcon;
  <TrayIcon.Icons>
    <TrayIcons>
      <TrayIcon Icon="/icon.ico" ToolTipText="HI1 Server">
        <TrayIcon.Menu>
          <NativeMenu>
			<NativeMenuItem Header="Chat">
			  <NativeMenu>
				<NativeMenuItem Header="{Binding swearFilterHeader}" Command="{Binding toggleSwearFilter}"/>
			  </NativeMenu>
			</NativeMenuItem>
          </NativeMenu>
        </TrayIcon.Menu>
      </TrayIcon>
    </TrayIcons>
  </TrayIcon.Icons>

and the following code for it;

public HispViewModel()
{
	swearFilterHeader = (ConfigReader.BadWords ? "Disable" : "Enable") + " Swear Filter";

	toggleSwearFilter = MiniCommand.Create(() =>
	{
		bool enab = !ConfigReader.BadWords;
		ServerStarter.ModifyConfig("enable_word_filter", enab.ToString().ToLowerInvariant());
		ConfigReader.BadWords = enab;
		swearFilterHeader = (ConfigReader.BadWords ? "Disable" : "Enable") + " Swear Filter";
	});
}

public String swearFilterHeader { get; set; }

however only the first swearFilterHeader = (ConfigReader.BadWords ? "Disable" : "Enable") + " Swear Filter";
when its first initialized actually works, after executing the toggleSwearFilter command it should change it to say "Disable swear filter" or "Enable swear filter" respectively in the tray icons menus.
however instead, the actual menu item stays on whatever it was before .
Expected behavior
The text actually change when i change the value of what its bound too.
Desktop (please complete the following information):

OS: Windows 11
Version 22H2 (OS BUILD 22598.200)

What Avalonia version are you using?
I also don't see INotifyPropertyChanged implementation on your view model, swearFilterHeader is just a simple property.

Avalonia, Avalonia.Desktop, Avalonia.Diagnostics 0.10.17
XamlNameReferenceGenerator 1.3.4
as for INotifyPropertyChanged, its implemented inside ViewModelBase its practily just copied right from the sample;
{
    public class ViewModelBase : INotifyPropertyChanged
    {
        public event PropertyChangedEventHandler PropertyChanged;
        protected bool RaiseAndSetIfChanged<T>(ref T field, T value, [CallerMemberName] string propertyName = null)
        {
            if (!EqualityComparer<T>.Default.Equals(field, value))
            {
                field = value;
                RaisePropertyChanged(propertyName);
                return true;
            }
            return false;
        }

        protected void RaisePropertyChanged([CallerMemberName] string propertyName = null)
            => PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
    }```

public String swearFilterHeader { get; set; }

should be
private String _swearFilterHeader;
public String swearFilterHeader
{
    get => swearFilterHeader;
    set => RaiseAndSetIfChanged(ref swearFilterHeader, value);
}

Hey it works now! thanks


Support for OpenJDK on Windows

What you were trying to do
Trying to launch Terasology with OpenJDK 8 on Windows will result on the next message being displayed instead.
Launched directly from the exe, didn't use TerasologyLauncher.

No Java Installation Found
Could not find a suitable Java 8 installation on your computer.

OpenJDK binaries are jdk8u232-b09 got from https://adoptopenjdk.net/?variant=openjdk8&jvmVariant=hotspot
I develop with these, so I'm sure javapath is correctly set.
Game version
Tried TerasologyOmega Alpha 12 and Alpha 13 RC, and none of them work.

Thanks for reporting! I'm not absolutely sure, but the runtime detection is probably done in:
https://github.com/MovingBlocks/TerasologyJavaLauncher/blob/master/terasology.cpp#L43-L50
 if (jreHome.empty()) {
 	// User installation error will be reported by detectJre
 	if (config.jreDirectory().empty()) {
 		Platform::showError("No Java Installation Found",
 			"Could not find a suitable Java 8 installation on your computer.");
 	}
 	return -1;
 }

It migth be looking for JAVA_HOME or JavaHome in your environment.
Are there any other logging outputs when starting the game from the .exe?

As I said, JAVA_HOME and Path are correctly set:

The installer for OpenJDK takes care of it during installation.
And from what I can tell, no log seems to be created.
Where the exe is located, no log can be found, and %AppData%/Terasology does not exist on my system.

This definitely does not behave as it should... maybe we are checking against some Oracle-specific names which are not met by the AdoptOpenJDK distribution 🤔
I'm not sure how the lookup in CRegKey deals with case sensitivity and underscores, but this line defines a string constant to "JavaHome": https://github.com/MovingBlocks/TerasologyJavaLauncher/blob/master/jredetector_win.cpp#L14
I'll try to reproduce the issue locally... Maybe @jdrueckert could help, I think she also uses AdoptOpenJDK...

I can confirm (not with AdoptOpenJDK, though) that the exe is not working 😕 I used the Powershell to run it, not more luck with logging, either.
Starting the game from Powershell via java -jar ./libs/Terasology.jar works fine...

I can confirm that it works launching it directly, the exe is the thing that fails.
Running through PowerShell made me aware that it saves data to the saved games directory, so now I have a log to show:
Terasology-init.log

Brief update on the way forward:

Whoever feels tempted to try their hands on fixing the .exes have a look at this repository: https://github.com/MovingBlocks/TerasologyJavaLauncher

Meanwhile, we want to bundle the JRE with the launcher and use that to run the game: https://github.com/MovingBlocks/TerasologyLauncher/issues/259
We should still provide a plain .bat file to run the game easily from command line without the launcher.

I think that I can add a bit of information here regarding why I think that this is happening.
The Java JDK, which is the default option for "AdoptOpenJDK" downloads, defines a different Windows registry path to the JRE.
The Terasology launcher used on Windows searches for the JRE by querying the registry key:
HKEY_LOCAL_MACHINE\SOFTWARE\JavaSoft\Java Runtime Environment\1.8\JavaHome
(as found in https://github.com/MovingBlocks/TerasologyJavaLauncher/blob/ed21d8ac0969af364ded8a8ab2fe53795b886789/jredetector_win.cpp#L13).
The JDK path is queried using a different, albeit similar, registry path, which is:
HKEY_LOCAL_MACHINE\SOFTWARE\JavaSoft\Java Development Kit\1.8\JavaHome
The launcher does not check for Java JDK installations, only for JRE installations. It will not run on systems where the JRE registry key is not defined, unless there is a pre-defined bundled jre dir present (located in the <installDir>/jre folder). Changing the searched registry path from the JRE key to the JDK key in the source code fixed the issue for me.
I have not installed the "AdoptOpenJDK" distribution to test this, however it would appear that their installer also writes to the registry keys mentioned above:
https://github.com/AdoptOpenJDK/openjdk-installer/blob/master/wix/Main.openj9.wxs#L134
So far as I can tell, the JAVA_HOME environment variable is not checked at all, although I have not searched the code in-depth enough to be certain of this.

I have issued a pull request for the changes I mentioned before, located at https://github.com/MovingBlocks/TerasologyJavaLauncher/pull/1. The changes made should fix this issue, although it will need further testing to be certain.

why has it not been merged yet?

@HybridShivam good question! Most likely because nobody got the time review and test it 🙈
We've spent some effort on improving the launcher for the regular players by including a JRE with it. Version 4.0.0 just got released.


getFieldState return undefined

Help us to manage our issues by answering the following:

How would you tag this issue?

Bug

Describe your issue:

$("onFormInit").subscribe(() => {
  const state = await actions.getFieldState("foo");
  console.log(state);
});

$("onFieldChange", "bar").subscribe(() => {
  const state = await actions.getFieldState("foo");
  console.log(state);
});

What is the current behavior?
return undefined after formInit or fieldChange

What is the expected behavior?
return the correct field state.

Which versions of UForm?
v1.0.0-beta.6

Did this work in previous versions of UForm?
No. Fine in v0.4.4

If your issue is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via Bug Report Template on CodeSandbox
.
https://codesandbox.io/s/snowy-sun-pvrr1?fontsize=14&hidenavigation=1&theme=dark

目前这个部分属于break change。不好解决，因为底层架构做了重写，这种场景，推荐在onFieldInit里处理


Control.TextInput ref returns null

Small issue I've recently come up against when trying to trigger an input to focus is that when I use Control.TextInput for a field in React Native, ref will only return null. Is there an alternative way of doing it or is this a bug?
<Control.TextInput ref={input => console.log(input)} />

Hey @bgriffith, that might be a bug - I'll investigate and fix it. In any case, the implementation is small for binding to React Native components:
Control.TextInput = (props) => (
  <Control
    component={TextInput}
    mapProps={{
      onResponderGrant: ({ onFocus }) => onFocus,
      value: (_props) => ((!_props.defaultValue && !_props.hasOwnProperty('value'))
        ? getTextValue(_props.viewValue)
        : _props.value),
      onChangeText: ({ onChange }) => onChange,
      onChange: noop,
      onBlur: ({ onBlur, viewValue }) => () => onBlur(viewValue),
      onFocus: ({ onFocus }) => onFocus,
    }}
    {...props}
  />
);

so if you'd like to figure out what's missing, perhaps you can make a quick PR and I can add it in. Also, I believe that for now, this will work:
<Control.TextInput controlProps={{ref: input => console.log(input)}} />

but ideally it should work the way you have it.

Hey @davidkpiano Thanks for the tips. Weird things be happening though! If I add the controlProps prop to the component (doesn't matter what I include within it) the whole input disappears from view!

Since React doesn't support propagating the ref prop through wrapped components (for whatever reason...), I think the solution is for me to add getRef={(node) => ...} to the API.
Does the name getRef sound intuitive enough? Or should I name it something different, like controlRef? It just can't be named ref.

getRef sounds intuitive enough to me :+1:

Okay, done! Published as 1.3.2.

Awesome thanks a lot! When you a min could you publish to npm?

Done! Published as 1.3.2


fetch_dependencies may crash when fetching stuff from github

fetch_dependencies.sh should delete all the git stuff before doing a checkout, otherwise it can happen that the process crashes in the middle.

💯


Google Cloud AutoMl V1 CreateDataset Error "OperationMetadata hasn't been added to descriptor pool"

When trying to run this code
use Google\Cloud\AutoMl\V1\AutoMlClient;
use Google\Cloud\AutoMl\V1\Dataset;
use Google\Cloud\AutoMl\V1\TranslationDatasetMetadata;

$config = $this->config->get('automl');
        $autoMlClient = new AutoMlClient(['credentials' => $config['service_account_key']]);
        $formattedParent = $autoMlClient->locationName($config['project_id'], $config['location_id']);
        $dataset = new Dataset([
            'display_name'                 => $datasetName,
            'translation_dataset_metadata' => new TranslationDatasetMetadata([
                'source_language_code' => $sourceLanguage,
                'target_language_code' => $targetLanguage,
            ]),
        ]);

    $operationResponse = $autoMlClient->createDataset($formattedParent, $dataset);

I am getting this error:
"Error occurred during parsing: Class google.cloud.automl.v1.OperationMetadata hasn't been added to descriptor pool"
The code is creating the dataset but this error prevents me to check if/when the operation is completed
BTW I checked the same code with 'V1beta1' and it works
Can someone tell me what is going on?

hi @zarkoET , I tried the following code with latest code in main branch ref and it prints dataset created. Is this not what you intend to achieve?
$PROJECT = 'thXXXXXXXX319';
$REGION = 'us-central1';

require 'vendor/autoload.php';

use Google\Cloud\AutoMl\V1\AutoMlClient;
use Google\Cloud\AutoMl\V1\Dataset;
use Google\Cloud\AutoMl\V1\TranslationDatasetMetadata;

$autoMlClient = new AutoMlClient([
    'credentials' => '/Users/vishwarajanand/XXXXXXX/GOOGLE_APPLICATIONS_CREDENTIALS.json',
]);
$formattedParent = $autoMlClient->locationName($PROJECT, $REGION);
$dataset = new Dataset([
    'display_name' => 'google_cloud_php_issue_5453',
    'translation_dataset_metadata' => new TranslationDatasetMetadata([
        'source_language_code' => 'en',
        'target_language_code' => 'es',
    ]),
]);
$operationResponse = $autoMlClient->createDataset($formattedParent, $dataset);
$operationResponse->pollUntilComplete();
if ($operationResponse->operationSucceeded()) {
    $result = $operationResponse->getResult();
    print("dataset created" . PHP_EOL);
} else {
    $error = $operationResponse->getError();
    print("dataset creation failed" . PHP_EOL);
}

Closing this issue, because we need more info to proceed on the debugging. Pls feel free to reopen if needed.


Polish up the design for existing components

A designer should go over the existing components and make sure that the components align with EDS.
TODO: Add screenshots of existing components

Settings screen

Feedback screen

Release note screen

Onboarding screen

Login screen

About screen

https://www.figma.com/file/GmYi1qTwOkAM4ZyK5Bsj31/Component-library?node-id=1%3A2&t=sWUu13JJYa9WhZ7L-1

This package is getting phased out, and we do not plan to make any improvements unless they are critical for the remaining apps. Closing this issue


error trap constant negative indexes and add run-time check for dynamic negative indexes

In sizeColonOperator, this inserts an error trap for negative constant indexes and, if indexes are not constant adds a check in run-time code for negative indexes.
This fixes issue #1134

Ok, I didn't think this through. Since the size processing handles node functions as well, this is injecting run-time checks in a lot of our core generated code. And even worse, there are times that we generate code where we expect negative indexing, such as 1:0 is usually the case for
for (i in seq_along(initFunctionList))

in my_initializeModel$run. Or in our overall MCMC, we have 1:niter and don't want to check that probably.
It's not clear how to know within sizeColonOperator when we want the check and when we don't...
I'm going to drop the dynamic check, but leave the setup code check for constant negative indexes.


ISSN Extractor

As a part of my colleague's work I have added new ISSN (http://www.issn.org/) extractor to get even more citations from Wikipedia articles.
PR also contains some minor code typos and cleanup - works like a charm passing all tests (including new ISSN extractor).

Sure, I have used the same pattern like in ISBN extractor.

Thanks for the pull request!


clang 14 support

What does this PR do?
Clang 14 compilation
Motivation
Latest toolchain on ubuntu 22.04

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

How much is clang-14 a priority for you ? Is CWS the only remaining blocker ?

How much is clang-14 a priority for you ? Is CWS the only remaining blocker ?

I upgrade my laptop to 22.04 and this issue came, so here the fix, it's not blocking or urgent
The previous patch (alternative fix) was to reduce the stack size : https://github.com/DataDog/datadog-agent/pull/13210/commits/1b9ab00781e34ce2b0f4bee5cb91f6426a360146#diff-54f316ac5735fec92aaa1384ac40f89f76d4e2e11fc9a35685b6a3e89c09f74bR20
There are only this issue with CWS

latest version here https://github.com/DataDog/datadog-agent/pull/13255


Update to match 3.5 and add missing modules

Reorganized in alphabetical order and added some missing modules (except those with editor-only).
Not sure if WebP should be considered critical to work.
If merged, I would like #6 to be considered superseded.

Not sure if WebP should be considered critical to work.

 As a result, I'd remove it from the list of modules you can disable.

Done. I also excluded etc since it is editor-only.


Back button style enhancements NEEDS #5361

Small enhancements to back button. Requires https://github.com/silverstripe/silverstripe-framework/pull/5361

Was

New

Closing in favour of https://github.com/silverstripe/silverstripe-framework/pull/5378


Create actual browser bundles and transpile everything else

Required reading: #779
This PR significantly changes up our bundling situation:

Rollup now produces two actual bundles: dist/browser.esm.js and dist/browser.umd.js. These are actual browser bundles — all dependencies except for our peer dependencies (react and styled-components) are included in the bundle. These are really only useful for dropping onto a page with the appropriate ESM/UMD setup (e.g. the user is also including the peer deps). We may decide we don't even need these, but they may be useful for tracking deltas in size over time. Note: these files are quite large (500KB, even after minification), but include all the dependencies.
Everything else gets transpiled directly from src/ into dist/. This will allow users to use both import {Box, BorderBox} from '@primer/components' and also import Box from '@primer/components/dist/Box'. This also enables import ShinyNewThing from '@primer/components/dist/experimental/ShinyNewThing'

Todo:

[x] If this is landing in 19.0.0, update the docs in #789 to include the new instructions.

Release Notes

[x] You can now import individual components without having to include the entire package or set up tree-shaking; for example, import {Box} from '@primer/components' can now also be written as import Box from '@primer/components/dist/Box'

Closes #779
Closes #456

Since this PR will make every component importable directly from its transpiled source file, each components exports shape will be part of the public API of the package. As such, I'm proposing making that contract explicit in our tests:
checkExports('Position', {
  default: null,
  Position,
  Absolute,
  Fixed,
  Relative,
  Sticky
})

checkExports('Box', {
  default: Box
})

checkExports will ensure that the file at the given path exports the things we expect; if there are keys in the expected exports that don't match, or there are things exported that we don't expect to be exported, the test will fail.
Before I go through the entire codebase adding this, I wanted to get some feedback from the team to see how people felt about such a thing.

As I was looking through the exports, I noticed two modules where importing directly from the dist folder was a little weird: Button* and the Position variant. I've modified them as such:

Button and variants can now be imported as:
import Button, {
  ButtonDanger,
  ButtonPrimary,
  ButtonOutline,
  ButtonGroup,
  ButtonTableList
} from '@primer/components/dist/Button'

Position and variants can now be imported as:
import Position, {
  Absolute,
  Fixed,
  Sticky,
  Relative
} from '@primer/components/dist/Position'

Open to other thoughts on organization, and also curious the best way to go about documenting individual exports when the component isn't directly available on the default export.

I started doing some visualizations of our bundle size and found something interesting: react-dom was in there — both development and production versions — taking up nearly 1MB of space. This is because @reach/dialog depends on it as a peerDep and we didn't list it as an external dependency. I've fixed this, and our minified, uncompressed bundle size is down to 335KB.

With some more changes it's sitting at ~170KB.

I like the checkExports idea!

Is there a real benefit to shipping bundles in the first place? How many consumers want a UMD module bundle? If the answer is "not really" could we just get rid of rollup completely and force consumers to use a bundler (because let's be honest, if you're writing frontend JS you're using a bundler).

Is there a real benefit to shipping bundles in the first place?

I don't think so, and in fact it's not documented. I think it's potentially useful for us internally for seeing how our package would behave if it was bundled; a good example is cutting off ~800KB off the minified library size based off the work in this PR.


Documentation

Some form of Documentation would be useful to help onboarding new contributors. Maybe Pdoc deployed to Github Pages?

Inline Documentation with Pdoc
Update README.md

Thanks to @Lachstec initial version of the documentation is now avaiable as github page and now also referenced as about at the top of the repo

The docs have been updated including information about the configuration and current limitations. I think this issue can be closed for now, as some major changes are planned, thus more documentation would be a bit wasted.

@Lachstec absolutely! Thanks again for your work! I wanted to take a look into generating the static pages of the documentation directly in the GitHub action, but this can also be done later. I'll close the issue.


Euro Truck Simulator 2: 3D rendering issue

Broken rendering when D9VK is used (even when graphic setting is low). See screenshots.
wined3d:

D9VK 5866917:

Software information
Euro Truck Simulator 2 (Demo version available from eurotrucksimulator2.com)
System information

GPU: Intel HD Graphics 510
Driver: Mesa 19.1.0-rc2
Wine version: 4.0
D9VK version: 5866917

Apitrace file(s)

eurotrucks2.trace.xz (233M, 1280x720 Windowed)

Log files

eurotrucks2_d3d9.log.xz.zip

I'm unable to test this as the demo constantly crashes back to desktop for me. I've checked winehq and the game has a platinum rating, so the demo is probably very outdated or something.

@tsalwierz What wine version are you using? I tested with EuroTruckSimulator2_1_34_0_34_setup.exe (Press "Run in Demo mode") and it works fine. Does it crash only when D9VK is used?

@kakurasan I was using the steam demo. I've since recreated my wine prefix so I'll give this another shot later on today.

This was recently fixed (It works fine as of 74f8922).

Thank you Joshua! 🐸


GitHub Action for testing Python scripts

This is a GitHub Action for the python test suite present in this repository.
Signed-off-by: Petr "Stone" Hracek<EMAIL_ADDRESS>

[test]


Hovering of Submit Button in signup from

Current Behavior
Submit button Not Hovered like login button or signup button hover
Desired Behavior
Want to Hover that Submit Button which will look good
Before changes
https://user-images.githubusercontent.com/96141990/220317431-838d23c0-2c51-4cba-9e6c-f6c0fd5813d4.mp4
After Changes
https://user-images.githubusercontent.com/96141990/220317536-70f8d273-d519-49d1-a4f7-f2f6a638a157.mp4

@leecalcote i would like to work on this issue. Can you please assign me this?

As I have Already Done it on PC

@123Vasu, I'm just seeing this issue... after I just reviewed @Shivam164's PR. Thank you for raising this issue, @123Vasu.


:bug: Rename bundle class name to follow Symfony best practice

This is a backward compatibility breaking change that should not have been released in a point release and thus breaks a build when composer update is done (until a manual change is made to bundles.php)

@PhilETaylor I don't know what is your version constraint in your composer.json, but by default it's ^0.3 and running composer update should not update the package to 0.4! I tested it on one of my project, and the package isn't update.
Moreover, i will not cover BC break until the package as reach a stable version (1.0)


Resume training progress and see images created every tick

I am training my custom dataset with run_low_shot.py on Google Colab. How can I resume my training progress and see the image created every tick. I saw the training_loop.py scripts but I don't know how to implement it

Use --resume=CHECKPOINT_PATH to resume; you can directly modify this line to 1.

what exactly is the CHECKPOINT_PATH? I saw 2 new file in my dataset, one *.tfrecords file and one *.pkl file. And about the training_loop.py script, do I just call it directly with arguments or is it called by another script?

*.tfrecords is the dataset file; *.pkl is the checkpoint file. You do not need to call training_loop.py, just modify it and then run run_low_shot.py.

*.tfrecords is the dataset file; *.pkl is the checkpoint file. You do not need to call training_loop.py, just modify it and then run run_low_shot.py.

Thanks, I get it now

I ran the run_low_shot.py with --resume="the/path/to/my/datasets/*.pkl"
ERROR REPORT
Traceback (most recent call last):
File "run_low_shot.py", line 171, in
main()
File "run_low_shot.py", line 165, in main
run(**vars(args))
File "run_low_shot.py", line 94, in run
dnnlib.submit_run(**kwargs)
File "/content/drive/My Drive/graduationthesis/data-efficient-gans/DiffAugment-stylegan2/dnnlib/submission/submit.py", line 343, in submit_run
return farm.submit(submit_config, host_run_dir)
File "/content/drive/My Drive/graduationthesis/data-efficient-gans/DiffAugment-stylegan2/dnnlib/submission/internal/local.py", line 22, in submit
return run_wrapper(submit_config)
File "/content/drive/My Drive/graduationthesis/data-efficient-gans/DiffAugment-stylegan2/dnnlib/submission/submit.py", line 280, in run_wrapper
run_func_obj(**submit_config.run_func_kwargs)
File "/content/drive/My Drive/graduationthesis/data-efficient-gans/DiffAugment-stylegan2/training/training_loop.py", line 160, in training_loop
rG, rD, rGs = resume_networks
ValueError: not enough values to unpack (expected 3, got 2)

Oh, *.pkl should not be in your dataset folder but in the experiment folder (usually inside a folder called training).

My training folder does't have any *.pkl files, it contains only __pycache__ folder and scripts of yours. I took a look at .stylegan2-cache folder and I can't find the *.pkl for my dataset

Sorry I meant the folder called results. If there are no *.pkl in the results folder, that means your training did not reach the progress of saving a checkpoint.

Great! Thank you so much, really appreciated


Unable/unclear to use keyword 'extent' with imshow

When setting the 'extent' keyword in the imshow function I receive the following error:
due to unexpected exception with message:
Call to imshow() failed
How I call the imshow function:
PyObject* mat;
plt::imshow(values, height, width, 1, {{"interpolation","bilinear"},{"origin", "lower"},{"aspect", "auto"}, {"extent","{0, 100, 0, 40}"}, &mat);
How could I provide imshow() with the extent keyword?

That's probably because "{0, 100, 0, 40}" is passed as string to matplotlib but it expects an array or list. That's another case of #184: currently it is only possible to pass string kwargs.

I suggested a solution for this in #241


RML doesn't work

v1:
import * as alt from 'alt-client';

const document = new alt.RmlDocument("/client/index.rml");

alt.log("Container", !!document.getElementByID("container"));

Container true

v2:
import * as alt from '@altv/client';

const document = alt.RmlDocument.create({ url: "/client/index.rml" });

alt.log("Container before delay", !!document.getElementByID("container"));

alt.Timers.nextTick(() => {
	alt.log("Container after delay", !!document.getElementByID("container"));
});

alt.Timers.setTimeout(() => {
	alt.log("Container after 1s delay", !!document.getElementByID("container"));
}, 1000);

Container before delay false
Loaded resource test-resource
Container after delay false
Container after 1s delay false

document.createElement also returns null.
index.rml:
<rml>
  <head>
    <title>RMLUI</title>
  </head>
  <body id="container"></body>
</rml>

document itself is valid object

but it's innerRML returns empty string (should be content of index.rml).

This has to be fixed by alt:V - see https://github.com/altmp/altv-issues/issues/2209

I temporarily added a workaround for ScriptObjects not being created for RmlElements, however, the real fix has to come from alt:V
Once alt:V fixes this, revert the following Pull Request: https://github.com/altmp/altv-js-module-v2/pull/167

On latest dev Version, OnCreateBaseObject method receives nullptr when creating an RmlDocument. (Basically logging the type inside the methods):
Crashdump Id: 631f276d-858b-4d88-a4e2-3cf88a2a2087


rhel-7-server-extras-rpms repo failing pharos cluster installation

I have tried to install Kontena Pharos Cluster (pharos-cluster 2.4.0+oss) but the installation (pharos up) failed with the following error:

Error: exec failed with code 1: configure-docker.sh
sudo env -i - http_proxy= https_proxy= no_proxy= HTTP_PROXY= HTTPS_PROXY= NO_PROXY= FTP_PROXY= PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/sbin:/usr/sbin:/sbin DOCKER_VERSION=1.13.1 DOCKER_REPO_NAME=rhel-7-server-extras-rpms 'INSECURE_REGISTRIES=[]' bash --norc --noprofile -x -s
set -e
. /usr/local/share/pharos/util.sh
configure_container_runtime_proxy docker
daemon=docker
systemd_runtime_dir=/etc/systemd/system/docker.service.d
systemd_proxy_config_file=/etc/systemd/system/docker.service.d/http-proxy.conf
mkdir -p /etc/systemd/system/docker.service.d
'[' -n '' ']'
'[' -n '' ']'
'[' -n '' ']'
'[' -n '' ']'
'[' -n '' ']'
'[' -n '' ']'
'[' -n '' ']'
'[' -f /etc/systemd/system/docker.service.d/http-proxy.conf ']'
unset daemon
unset systemd_proxy_config_file
unset systemd_runtime_dir
'[' -z 1.13.1 ']'
mkdir -p /etc/docker
cat
yum install --enablerepo=rhel-7-server-extras-rpms -y docker-1.13.1
Loaded plugins: langpacks, product-id, rhnplugin, search-disabled-repos,
: subscription-manager, versionlock
This system is receiving updates from RHN Classic or Red Hat Satellite.
Error getting repository data for rhel-7-server-extras-rpms, repository not found

I can see that the fixing of bug #559 with pull request #624 where you have hardcoded the name of the repository is giving the problems.
I am behind a proxy where the repos are set up using Sattelite and we have no say in their naming standards used. They can either follow redhats standards, which are rhel-x86_64-server-extras-7 or our hosting suppliers, which isn't any closer to the hardcoded one.
Proposed fix: Could it be possible for if this could either be configured in the cluster.yml which repo it should use or maybe a check that will see if docker is installed in the right version and if yes, just skip this part?
Because right now it is sadly unusable for us.
to the existing issue instead.

Found out that this can be circumvented by adding any kind of repo with the same name, as long as you have set one yum repo where docker can be installed from. In our case we changed the cluster.yml to setup a repo with the same name as pharos creates on each of our hosts:
repositories:
 - name: kontena-pharos.repo
   contents: |
    [kontena-pharos]
    name=kontena-pharos
    baseurl=https://point-to-artifactory-remote
    enabled=1
    gpgcheck=0

    [rhel-7-server-extras-rpms]
    name=rhel-7-server-extras-rpms
    baseurl=point-to-artifactory
    enabled=1
    gpgcheck=0


MailboxMovePublishedScopes missing description

A critical paremeter used with tenant to tenant mailbox migration and the description is missing.
Please know that Microsoft documents and articles reference this.
Thank you.

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

ID: 61bdc5c8-91af-66ca-cc94-3127fea1b8c3
Version Independent ID: 29ec4bea-aa01-8f74-f820-adb77c57aa4f
Content: New-OrganizationRelationship (ExchangePowerShell)
Content Source: exchange/exchange-ps/exchange/New-OrganizationRelationship.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@Karr, which parameter are you talking about?

@Karr, OK I added descriptions for the MailboxMovePublishedScopes, MailboxMoveCapability, and OAuthApplicationID parameters per the information in this topic.
Thanks for the feedback.
Closing


Remove whitespace between words in extern declarations

These changes aim to fix #1815
This is my first PR here and although the added test cases pass I'm pretty sure my implementation is extremely naive and doesn't use the available tools in rustfmt. Also it allocates (probably) unnecessarily.
Any suggestions?

Hi @brainlessdeveloper , thanks for the PR!
@topecongiro would you be able to review this please?

@brainlessdeveloper Thank you for your PR! It looks fine to me for most cases.
In theory, this PR might break the code when there are comments in between extern crate foo:
extern crate /* a   b   c */ foo;
extern // comment
    crate // comment
    bar;

In these cases, the spaces inside comments will get removed or the code will get broken.
I think we could just use the original snippet when we detect the comments. You could use contains_comment() to see whether the given str contains comment.

I wish that we could use AST node to manullay construct the output, but looks like rust parser does not give enough information to do so.

I'll incorporate your suggestion with contains_comment() when I work on it this evening :) thanks!

Sorry this took so long! I've been extra busy.

@topecongiro r?

@brainlessdeveloper Sorry it took so long. LGTM, thank you for the update!

Thank you for the awesome library!


[BUG] Querying the default swagger route ends with "invalid status code: undefined"

Information

Version: 5.62.3
Packages: tsed

A few sentences describing the overall goals of the issue.
Example
After having started the scaffolded project I tried to open http://localhost:8083/docs which provoked an internal error.

[2020-08-18T14:35:21.223] [ERROR] [TSED] - {
  method: 'GET',
  url: '/docs',
  headers: {
    host: 'localhost:8083',
    connection: 'keep-alive',
    'cache-control': 'max-age=0',
    dnt: '1',
    'upgrade-insecure-requests': '1',
    'user-agent': 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 ' +
      '(KHTML, like Gecko) Chrome/84.0.4147.125 Safari/537.36 ' +
      'Edg/84.0.522.59',
    accept: 'text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9',
    'sec-fetch-site': 'none',
    'sec-fetch-mode': 'navigate',
    'sec-fetch-user': '?1',
    'sec-fetch-dest': 'document',
    'accept-encoding': 'gzip, deflate, br',
    'accept-language': 'en-US,en;q=0.9,de;q=0.8,de-DE;q=0.7,en-GB;q=0.6,fr;q=0.5',
  },
  body: undefined,
  query: {},
  params: {},
  reqId: '44198a456ece461492fb73a884ceca18',
  time: 2020-08-18T12:35:21.223Z,
  duration: 27,
  error: {
    status: 500,
    message: 'Invalid status code: undefined',
    stack: 'RangeError [ERR_HTTP_INVALID_STATUS_CODE]: Invalid status code: ' +
      'undefined\n    at ServerResponse.writeHead (_http_server.js:220:11)\n    ' +
      'at ServerResponse.writeHead ' +
      '(C:\\....\\node_modules\\on-headers\\index.js:44:26)\n' +
      '    at ServerResponse._implicitHeader (_http_server.js:211:8)\n    at ' +
      'ServerResponse.end ' +
      '(C:\\...\\node_modules\\compression\\index.js:103:14)\n' +
      '    at ServerResponse.redirect ' +
      '(C:\\...\\node_modules\\express\\lib\\response.js:951:10)\n' +
      '    at ' +
      'C:\\...\\node_modules\\@tsed\\common\\lib\\mvc\\decorators\\method\\redirect.js:8:14\n' +
      '    at Layer.handle [as handle_request] ' +
      '(C:\\...\\node_modules\\express\\lib\\router\\layer.js:95:5)\n' +
      '    at next ' +
      '(C:\\...\\node_modules\\express\\lib\\router\\route.js:137:13)\n' +
      '    at Route.dispatch ' +
      '(C:\\...\\node_modules\\express\\lib\\router\\route.js:112:3)\n' +
      '    at Layer.handle [as handle_request] ' +
      '(C:\\...\\node_modules\\express\\lib\\router\\layer.js:95:5)',
    origin: undefined
  }
}

I used tsed init . to create this project. Querying for swagger's json file runs fine, no errors there.
I'm running node v12.0.0 and npm v6.9.0 on a Windows 10 x64.

Another interesting thing to report is that when I clone v5.60.2 branch from https://github.com/TypedProject/tsed-getting-started and run it instead then no errors happen. But as soon as I update the packages (I use "npm-check -u" for that) the same error appears.
There must be some important difference between versions 5.60.2 and 5.62.3 (or any other of the packages that got updated too).

@brakmic Thanks for your issue. I recently updated v5 to prepare the v6 version. I try to find the problem and fix the issue :)

Ok Got it. I fix that ;)

Awesome! Many thanks!

release is on going

:tada: This issue has been resolved in version 5.62.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 6.0.0-alpha.6 :tada:
The release is available on:

v6.0.0-alpha.6
GitHub release

Your semantic-release bot :package::rocket:


Setup.py requirements are strict

Setup.py requirements should allow for minor version differences, or otherwise allow for small version changes.

Resolved in #27


Set a sensible value for localStorage.size

It's problematic on self-hosted deployments, especially with the 1Gi default size; let's keep it for people who need it and know what they're doing?
Alternatively, perhaps increase size to 20Gi as well?

what is it used for in our cluster? if we don't have a local storage, what kind of storage is it using instead?

@shcheklein The node storage. Node allows to allocate storage until it will node storage will not be lower that 10% of the total disk space or 10GB(whichever is smaller)

@0x2b3bfa0 I would disable it for studio-worker as well

so, what was the localStorage used for ? why is it needed at all? (can we clarify this in helm comments / docs)?

localStorage is needed for dependency (virtual environment) caching; I'll just bump the default value to 50Gi mirroring our production deployment; it's the easiest.

@shcheklein / @mjasion, better now?

@shcheklein / @mjasion, better now?

there is still no explanation for what it is used for
is it a permanent storage (survives pod restarts / redeployments)?
if we disable it - where does it store those env then? what kind of other storages do we have on pods?

there is still no explanation for what it is used for

https://github.com/iterative/helm-charts/pull/435#issuecomment-2209335042

is it a permanent storage (survives pod restarts / redeployments)?

Not by default; it'll be ephemeral.

if we disable it - where does it store those env then? what kind of other storages do we have on pods?

It would use the default /tmp (with no guarantees on free space availability)

https://github.com/iterative/helm-charts/pull/435#issuecomment-2209335042

I mean the docs / or helm comments. We assume that help is used by customers, right? We should try to put some explanations if possible (not sure if we do this atm, but we were getting negative feedback about this recently).
But even for me, deploying this- It's hard to understand the purpose, if I need this or not, etc.

It would use the default /tmp (with no guarantees on free space availability; not a good idea)

okay, so, space guarantee is the only thing here? (just to make sure)

okay, so, space guarantee is the only thing here? (just to make sure)

Yes, basically it guarantees /tmp/local has enough free space, is writable and so on (i.e. is usable)

I mean the docs / or helm comments. We assume that help is used by customers, right? We should try to put some explanations if possible (not sure if we do this atm, but we were getting negative feedback about this recently).

Right; let me try to fix it.

But even for me, deploying this- It's hard to understand the purpose, if I need this or not, etc.

Long story short, it's needed to initialize & store virtual environments.

Happier?
https://github.com/iterative/helm-charts/blob/8b0df92c79fce8ba5191c8f511ea8b834faaa9cc/charts/studio/values.yaml#L706

Happier?

Well :) nobody outside would understand this ... even internal users - some people will not be able to connect the dots. Anyways, that's fine considering that we don't have much of the docs for the rest of the Helm anyways. It should be a separate issue / epic if we can get to it at some point.
Not a blocker, @0x2b3bfa0 . Thanks for taking care of this.


Web update

Hi everybody.
I'm developing a PWA with flutter web and inside the app I need to print some data with thermal printer.
I wonder wheter this package has complete web compatibility, has it?
Thank you

@bobosette Hi, I am looking at this too. The package platform specifies web, but not sure if anyone has a working example. Have you managed to get something going yet? Thanks!

Hi. After some searching, I decided t9 change the approach to fix my issue.
Il Mar 26 Mar 2024, 11:29 Wesley Barnes @.***> ha
scritto:

@bobosette https://github.com/bobosette Hi, I am looking at this too.
The package platform specifies web, but not sure if anyone has a working
example. Have you managed to get something going yet? Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/andrey-ushakov/esc_pos_utils/issues/101#issuecomment-2020059141,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADEWB4KUEP4ZQ2P2CVBSIQLY2FEXDAVCNFSM6AAAAAAWLO7U42VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRQGA2TSMJUGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@bobosette So you did not use a web to print approach?

Yes but I made an API call.
So the frontend web application call the API printTicket, and from my
backend I handle the request, and I send the text to print to the printer,
that is connected to the network through LAN.
Is the best way to print something from a printer from a web application.
Il giorno mar 26 mar 2024 alle ore 12:36 Wesley Barnes <
@.***> ha scritto:

@bobosette https://github.com/bobosette So you did not use a web to
print approach?
—
Reply to this email directly, view it on GitHub
https://github.com/andrey-ushakov/esc_pos_utils/issues/101#issuecomment-2020189725,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADEWB4I5XG3H2FYSSMQ7C53Y2FMTHAVCNFSM6AAAAAAWLO7U42VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRQGE4DSNZSGU
.
You are receiving this because you were mentioned.Message ID:
@.***>


Broken URL

Not sure if this is the place to create such issue, but the URL to ndc oslo talk is broken: http://ndcoslo.com/talk/state-of-the-net-performance/

I've seen other talk with same title - https://www.youtube.com/watch?v=dVKUYP_YALg
Maybe you could link to this talk from .NET Summit instead ?

@gmanvel  I updated the description of the repo and now it points to the link that you have provided.  thank you!


Where to get "x-cds-client-headers" and how to get we don't have any idea?

How and where to get below headers
x-fapi-interaction-id: string
x-fapi-auth-date: string
x-fapi-customer-ip-address: string
x-cds-client-headers: string
The get account API we need to pass those info and how?

Hi @gobi-it ,
We note that you have raised this via the CDR Support Portal. The following response was provided: The requirements for x-cds-client-headers can be found here: https://consumerdatastandardsaustralia.github.io/standards/index.html#http-headers
Further details are also available here: https://github.com/ConsumerDataStandardsAustralia/standards-maintenance/issues/104
We will close this issue, thank you for your question.
Thanks,


Date shown incorrectly after opening edition mode on a record

Winter CMS Build
1.1
PHP Version
7.4
Database engine
MySQL/MariaDB
Plugins installed
No response
Issue description
I use Winter 1.1.9 and PHP 7.4.32. I have a model called Promotions. The logic behind it is to show promotions in detail within a range of dates. Among the input fields, there are 2 date fields oriented to define said range. Let's call these "From" and "To". After creating a couple of records (or promotions), I go back to one of them to change the range of dates. Before I retouch anything, I realized the dates shown were from one day before.
Nevertheless, the dates display correctly in the list of promotions and in the database. This weird issue only takes place if I want to edit a record (or promotion), that is when I open a record, which takes me to the form to edit it.
Steps to replicate
Let's suppose I set Promo A to be valid from Nov 6th to Nov 25th. Then, I decide to change something from Promo A so I go to its record. I'm taken to the form to edit it and the dates retrieved end up being Nov 5th and Nov 24th.
Workaround
No response

Try setting ignoreTimezone: true on your date fields in your fields.yaml file.

It worked. Thank you so much, @LukeTowers


QR verification and inviting

WeChat has invitation by QR, and millions of Chinese are accustomed to that - and it is a very smooth process.  It could do invitation and verification in a single step.

I would keep it at invite through qr and implement a verification system in the way natrius just recently proposed here:
https://github.com/vector-im/riot-android/issues/3140
I don't think You should combine the two steps.

On a functional level the two should clearly be separate, but the user experience should be as smooth as possible, with as few steps as possible, I think.


Update README.md

add details on how to run Platform.Bible with extensions

This change is

Please help me understand. Are the 2 templates

https://github.com/paranext/paranext-multi-extension-template?tab=readme-ov-file#to-run and
https://github.com/paranext/paranext-extension-template?tab=readme-ov-file#to-run ?
So are those two readmes kept in sync manually? If yes, is there any chance to improve this?

Please help me understand. Are the 2 templates

https://github.com/paranext/paranext-multi-extension-template?tab=readme-ov-file#to-run and
https://github.com/paranext/paranext-extension-template?tab=readme-ov-file#to-run ?
So are those two readmes kept in sync manually? If yes, is there any chance to improve this?

Yep, those are the two readmes! They are manually synced. Maybe there is a way to improve it, but we haven't thought of something yet. Matt and I are actually planning to discuss this kind of duplication stuff pretty soon. Definitely interested to hear if you have ideas!


Cleaned Up Code in git_test

Cleaned up some code in git_test.go
There is no need for numCollectors and collectorsDone, because numCollectors is always 1 and collectorsDone is only incremented when case err := <-errChan

Signed-off-by: nathannaveen<EMAIL_ADDRESS>

I think I prefer the first version since it is easier to update to run multiple collectors in parallel during the test.

Wouldn't it better to do that when we have multiple runners? Thoughts?

Hey @nathannaveen! Thanks for the PR. So there is issue #281 where we wanted to call the collector.Collect in all the collector tests (i.e. file, gcs, git...etc). That would be a better approach to cleaning up the collector tests. Let me know and I can expand on the issue if you need more guidance.

@pxp928 I can take it up, but I don't fully understand, could you explain? Thanks

@pxp928 I can take it up, but I don't fully understand, could you explain? Thanks

@nathannaveen I updated the issue with an example that should make it easier to understand. That will have to be done for all the collector tests (file, gcs, git and oci). Let me know if you need more guidance

Closing PR as there is a better path forward on issue #281


Endpoint apply_workflow is not compatible with submit_workflow function

Compare endpoint apply_workflow in app/api/v1/project.py and function submit_workflow in app/runner/__init__.py.

Note: this is still a valid issue after #198


Incorrect upgradable version shown on the helm v3 apps deployed from legacy partner apps

Note: This ticket needs an RC while validating
Rancher Server Setup

Rancher version: 2.6.3 & 2.6.3-patch1-rc3
Installation option (Docker install/Helm Chart): v1.21.7

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): RKE1

Proxy/Cert Details:

Information about the Cluster

Kubernetes version:  v1.21.7
Cluster Type (Local/Downstream): DO infrastructure RKE1 1 etcd+ cp, 1 worker

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)
Admin

Describe the bug
This was found while testing an apps migration scenario. If we install an helm3 v1 app on ember UI [rancher max version set to 2.6.3] eg., aws-event-sources and then from explorer --> apps & market place, the app displays an incorrect upgradable version of the app.
To Reproduce

In 2.6.2 rancher, enabled legacy feature flag
installed aws-event-sources version:0.1.1 from legacy --> project --> select default namespace in the downstream cluster
upgraded to 2.6.3 or 2.6.3-patch1
Navigate to Apps & Marketplace on the downstream cluster > Installed apps

Result
An incorrect upgradable version 0.1.901 is shown on the app aws-event-sources.
Expected Result
The version that is available in the app is expected to be displayed.
Screenshots

Additional context
Should be verified for migration steps as well:
Migration steps:

From cluster --> default project --> apps, install the app AWS-event-sources, version 0.1.1 - app1 from ember UI

Add helm3 repository in the apps & market -> repositories: name: helm3-library | URL: https://git.rancher.io/helm3-charts | Branch: master from vue UI
 kubectl get apps -A | grep aws-event-sources
kubectl edit app aws-event-sources -n aws-event-sources

add annotations under spec.chart.metadata.annotations

catalog.cattle.io/ui-source-repo: helm3-library
catalog.cattle.io/ui-source-repo-type: cluster
apps.cattle.io/migrated: "true"

From the local cluster
kubectl get apps.project.cattle.io -A | grep aws
kubectl edit apps.project.cattle.io aws-event-sources -n p-mr57l and add the annotation at metadata.annotations

"cattle.io/skipUninstall": "true"

From apps & market place, verify the app version displayed on the app which would be 1.9.1

@anupama2501 Is the migration path somewhere documented?
I've the same issue, after the Rancher upgrade I prefer to go with the recommended Apps & Marketplace feature. I’ve created the new Repository form Git (same as the old Rancher Catalog) and see all my apps in the new UI screen. But wenn I want upgrade my app I receive the message:
This is a Legacy App and it cannot be modified here
Navigate to Legacy Apps

same problem as @mguggi. But in my case, I can upgrade applications in the new Apps & Marketplace part in older kubernete version. After upgrade to kubernetes 1.20, I have this message

@AndrewHoffmanQA would you mind trying to repro this in 2.7-head?

Didn't reproduce exact behavior - attempt made in 2.7-head
Legacy tag: active
Installed from ember UI
added annotations as directed in reproduction steps

catalog.cattle.io/ui-source-repo: helm3-library
catalog.cattle.io/ui-source-repo-type: cluster
apps.cattle.io/migrated: "true"
"cattle.io/skipUninstall": "true"

In legacy apps view (ember UI) shows correctly with an upgrade and correct version listed. Installed aws-event-sources:0.1.0

In the rancher UI view, correct installed version is listed; no upgrade version listed; no 'upgradeable' tag:

closing
discussed issue with eng manager and project manager.
this is a legacy feature. it's no longer showing an incorrect version.
due to the edge case nature of this issue, it won't be prioritized for further work.


Request: Ability for ChoiceSet and Toggle text to wrap so that the text is all displayed

Summary
Sometimes authors have choice sets that have lots of content, and they need that content to all be displayed. Truncating isn't an option, since information would be lost.
Same would be desired for Input.Toggle.

Asks

Outlook
kmkaliraj (Bot author)
dkonyayev (Author?)
Shon-Elias

Thank you for creating an issue on this. Our team is waiting on the fix.

Shipped in 1.2!


API Endpoint has now been depreciated

As per https://developer.microsoft.com/en-us/office/blogs/deprecatation-of-the-identityriskevents-api/ the new API Endpoint in Graph is riskDetection instead of the beta page.

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

ID: 5a1b0992-84f5-43f1-6b18-b690dde9753b
Version Independent ID: db4234aa-2047-33db-6c8a-19a7b732074b
Content: Microsoft Graph API for Azure Active Directory Identity Protection
Content Source: articles/active-directory/identity-protection/howto-identity-protection-graph-api.md
Service: active-directory
Sub-service: identity-protection
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@Ziggista Thank you for reaching out. We will investigate and update the thread further.

@MicrosoftGuyJFlo Could the Endpoints be changed to the new "riskDetections" in these examples ?


Implement GitHub actions for CI

Successfully set up actions for linting, building, and testing Go. As well as code scanning.


Issue with closing brakets

Hi guys!

VSCode Version:

OS Version: Windows 10

Steps to Reproduce:
For some reason, some times, the powershell functions I write cannot be collapsed. In the GIF here under you that the small "+" in front of the function Manage-PowerPlan doesn't appears.
Sometimes adding more spaces between the function body and the command based help solves the issue, but it doesn't fix the issue at 100% of the time. I haven't found a logic on 'why it occurs', it seems to be random.
see here the recorded issue:

Cheers

copy code from gist--> https://gist.github.com/Stephanevg/5e345eb976da1eb276384b694fe8e5cd
Try to open close the function brakets

Reproduces without extensions: No

Folding is purely based on indentation. You have to indent the function body.


16colors2shades

I changed the generic_adjust function, so it produces 16 colors by default.
Added command line option "--nine". which reverts to the original 9 color output.
Colors 1-7 are darker shades of the latter 8.
All backend scripts, use the generic_adjust function, instead of "manualy adjusting". This means that the different backends, produce similar outputs, but not with the same palletes.

By the way, If someone wants the original functionality, one only needs to change his templates. Could perhaps be easier if you could choose this fith a command option.

tried this version, if you run wal with --backend wal it produces the original functionality of 9 different colors

New commit should fix the issue.

really good pr, hopefully it gets merged soon

Also adding my support for this PR, hope it gets pulled in soon.

I really like it. Thank you! I guess I will stick to this fork for now.
With this feature building themes with hover/focus/active state will be easier for me

Wonder why isn't this merged yet?

Perhaps making this a permanent fork ?

@ZerdoX-x nice kaga btw

going through the diff i think i see some reasons why this isn't getting merged any time soon.

change of default behaviour, if anything the default of 9 colors should be kept and the generation of 16 colors should be the new flag
2 alacritty templates added, the templates should be their own PR
colorscheme paths now separated into 9 and 16 colors, if anything pywal should not make that difference and differentiating the number of colors in a theme should not be done in the path but rather appended to the theme name, as this breaks compatibility with the built in colorschemes like the base 16 ones
color 0 was changed shade for no good reason, the shade generated by default was perfectly fine

@dylanaraps did i hit the nail with this?
honestly i don't feel like doing this myself and opening a new PR just for that, nor forking for that matter, but if i have i will do it and create an additional branch to track upstream commits if 16 colors functionality is to never be merged.

Hey @eylles,
You have some valid points.

change of default behaviour, if anything the default of 9 colors should be kept and the generation of 16 colors should be the new flag

I agree with you, that 9 colors should most probably have been kept as the default functionality, as to change the familiarity of the colorschemes the software created by default up to now. Perhaps it is one of the reasons the pull request hasn't been merged, as if @dylanaraps wanted pywal to output 16 colors by default, they would have done so from the start.
The main reason I changed it, is because it seemed to me as if 16 colors was the sensible default.

2 alacritty templates added, the templates should be their own PR

I don't understand this. Adding the 2 extra templates, doesn't make sense to me as being a reason to not merge the pr. Why create another pull request when this is something extra added with this one?

colorscheme paths now separated into 9 and 16 colors, if anything pywal should not make that difference and differentiating the number of colors in a theme should not be done in the path but rather appended to the theme name, as this breaks compatibility with the built in colorschemes like the base 16 ones

I don't really understand what your suggested alternative here is.

color 0 was changed shade for no good reason, the shade generated by default was perfectly fine

I have no good reason for this. I guess just ocd. I just thought the code looked better that way.
I'm planning on returning to this pr and perhaps creating a permanent fork, as I don't see @dylanaraps ever mergin this. I'm just very busy with school atm.

I like to put my actions where my mouth is, so i already started working on a branch to correct the probable reasons as to why the 16 colors functionality isn't being merged:
https://github.com/eylles/pywal/tree/16_cols
My blockade now is thinking of a good nane for the 16 cols flag, ideally this would be merged into pywal after the streamlined PR is done as i really do not want to maintain a fork of pywal since i really don't think it is ideal, bit if it has to come about that.... welp...
On the templates thing, the usual way i think of PRs is that they provide 1 feature or fix 1 bug, as such making pywal generate 16 color palettes would be 1 feature, while providing alacritty templates would be another 1 feature, thus being 2 features in the same PR...

after quite some hiatus i finally got my ass to address the remaining work, also had to make the fast_colorthief backend use the adjust function.
https://github.com/eylles/pywal/tree/16_cols
@sonjiku i can always PR to your master branch first if u want.

yeh, forgot to mention now a permanent fork exists https://github.com/eylles/pywal16
already merged some other pending PRs.

on the way to 3 years since and finally available at pypi https://pypi.org/project/pywal16/

@eylles I can see! Some time ago I found out that your fork of my fork made it's way to the AUR, has had videos made about it, and has even inspired some functionality in wallust, which is what I'm using right now. Until I get back to finishing a small project of mine, at least, haha.
I'm really happy with how a 2 days' summer project ended up being liked by as many people as it has! I'd appreciate though if some credit was given to me in your repo, since, even though you're the maintainer doing the heavy lifting, a lot of people think that you came up with this which is wrong.
Anyways, I appreciate you taking the torch and keeping it alive!

I'm really happy with how a 2 days' summer project ended up being liked by as many people as it has! I'd appreciate though if some credit was given to me in your repo, since, even though you're the maintainer doing the heavy lifting, a lot of people think that you came up with this which is wron

i don't even credit myself on pywal16, author email still points to dylan and didn't even add a maintainer field, the commit history is king


chore: [main/1.2] wrong paths to plugins in supported plugins table

What does this PR do?
chore: wrong paths to plugins in supported plugins table
Signed-off-by: Nick Boldt<EMAIL_ADDRESS>Screenshot/screencast of this PR
N/A
What issues does this PR fix or reference?
N/A (or see commit message above for issue number)
How to test this PR?
N/A
PR Checklist
As the author of this Pull Request I made sure that:

[x] The Eclipse Contributor Agreement is valid
[x] Code produced is complete
[x] Code builds without errors
[ ] Tests are covering the bugfix
[x] The repository devfile is up to date and works, if one exists
[x] Sections What issues does this PR fix or reference and How to test this PR completed
[x] Relevant user documentation updated
[x] Relevant contributing documentation updated
[x] CI/CD changes implemented, documented and communicated

Reviewers
Reviewers, please comment how you tested the PR when approving it.

note: this needs to be cherrypicked to 1.1.x docs, AND is not complete. some paths may be missing the -dynamic suffix. cc: @davidfestal

squashed and rebased; see also https://github.com/redhat-developer/red-hat-developers-documentation-rhdh/pull/155 for 1.1.x branch.


Can InputBlockResponse include metadata?

I would like to include a metadata into a webhook response via ViewSubmissionContext.response(InputBlockResponse).
Currently, InputBlockResponse doesn't include metadata property. Is that missing from this object or it doesn't allow to include a metadata into a webhook response?

Hi @baole, thanks for taking the time to report this. At a glance, this seems to be an existing bug. We will look into this later and resolve it in future versions.


Difference between dialects not solvable by sqlglot

concat problem
concat means different things in duckdb and pyspark.
In pyspark, concat(a,b) returns null if a or b is null
In duckdb, concat(a,b) treats null as '' when it concats
This could create problems for computing the number of comparisons generated by a blocking rules

@RobinL this can be fixed in sqlglot, you need make this a first class concept and have each output the correct IR. this is similar to the different treatments of dates between duckdb and pyspark

Thanks @tobymao  - that's very useful to know.
I've been roughly making notes of SQL transpilation issues here and here so we can triage the ones we want to try and do PRs to sqlglot for.  Didn't want to litter issues in sqlglot itself until we can express them a bit more concisely and hopefully offer PRs.
Another thing we'd like to do is add support for Microsoft SQL server.  If we were to do this, this is something we are more likely to be able to do incrementally, rather than a 'big bang' PR that adds full support.  Is that way of working something you'd be amenable to?
No obligation whatsoever, but if it'd be useful to discuss how we can contribute (and what we're using sqlglot for) I'm at:
a = ".gov.uk"
<EMAIL_ADDRESS>+ a)

Otherwise we're happy to continue doing sporadic PRs as and when we find time.

i'll send you an email


Add "hospital medicine" as a pre-set clinical tag

Requested by a faculty member.

I just retried this by setting "Hospital Medicine" to "public" on the admin end but it does not show up as a pre-set tag in the user interface.

Yeah, "public" tags don't work yet. I'll let you know when I get around to that.

9751153


Renamed codeql workflow filename

Current dependencies on/for this PR:

main

PR #9

PR #10   👈

This comment was auto-generated by Graphite.

@CatChen queued this pull request to merge with Graphite.

@CatChen merged this pull request with Graphite.


Experiencing a double canvas creation on initialisation

Not sure if this is due to my coding or a bug in the plugin. Has anyone else experienced this?
When I run .maphilight on a DOM element, I get 2 canvas elements created instead of just one.
If I then rerun .maphilight on the same DOM element updates affect the second canvas element, the first remains unchanged.
The DOM element I'm working with comes in via Ajax, but the plugin doesn't generally have a problem with that.
Anyone else come across this?

There should indeed be two canvases... if you have an alwaysOn area.


fix: out of scope let-variable can appear in fun binder type of metav…

This PR fixes a bug in checkAsssignment.
When unification tries to solve ?m a₁ ... aₙ =?= v by assigning ?m := fun a₁ ... aₙ => v, a number of things needs to be checked.

occurs check, that is, check that ?m doesn't appear in the assignment
check that the free variables in the assignment are present in the local context of ?m. If a let-varaible isn't in the local context, then this can be resolved by instantiating it with its definition.
check that the metavariables in the assignment have local contexts that are at most as large as the local context of ?m. If a metavariable has a local context with extra free variables, this can be resolved by assigning it to a new fresh metavariable with a restricted local context.

Previously, checkAssignment does checks 1-3 on v, and does check 1 on the types of a₁ ... aₙ and on the type of v.
However, as reported in #5939, checks 2-3 also need to be executed on the types of a₁ ... aₙ, because these types appear as lambda binder types in ?m := fun a₁ ... aₙ => v.
This PR fixes checkAssignment: besides returning an updated v, it now also returns an updated LocalContext. This LocalContext is then used in mkLambdaFVars in order to obtain the correct lambda binder types.
In order to also fix #5387, I modified the function Lean.Meta.CheckAssignment.checkMVar, so that it can use the checked types of a₁ ... aₙ, which makes it so that it can avoid removing some of a₁ ... aₙ from the context of the metavariable that is being reduced.
This PR also removes the function mkLambdaFVarsWithLetDeps. This function was introduced as a fix for a bug that is independently fixed by this fix. Additionally, mkLambdaFVarsWithLetDeps gives surprising behaviour in that it creates let-binders for let-variables that would normally just be instantiated (see check 2). Additionally, mkLambdaFVarsWithLetDeps has a faulty implementation, because it assumes the free variables appear in the same order as they appear in the local context, which may not be true (see this comment).
I added the tests to the file that contains the original test for mkLambdaFVarsWithLetDeps
Closes #5939
Closes #5387

We cannot accept such an invasive change to a core component, please consider contributing to less critical parts first!

Ok, I hope this at least helped with understanding what the problem is and how it can be fixed.


First checkin. WIP, fixes for casper-client-rs #57. Needs tests

WIP, so no merging yet. We need to talk about tests, but please check it out and tell me what was missed.

What is an appropriate test for this PR?


Support caching

Could you support caching in some way?  Perhaps using something like #74?  I maintain jade and get many issues such as https://github.com/visionmedia/jade/issues/1581 where people are struggling with performance of compilation.  This is usually the result of compiling a small number of templates many times (once per page where each page has different data but the same template).  I think that if this module supported caching, people would not struggle with this issue.

I would love this :)

+1


[FancyZones] Issue with multiple monitors that have different zone setups

Environment
Windows 10
Version 1909 (Build 18363.778
Windows build number: [run "ver" at a command prompt]
PowerToys version:
Version v0.17.0
Steps to reproduce
Hold Shift and Windows Key on current monitor
Expected behavior
Should only show the layout for that current selected monitor
Actual behavior
Shows monitor layout but then immediately overlays the other monitors layout on top so you get confused on which layout is which
Screenshots
Can't get a screenshot

@vinodj
is this till a problem?


fix(scanner): Revert the merge of read functions in scan storages

With this change it is no more possible to obtain scan results by identifiers, without provenance. In particular, this breaks helper-cli's Create command which always passes an empty provenance, see 1.
This reverts commit 70b1b869815ae1ff4fbdf53828939e25ad98d829.

To me the proper fix would be to start using the PackageProvenanceStorage to retrieve the resolved provenance instead of reverting this commit (there is readProvenances(id: Identifier): List<PackageProvenanceResolutionResult> to get all resolved provenances for a package).

The use case is as follows:

orthw init <ort-result> creates a new empty local file storage and imports scan results by package into that storage
Then orthw uses further orth commands which operate on that storage, such as the now broken Create command.

So, the storage involved is not the shared (usually postgtres storage, but a local one which just contains the scan results of one single OrtResult). The workflow shall remain usable given any OrtResult, without having access to a database. IIUC the above proposal it would not fullfill that requirement.

I've crafted a minimal invasive solution to fix the issue timely: https://github.com/oss-review-toolkit/ort/pull/8613
Please take a look.

To me the proper fix would be to start using the PackageProvenanceStorage to retrieve the resolved provenance instead of reverting this commit (there is readProvenances(id: Identifier): List<PackageProvenanceResolutionResult> to get all resolved provenances for a package).

The use case is as follows:
1. `orthw init <ort-result>` creates a new empty local file storage and imports scan results by package into that storage

2. Then `orthw` uses further `orth` commands which operate on that storage, such as the now broken `Create` command.

So, the storage involved is not the shared (usually postgtres storage), but a local one which just contains the scan results of one single OrtResult. The workflow shall remain usable given any OrtResult, without having access to a database. IIUC the above proposal it would not fullfill that requirement.
Would you have an alternative idea which fullfills this requirement @mnonnenmacher?

The provenance resolutions can also be stored in a local file storage, this is the default if no scan storage is configured. So I think the cleanest solution would be that orthw init not only populates the scan results storage but also the provenance storage. This should only require a few lines of code, as the resolved provenance can directly be read from the scan results.

In particular, would it be an option to drop the filtering by provenance again, which has been introduced in your commit?

I would really not like that as for the ORT scanner it is a central concept that scan results have to match not only the id but also the provenance of a package, this is not only the case for scan result storages but for example also for package configurations.

The provenance resolutions can also be stored in a local file storage, this is the default if no scan storage is configured. So I think the cleanest solution would be that orthw init not only populates the scan results storage but also the provenance storage. This should only require a few lines of code, as the resolved provenance can directly be read from the scan results.

I would really not like that as for the ORT scanner it is a central concept that scan results have to match not only the id but also the provenance of a package, this is not only the case for scan result storages but for example also for package configurations.

We had for a long time a central concept which allowed to retrieve scan results by ID from any storage.
I would have objected to making this impossible in [1], but the rationale provided there was different.
Also the commit which introduced the filtering, did sound like a non-functional change in the commit message.
To me it does not look too unnatural if an API which allows storing things by ID allows retrieving things by ID.
Anyhow, I propose to shortcut discussion by merging https://github.com/oss-review-toolkit/ort/pull/8613.
[1] https://github.com/oss-review-toolkit/ort/pull/7641

Alternative solution has been merged: https://github.com/oss-review-toolkit/ort/pull/7641


Add Images with Underscore in Busy.org

When I upload Images in Steemit it creates a path which includes steemitimages.com and then the whole path.
For Example : https://steemitimages.com/ +  DQmaYauBqmx5z8GiwJTKPpinTQ6ebxetwhJKqakruLEwDQJ/IMG_20170302_105004917.jpg
Now if you have an Image which has underscore ("_") in it, busy.org remove that underscore from the URL.So, when you try to copy the URL to busy.org, it looks something like below where the underscore is missing.

Open Source Contribution posted via Utopian.io

This issue is considering Busy 1.

Hey, thanks for reporting
This bug is related to Busy v1 which is about to retire on 1st of the November. Because of that only security bugs are in our scope. You should really use Busy v2 (which now live here: https://busy.org).


[6.0.14] add tarball tests

This change is

Create a PR with the correct base branch. Currently it's merging of 6.0.14-devel into the 6.2.6-devel branch.


FXML is not a data language

Preliminary Steps
Please confirm you have...

[x] reviewed How Linguist Works,
[x] reviewed the Troubleshooting docs,
[x] considered implementing an override,
[x] verified an issue has not already been logged for your issue (linguist issues).

Problem Description

".fxml" is currently listed as data language, however it should be listed as a markup language.
(sources: 1, 2)
Expected language:
FXML
Detected language:
None

Thanks for the report @hypothermic!
What's the user-visible issue you're trying to fix here?

@pchaigno It would be nice if user's could see that there's FXML included in the project.
Keep in mind that this is just a suggestion to make Linguist more accurate.


[INFINITY-2046] Tests for Failed sidecars causing scheduler to recover healthy RUNNING tasks

Add necessary utilities.
Enhance sidecar.yml to have a toxic sidecar.
Add test.

@benclarkwood would you like to do the honours?