Playing youtube video fails all the time after loading once in airplane mode

I'm running into a weird behavior in my app where if I try to display a youtube video by setting the source:html of a webview to the youtube API reference code while the phone is in airplane mode (or with a bad network connection causing the request to fail), any attempt to load any youtube video with the same code on that phone will be failing. (restarting, uninstall/reinstall do not fix the issue)
Here are steps to repro:

react-native init AwesomeProject
cd AwesomeProject
modify the content of app.js as follow:

/**
 * Sample React Native App
 * https://github.com/facebook/react-native
 *
 * @format
 * @flow
 * @lint-ignore-every XPLATJSCOPYRIGHT1
 */

import React, {Component} from 'react';
import {Platform, StyleSheet, Text, View} from 'react-native';
import { WebView } from "react-native-webview";

const instructions = Platform.select({
  ios: 'Press Cmd+R to reload,\n' + 'Cmd+D or shake for dev menu',
  android:
    'Double tap R on your keyboard to reload,\n' +
    'Shake or press menu button for dev menu',
});

const htmlBody = `<!DOCTYPE html>

<meta name="viewport" content="initial-scale=1.0">

`;

type Props = {};
export default class App extends Component<Props> {
  render() {
    return (
      <View style={styles.container}>
       <WebView
                style={{
                  height: 200,
                  width: 340
                }}
                javaScriptEnabled
                domStorageEnabled={false}
                source={{
                  html: htmlBody
                }}
              />
              <Text style={{flex:1}}>extra test</Text>
      </View>
    );
  }
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    justifyContent: 'center',
    alignItems: 'center',
    backgroundColor: '#F5FCFF',
  },
  welcome: {
    fontSize: 20,
    textAlign: 'center',
    margin: 10,
  },
  instructions: {
    textAlign: 'center',
    color: '#333333',
    marginBottom: 5,
  },
});

react-native run-android

let the video load, see the video

turn airplane mode on on your phone/computer

reload app, video is not shown

turn airplane mode off

reload app

=> video does not load
I tried on android 8 and this repro consistently. Android 9 seems to fix the issue according to my test on the emulator.

So the repro steps are not correct as I can't repro it anymore but we still have an android phone that is suffering from those symptoms.
Essentially the webview is just showing up white, looks like no html is ended up being loaded

I did some more digging and I think this is a dup of #298


fix(view-compiler): fallback to node.data when wholeText is not implemented

node.wholeText does not exist in jsdom's DOM implementation (used e.g. for server-side rendering)

As a workaround in pal-nodejs's dependencies, jsdom can be changed to @jdanyow fork github:jdanyow/jsdom#wholeText.

I feel we should do what @Gheoan suggests since we cannot be sure how soon jsdom will review @jdanyow's PR. Otherwise we could be blocked indefinitely. @EisenbergEffect I'll do the dependency swap in pal-nodejs if that's okay with you, unless you'd prefer money-patching the original jsdom for now?

Let's go with the dependency swap, assuming @jdanyow is good with keeping that fork alive/updated until his PR is merged.

sure- @niieani, let me know if you need me to do anything.

So I just tried, it's not enough to install the github:jdanyow/jsdom#wholeText fork because it needs some generated files which aren't there in the repo. @jdanyow would need to publish his fork on NPM after running the jsdom build script, or make a #wholeTextBuilt branch which we could then use.

OK, maybe a polyfill for jsdom is the best idea after all. Maintaining a fork is just too involved.
I've made a PR to add the polyfill to pal-nodejs.

Closing this since we merge the polyfill into our pal-nodejs.


[crypto] Harden mod_inv computation for ECDSA P-256.

Partial fix for https://github.com/lowRISC/opentitan/issues/15507 (fixes for P-256 but P-384 still required)
The previous implementation of mod_inv (modular multiplicative inverse) handled the secret scalar k in plaintext. Even though k is ephemeral, even a very small amount of leakage from k over many signatures can allow an attacker to compute the ECDSA long-term secret using lattice attacks. Therefore, this changes mod_inv to handle k (and its inverse) in two shares so that k is no longer handled in plaintext anywhere in the P-256 code.

Better methods for this have come to my attention, so I'm going to close this PR and open a new one with different techniques.


fix: improve slash command conditional

Description
Fixes #8651
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Manual
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

/ok-to-test sha=edc4051

/ok-to-test sha=96f1f18


v0.8.3 bug fix: ping endpoint

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


fix(docs): Add polyfill for path

Webpack 5 doesn't have polyfills for path anymore; this adds a polyfill so users don't have to do this on their side.

Preview: https://virtual-assistant-pr-virtual-assistant-161.surge.sh


Sort channel names the same way as Zulip web

On the subscriptions / channels page, the sorting of channels by name doesn't fully match the behavior we have in the Zulip web app. In particular, we got a user report that names starting with an emoji are being sorted to the end, while they appear at the beginning in Zulip web.
We should match the ordering of Zulip web, so that people see their channels in the same order when they move between using Zulip on one platform vs. another.
If doing so exactly is impractical or difficult, then we should (a) do a hack that at least handles an emoji at the start of the name, matching Zulip web's ordering for emoji vs. non-emoji and for emoji vs. other emoji; and (b) file a follow-up issue for matching the ordering exactly.
Implementation
The name isn't the only key that the subscriptions / channels list is sorted by; it's also sorted by the channel's status as pinned and/or muted. The role of pinning and muting should remain unchanged.
This issue applies to everywhere a list of channels appears. That means:

the inbox page
the subscription-list page (accessed through the "channel" icon in the bottom nav bar)
the all-channels page, once implemented: #188

Web's left sidebar has three headered groups: "Pinned", "Active", "Inactive", in that order. Within a group, non-muted channels come before muted ones.
After that, sorting is done on channels' names with this compare function:
// If we can, use a locale-aware sorter.  However, if the browser
// doesn't support the ECMAScript Internationalization API
// Specification, do a dumb string comparison because
// String.localeCompare is really slow.
export function make_strcmp(): (x: string, y: string) => number {
    try {
        const collator = new Intl.Collator();
        return collator.compare.bind(collator);
    } catch {
        // continue regardless of error
    }

    return function util_strcmp(a: string, b: string): number {
        return a < b ? -1 : a > b ? 1 : 0;
    };
}

export const strcmp = make_strcmp();

I assume the Intl.Collator behavior is what most web-app users get these days.
Dart doesn't have an analog of Intl.Collator. Could we get close to that behavior by using libicui18n via ffi?
Otherwise, would unorm_dart be useful for this? That library has come up as the preferred library for filtering lists of strings insensitively to diacritics, like #1067.

Thanks, that's helpful!
Yeah, I think using an ICU library via FFI is probably the full solution. That could be something we do ourselves once FFI support is more mature, as the Flutter roadmap calls for next year. Or it looks like people are working on building such a thing for everyone (and for the full scope of ICU functionality): package:icu4x, published by "unicode.org". Still a work in progress but has Unicode engineers working with the Dart team to build the things they need to make it all work:

https://github.com/unicode-org/icu4x/issues/4689

(And that is indeed using FFI; here's the Dart code implementing Collator.compare.)
In the meantime let's do this:

a hack that at least handles an emoji at the start of the name, matching Zulip web's ordering for emoji vs. non-emoji and for emoji vs. other emoji

I'll file a separate issue for that nearer-term hack, and edit this issue to focus on the full solution. (Revising the plan I said above.)

matching Zulip web's ordering […] for emoji vs. other emoji

This part will also require essentially the same data, so I'll leave it out of the near-term issue. The emoji portion of the ordering is in CLDR here:
https://unicode.org/cldr/charts/46/collation/root.html
https://github.com/unicode-org/cldr/blob/9bbbc7769d6/common/collation/root.xml#L924-L1364
(Though we have a separate plan to get a version of that data, from the server:

#1201

So once we've done that, we might revisit the hack to use that information, if this issue for the full solution is still open.)

OK, filed:

#1202

And I'll push this back to M7 Future.


‘Fail to import contact’ error is shown and contact does not get synced when user navigates to ‘Send or Request’ page

Frequency: 90%
App version: Android Celo Alfajores Wallet build : v1.6.0 (101648579), Android Playstore build : 1.6.0 (104794306)
Repro on: Samsung Galaxy A5 (7.0), Samsung Galaxy J7 Pro (8.1.0), LG W30 (9.0),  Samsung Galaxy J7 Prime (6.0.1)
Testing Account:<PHONE_NUMBER>4 /backup Key
Repro Steps:

Launch the app and Login with testing account
Tap on ‘+’ icon and Make sure syncing contact process is on
Wait for some time till syncing gets completed and Observe

Current Behavior: ‘Fail to import contact’ error is shown and contact does not get synced
Expected Behavior: Contact should be synced successfully
Attachment: Fail to import contact error is shown.mp4 (Bug observe at 40 sec)

Verified with Android Play store build v1.6.1 and observed that issue is still reproduced
Verified on Device: Samsung Galaxy A5 (7.0), Samsung Galaxy S5 (6.0.1)

@jeanregisser and @jmrossy  Verfied with iOS Test Flight Build v1.6.1 (23) and observed that issue is still reproduced.

‘Fail to import contact’ error is shown and contact does not get synced when user navigates to ‘Send or Request’ page

Attachment: IOSFailToContactError.mp4

@Lss-Ankit Can you share the logs after seeing this failure?

Sure @jmrossy Please find the logs after seeing error message.
Invitelogs.txt

@Lss-Ankit Thanks for this but I need react native + geth logs.
Can you go to the app settings screen and use the Send Logs option?

Sure @jmrossy Please find the logs from "Send as Issue Report" option from setting screen.
Log: celo_logs.txt

@Lss-Ankit I've had a look at the logs. It seems like it wasn't just the contact sync that failed but also the fee calculation.
The logs suggest that geth (our blockchain client) didn't initialize properly. Can you clarify which build you tested with here? Can you also try killing the app and reopening it to see if that still occurs?

@jmrossy I have verified this issue in Android play store build : 1.6.1 (104794404)
Verified on Devices : Samsung Galaxy A5 (7.0), Samsung Galaxy S5 (6.0.1)

I have verified this scenario in device having ~100+ contacts.
Contact does not get imported for the first time, it shows 'Fail to Import contacts' error message.
50 contacts gets imported after killing-relaunching the app first time
All contact gets imported after killing-relaunching the app second time

Attachment : Contact does not get imported

@Lss-Ankit are you able to reproduce this on latest build? If so, please feel free to reopen it.

Hi @tarikbellamine i have verified that issue on Play store build v1.8.2 and Test flight celo integration build v1.8.2 (40) and observed that issue is not reproduced.


Clicking Party Mode in Table of Contents doesn't bold it

Clicking Party mode still highlights Marathon mode.
Suggested improvement: add some margin to the bottom so screen can scroll to have Party mode more centered/higher on the screen like when clicking the rest of the options
https://user-images.githubusercontent.com/17424008/179579663-f2528887-5563-4d6d-b30f-c50be711bbb1.mov

I think this has to do with screen height? For my screen its Marathon mode that doesn't get bolded instead. Not sure what we can do about this


Fixed testnet exchanger address, corrected exchanger contract name

Fixed the testnet exchanger address, which can be found in the storage of the evm_bridge contract.
Corrected the contract name: previously, both the exchanger and evm_bridge contracts shared the same evm_bridge name.

Hi @timothymcmackin,
The Ghostnet exchanger address in the docs is mixed up with the evm_bridge address. It’s nothing serious, but there’s a simple fix for it.
Thank you!


Load project name follow docker compose standard

Use ProjectOptions to load the project as what docker do, with the following differences:

Include only COMPOSE_PROJECT_NAME env var in the loader environment, instead of all
We currently do not support env files
Do not read .env files (could be changed soon)

ProjectOptions takes care of the logic of using COMPOSE_PROJECT_NAME or parent directory name as the project name if it is missing from the compose file, it also contains the same logic how docker normalises the project name.
Also enable .env file support

Fixes #469


how to add images dynamically for *ngfor loop

how to create multiple gallery in *ngfor loop in angular 2 project

@kathiraashi  I am closing this, if you still have problem let me know or reopen


getting statistics for batchnorm

Hi Chelsea,
I'm reimplementing MAML for few-shot classification, and I have troubles understanding how exactly you use batch norm. I was hoping you can help me out and clarify so I can understand this better. Below I'm assuming N=5-way k=1-shot learning and 4 tasks per meta-update.
During training, in the inner loop update, how do you compute the mean/var for batch normalisation? Do you

use the (5x1)=5 images from the current batch, so the task-train set?
use the (5x(1+15))=80 images from the current task, so the task-train and task-test set?
use the (4x5x(1+15))=320 images from all tasks in the current meta-batch?

Also, does this differ depending on how many gradient update steps you do? And when you're evaluating, will you use the same procedure?
Thanks a lot in advance!

The batch statistics are always computed using the current batch. This does not differ with variable number of gradient steps, and it is the same during meta-training and meta-testing.

Thank you so much for the quick response!


Add brand colour for Department for Levelling Up, Housing and Communities (DLUHC)

This is a new department, so adding a new brand colour for this organisation.
Zendesk ticket

Thanks @owenatgov.  I don't have permissions to merge on this repo, would you be able to do it for me?

I don't have permissions either sadly. Have prompted the design system team who'll have a look in a bit.

Thanks for raising this @brucebolt - looks good to me. This just needs a second review from someone in the Design System team before it can be merged.


Preview not working

Describe the bug
The preview isnt working.I tried almost everything.Someone said to try and run it in canary but it still isnt working.I have tried running manually in the terminal and then open the link but the page will not load and gives error "ERR_CONNECTION_REFUSED".Any so that i can get to see my preview?
Link to the Bolt URL that caused the error
bolt.diy
Steps to reproduce
When its finished with the code,it will execute the "npm install and npm run dev" command and then the preview will be loading forever
Expected behavior
i want to see the normal preview tab
Screen Recording / Screenshot

Platform

OS: Windows 10
Browser: Google Canary

Provider Used
Open Router
Model Used
Gemini flash 8B
Additional context
No response

Same issue here.

same here.

Can you please provide your chat history ("export chat" button). Easier to verify on my system.

chat-2024-12-15T14_34_45.832Z.json
of course, here is the json file

Same issue but on M1 Macbook Pro using Arc Browser.

Same issue here. We need a general Error fixing element for the errors appearing. We do not know exactly. It could be Hallucination, it could be vite config error or just network connection.

This has nothing to do with the browser you use or adblockers as previously stated. This is blocking the core functionality of the bolt diy app.

Who wants to fix this with me?

preview not working for me either. I just prompted "Create a hello world html page" and it doesn't show a preview

Hi guys,
please take a look a the community and search for your problems:


World of Warcraft: The Burning Crusade Black Screen

Loading game presents a black login screen. Logging in blind you can hear yourself move around but you can only see the minimap move on screen.
Software information
World of Warcraft: The Burning Crusade
Log
https://mega.nz/#!bgQhxa7a!eWYxUOYJMh7XL7Fr18miNVAWciu_a-otS40NY1reYSI
System information

GPU: GTX 760
Driver: NVIDIA
Wine 4.9-staging + Windows: 8.1
D9VK version: 0.12

Apitrace file(s)

https://mega.nz/#!2h5GyAiA!Ef9C737YEY_XDQtKyAa2YKxubZ3_jfAwkAob9L3_t88

I ran the trace under Windows which I read on another issue that would be even better.

@crborga Your apitrace does not render black anymore on latest master. Please recheck your issue. Automated builds of latest d9vk can be found here: https://git.froggi.es/joshua/d9vk/-/jobs

Okay everything besides inventory icons and actionbar icons seems to work now. I will have to run another apitrace and post screenshot.
Sent from BlueMail
On Jul 4, 2019, 3:01 AM, at 3:01 AM, Riesi<EMAIL_ADDRESS>wrote:

@crborga Your apitrace does not render black anymore on latest master.
Please recheck your issue. Automated builds of latest d9vk can be found
here: https://git.froggi.es/joshua/d9vk/-/jobs
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/Joshua-Ashton/d9vk/issues/156#issuecomment-508365112

Nahh its fine the The Burning Cursade apitrace shows the issue and I already forwarded it to Josh. There is an issue open that already mentions the icon issue, so you can close this one. :)


HBASE-26034 add support to take parallel backups

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
7m 22s
Docker mode activated.


Filtering does not display folder content when it should

Expected Behavior
When using the filter search, if a folder matches, it should display all requests inside the folder.
Let's say I have a folder called Github, with a request inside name "GET Something", if I search for "Github", it should display the Github folder and all the content inside.
This is problematic because in my usage of Insomnia, all requests are basically named the same ("GET Products") and all domains are retrieved by a plugin.
Actual Behavior
Requests are only displayed when they match the filter
Reproduction Steps

Create a Github folder
Create a request inside this folder called "something". The request is a GET to example.com
Filter by searching "github"
Only the folder name is displayed, not the "something" request

Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
No response
Insomnia Version
2023.5.6
What operating system are you using?
Ubuntu
Operating System Version
Linux 6.2.0-26-generic #26~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Thu Jul 13 16:27:29 UTC 2 x86_64 x86_64 x86_64 GNU/Linux
Installation method
insomnia.rest deb
Last Known Working Insomnia version
No response

same problem

Filtering has become useless, if you put requests into folders by project and then try to search for the project folder with the filter, you can see the folder, but nothing in it, and selecting the folder and turning off the filter does not position you to the folder, so the filter has become a waste of space. I wish I had retained the earlier release like Farenheith] had.

Issue closed, we switched to Bruno 👍


App stops working with OneNote W10

Sometimes app stops working when i'm using OneNote on W10, the keyboard and the sounds don''t work so it gets solved by closing mechvibes.

Works for me, no crashes and the sounds work in OneNote too. Strange.

OneNote is unrelated to this, see #242 #12 #13


Some additional work to get ARIES-1565 working

Wouter, I have been reviewing your pull request for ARIES-1565.  It currently is not working because it uses internals from aries.util which are not exported.  This makes the aries.subsystem.core bundle unresolvable at runtime.  The error from pax exam are far from useful but they are being caused because the aries.subsystem.core bundle cannot resolve.  To hack around this I made all the util packages as package private to the aries.subsystem.core bundle.  This is not ideal, I probably would prefer we add some new API to util and move the unpacking stuff from aries.subsystem.core to aries.util bundle.
After I got past the resolution issue I ran into two more issues.  One was an issue in aries.util which I fixed in one commit.  The other was in the unpacking code which assumes directories are created before each zip entry file is extracted.  I added a fix to ensure the parent directory exists before extracting a zip file entry.  This is important for jar zips which have META-INF/MANIFEST.MF as the very first entry with no META-INF/ directory entry.
After this fixed I am still seeing some issues in org.apache.aries.subsystem.itests.defect.Aries1383Test where it is timing out.  I have not determined if this is caused by the changes in this pull request or not yet.  But I have run out of time to look at this for the week.  I thought it would be good to hand over what I did to you for a look.

@tjwatson Thanks for the contribution!
We detected the resolution issues as well (see https://issues.apache.org/jira/browse/ARIES-1565 for the discussion). It only seems to occur during the tests. We did not experience the issue at runtime. As for the solution, since we're going to have to change aries-utils for your bug fix, I'd suggest we immediately try to include the unpacking filesystem itself in utils as that is where it actually belongs. The only reason we placed it in subsystems is to avoid impacting aries-utils but that does not seem to be possible.
I've been absent for the past two weeks so I'll have some catching up to do but I'll look into it as soon as possible so we can fix the issues in Aries1383Test and include ARIES-1565 in subsystems.

I don't see how the resolution issue is not an issue in general.  The subsystem.core bundle will be importing a package that should never be available for import org.apache.aries.util.filesystem.impl


Bug:  certain links in README file failing

What happened?

How can we reproduce this bug?
Renaming projects in ./projects and not reflecting the same in README.md file
Additional context (if any)
No response
Urgency (Optional)
Medium priority

I can work on this issue.

So I did solve the issue but it eventually got merged. But people did not pull before pushing; undoing what I did.


listener: support update socket_options

Signed-off-by: He Jie Xu<EMAIL_ADDRESS>Commit Message: listener: support update socket_options
Additional Description:
Support to update socket_options through create a new socket
Risk Level: high
Testing: unittest added
Docs Changes: API doc
Release Notes: a behavior change item added
Platform Specific Features: n/a
Fixes #22634

@mattklein123 @ggreenway would you like to give a high-level review on this? I didn't add any tests yet but I did a manual test. And there are some checks that should be added for rejecting the update non-STATE_LISTENING socket options when using clone.
@kfaseela if you are interested, you can have a test, and ensure this is what are you looking for.

@mattklein123 we have three listener fields for three specific socket options, they are transparent, freebind and tcp_fast_open_queue_length, I think they should work with an update. Do we want to support updating those fields if they work? If we reject that, but people still can do a workaround by setting those options in the socket_options manually, then they got the ability to update them.

@mattklein123 we have three listener fields for three specific socket options, they are transparent, freebind and tcp_fast_open_queue_length, I think they should work with an update. Do we want to support updating those fields if they work? If we reject that, but people still can do a workaround by setting those options in the socket_options manually, then they got the ability to update them.

feel freebind's update is meanless, if people change its value, then the address should be changed also.

/wait
the duplicate address check isn't right, working on a fix

@mattklein123 we have three listener fields for three specific socket options, they are transparent, freebind and tcp_fast_open_queue_length, I think they should work with an update. Do we want to support updating those fields if they work? If we reject that, but people still can do a workaround by setting those options in the socket_options manually, then they got the ability to update them.

I would support update if possible so it's not surprising. cc @ggreenway

/wait
still have test need to fix

This approach LGTM.
Will this also work for hot-restart? I think socket options changes aren't applied for hot-restart either.

Yes, we doesn't change socket options when hot-restart, the existing socket option already applied on the socket which passed from the parent, the OS suppose ensure there is no change.
Also checked, although we have the same address between new listener and drainning listener, the hot restart always get the socket from the active listener
https://github.com/envoyproxy/envoy/blob/ad186c448cf8c5e78454d2adf5f228e9c90a997c/source/server/hot_restarting_parent.cc#L110
ListenerManager::listeners() return active listener default. So hotrestart won't get old socket with old options from the parent.

Thanks this is great. Flushing out some more comments. For all the new tests, please try to run them many times to look for flakes. https://github.com/envoyproxy/envoy/tree/main/test/integration#reproducing-test-flakes
/wait

good call, thanks! I saw a segmentation fault in the end of UpdateListenerWithDifferentSocketOptionsWhenReusePortDisabled, let me dig into it.

Yes, we doesn't change socket options when hot-restart, the existing socket option already applied on the socket which passed from the parent, the OS suppose ensure there is no change.

I meant will it work if the socket options have changed in the listener configuration between the parent and the child process in hot-restart? It should probably have the same semantics as an LDS update; logically it's the same change, even though the mechanism is a bit different.

Yes, we doesn't change socket options when hot-restart, the existing socket option already applied on the socket which passed from the parent, the OS suppose ensure there is no change.

I meant will it work if the socket options have changed in the listener configuration between the parent and the child process in hot-restart? It should probably have the same semantics as an LDS update; logically it's the same change, even though the mechanism is a bit different.

I see now, thanks for explaining to me again! I assume people don't change the config when hot-restart at the beginning since some experience told me it will be easy to have separate steps to do the upgrade, then things become easy. But after second thought, seems like everything should be fine except the config related to socket for envoy hot restart
For this case, I don't think it works perfectly, I can imagine two cases don't work well.

if the user removed a socket option after hot-restart, that setup won't be clear after duplicated from the parent. And there is no way to clear the setup since we don't know the platform default value for each socket option.
prebind socket option can't ensure works as expected, since the code will apply the prebind socket option again, but the socket is already bound since it is duplicated from the parent.
https://github.com/envoyproxy/envoy/blob/a3b59dd87abfe6cc65b3f0af31dda1c42b725796/source/common/network/listen_socket_impl.h#L83-L87
setListenSocketOptions will set the prebind socket options

The simple solution here is to suggest the user don't change the socket option when hot-restart, the user can do another update for socket option after hot-restart. Also the user only can append new socket option for STATE_LISTENING when hot-restart. We can document those somewhere. Does make sense?

/wait

The simple solution here is to suggest the user don't change the socket option when hot-restart, the user can do another update for socket option after hot-restart. Also the user only can append new socket option for STATE_LISTENING when hot-restart. We can document those somewhere. Does make sense?

Another option would be, in the hot-restart protocol, for the parent to send the current socket options, and if they're different from the child socket options, invoke the same code path you're writing in this PR to update them. But right now, the child doesn't know the current socket options, so this can't be done without adding that additional information to hot-restart.

The simple solution here is to suggest the user don't change the socket option when hot-restart, the user can do another update for socket option after hot-restart. Also the user only can append new socket option for STATE_LISTENING when hot-restart. We can document those somewhere. Does make sense?

Another option would be, in the hot-restart protocol, for the parent to send the current socket options, and if they're different from the child socket options, invoke the same code path you're writing in this PR to update them. But right now, the child doesn't know the current socket options, so this can't be done without adding that additional information to hot-restart.

Yea, we probably need a separate hot-restart API to get the parent's socket options config. @mattklein123 WDYT? do you have preference on which solution?

Personally I would probably try to keep it simple until someone actively wants this to work for hot restart. If we do want to implement it, passing the old socket options seems like a fine approach but it would be a bunch of work.

Personally I would probably try to keep it simple until someone actively wants this to work for hot restart. If we do want to implement it, passing the old socket options seems like a fine approach but it would be a bunch of work.

I did a check the hot restart api, we probably need to add a new parameter to this hot restart API
https://github.com/envoyproxy/envoy/blob/f3c6c6d250b2387121e113f3ae5c47b81312e05a/source/server/hot_restarting_child.h#L17
The new parameter is used to pass the new socket_options config to the parent. Then the parent will find a socket with the same address and same socket_options, otherwise consider no socket can pass to the child. Then the child found there is invalid fd returned from the parent, then it will create new socket
https://github.com/envoyproxy/envoy/blob/f3c6c6d250b2387121e113f3ae5c47b81312e05a/source/server/listener_manager_impl.cc#L248-L249
new parameter should be a compatible API change for the hot restart API, the old envoy just ignore that parameter.

Yeah I agree ^ will work fine. I just wouldn't implement it until someone actually asks for the support.

Yeah I agree ^ will work fine. I just wouldn't implement it until someone actually asks for the support.

got it, thanks for the feedback! let me add some doc for the hotrestart.

Thanks LGTM with small comment. I would suggest we hold this until after the release. @ggreenway is this OK with you regarding hot restart? Any other comments?
/wait

agree with that, it isn't worth rush before release

/wait
waiting for 1.24 release, then this PR is ready for merge

/retest

Yeah, hot-restart should be a separate PR if/when someone wants it. I agree with merging this post-release.

@mattklein123 thanks!

@ggreenway if no further comments can you approve/merge?

@soulxu seems like this needs merge to resolve conflict
@ggreenway hope you are feeling better, can you take a look when you have a chance? Thank you!
/wait

I'm hoping to review next week (hopefully early in the week)

I'm hoping to review next week (hopefully early in the week)

thanks! no hurry.

/wait

LGTM. Needs merge conflicts resolved.

done, thanks!

Thanks a lot @soulxu @ggreenway @mattklein123  :)


Question: How are pending interrupts undelegated to S-mode handled when mstatus.SIE is unset?

In this issue, https://github.com/riscv/riscv-isa-manual/issues/1393, mentioned that

The global interrupt enables, like mstatus.SIE, have no interaction with delegation. Delegation occurs anyway. Since interrupts are globally disabled for S-mode, any interrupts delegated to S-mode simply won't be taken while in S-mode.

In the spec, https://github.com/riscv/riscv-isa-manual/blob/main/src/machine.adoc, said that

When a hart is executing in privilege mode x, interrupts are globally enabled when xIE=1 and globally disabled when xIE=0. Interrupts for lower-privilege modes, w<x, are always globally disabled regardless of the setting of any global wIE bit for the lower-privilege mode. Interrupts for higher-privilege modes, y>x, are always globally enabled regardless of the setting of the global yIE bit for the higher-privilege mode. Higher-privilege-level code can use separate per-interrupt enable bits to disable selected higher-privilege-mode interrupts before ceding control to a lower-privilege mode.

But in Spike code, https://github.com/riscv-software-src/riscv-isa-sim/blob/1b1a333763eae2e74dbf38b39d9adab39c4bed7c/riscv/processor.cc#L664
void processor_t::take_interrupt(reg_t pending_interrupts)
{
  // Do nothing if no pending interrupts
  if (!pending_interrupts) {
    return;
  }

  // Exit WFI if there are any pending interrupts
  in_wfi = false;

  // M-ints have higher priority over HS-ints and VS-ints
  const reg_t mie = get_field(state.mstatus->read(), MSTATUS_MIE);
  const reg_t m_enabled = state.prv < PRV_M || (state.prv == PRV_M && mie);
  reg_t enabled_interrupts = pending_interrupts & ~state.mideleg->read() & -m_enabled;
...

The pending_interrupts masked by state.mideleg, controlled by the m_enabled.
It seems that interrupts undelegated to S-mode controlled by the mstatus.MIE, not the mstatus.SIE.
Pending supervisor interrupts undelegated to S-mode are not controlled by the mstatus.SIE?
These makes confusion about this situation:

disable the mstatus.SIE, enable the mstatus.MIE
disable interrupt delegation for supervisor software interrupts mideleg = 0x0
mark a supervisor software interrupt to be pending mip = 0x2

What should happen?

If an interrupt is not delegated to S-mode, then SIE is irrelevant.  Only MIE is relevant.

If an interrupt is not delegated to S-mode, then SIE is irrelevant. Only MIE is relevant.

Thanks for reply.
If an interrupt is not delegated to S-mode, the current executing privilege mode is also irrelevant?
All interrupts which are not delegated to S-mode, use the MIE as the global interrupt-enable bit, ignoring the current privilege mode.
Is that right?

If a pending-and-enabled interrupt is not delegated to S-mode, there are two cases:

If the mode is M, the interrupt will be taken if mstatus.MIE=1
If the mode is less than M, the interrupt will be taken regardless of mstatus.MIE.

Thanks a lot, that answers my question.


Access

Mobile popup Menu Created
Project & Work Task.
-Details Fed off an Array Object to display.
-Pop Window Created
Client Side Form Validation Embedded.
Data Preservation added.

Kindly with Due Respect, Time and Partner issue got me in a Time Race that i had to Bundle the Tasks in the interest of Time.
Bear with me and Perrharps Review the Entire Week's Bundle.

@canon20paul, I already told you to create a new pull request for this project. I kindly suggest you, please do it.


403 - Forbidden: Access is denied.

403 - Forbidden: Access is denied.

hi @shine7430
Could you please elaborate?

I will close this issue, Feel free to reopen it if you still have problem.

Hi!
I am trying to access an IIS hosted ASP.NET site from a container. Currently I cannot access the site from my host device.
I am using:
Docker ver. <IP_ADDRESS> , Engine 19.03.12
mcr.microsoft.com/windows/servercore/iis:latest as base image
I have access from outside the container to the the Default Web Site and basic sites (HTML and ASP.NET) I have created. I also have access to the websites inside the container using Invoke-WebRequest -Uri "http://localhost:{Port}" -UseBasicParsing [-UseDefaultCredentials].
One of the sites should be public and not require any credentials, so invoking that one without the default credentials works out. However, I cannot access any of them, public or requiring a login, using the powershell command. The answer is always:
Forbidden Fordidden URL HTTP Error 403. The request URL is forbidden.
The logs within the container (C:\Windows\System32\LogFiles\HTTPERR) show 403.3, write access forbidden, for the public site, which shouldn't be, since the apppool user and the IIS_IUSRS both have received full access on all files in C:\inetpub\wwwroot, given to them using icacls C:\inetpub\wwwroot /grant apppool:(OI)(CI)F /t, same for IIS_IUSRS. Originally I also used "everyone" as the SID for icacls, same result.


Picker design

Add a better design to add/update a picker.
There should be:

a select to choose the type
a picker for the value according the type (color, string etc...)
a checkbox to choose whether it's a getter or not
the list of existing getters for this type
the toolbar can be used to search through existing getters

Missing pickers :

Color
Sound


[Feature request] Reformatting variable names

I would like to see a feature which can identify (and/or fix) variable names written in one format (e.g., snake_case) to another format (camelCase).
This would be risky. This is a regex that I was able to use locally to identify snake_case variables in my project.
\$(this->)?([0-9a-z]+(_[0-9a-z]*)+)

it will not be risky. it will be insanely risky
looking forward for more opinions

Fixing, yes. Identifying, no.

This project is PHP CS Fixer.
We do not identify issues. We fix them!

We do not identify issues. We fix them!

What I think I'm hearing is that you believe that this tool should not bother identifying CS issues if it is unable to completely resolve them. I believe that this is a false assertion.
The --dry-run parameter performs identification of issues. Regardless of this parameter setting, the tool should output the same messaging that occurs whenever it encounters any issues that cannot be fixed.

the output is the same regardless one use dry-run or not.
dry-run is to not store output on disk. not to change used rules.
if fixer cannot fix sth then the rule is problematic to follow in all cases, thus cannot be blindly enforced, so it would bring conflicts between different developer should that case be fixed or not. here you may inherit from parent class that is outside your repo, and for that you stick to camel case name. for that, you cannot fully automate the process.

What I think I'm hearing is that you believe that this tool should not bother identifying CS issues if it is unable to completely resolve them. I believe that this is a false assertion.

That is the scope of the tool, it is also what makes it awesome. It does not only detect but also fixes! happy days :)
The tool could be used to detect a lot of cases it cannot fix, besides the naming of variable also of classes, interfaces, functions and methods, detecting usage of globals, assignment of variables inside conditions etc. etc.. I've a bunch of such fixers I use to profile my projects, however these are not real fixers. These do not fix, just detect. Therefor these do not belong here in the core of project.
What I think will happen is that such detecting tool will be made by someone one day based on the tokenizer of this project. However the first step would be to put that into separate repo so both this fixer tool and the new detector tool can use it. This first step will take a bit (or better a lot ;)) of time to be done by however is concerned.

Closing this issue. Cross-linking with a similar issue posted to PHP_CodeSniffer: https://github.com/squizlabs/PHP_CodeSniffer/issues/1228

In case anyone still needs it, I've made a Rector rule that handles it:
https://github.com/rectorphp/rector/pull/3415


Can generamba be used to generate swift modules?

It may seem stupid question, but I can't find any docs about if it supports swift.

Generamba is only generator,
it supports all languages by using different templates.
We have already PR with Swift template. Some corrections and you will be able to use Swift from box.

PR? Can you provide a link to an example of such template?

https://github.com/rambler-ios/Generamba/pull/59

Here https://github.com/rambler-ios/generamba-catalog/pull/3

Thank you. I've succeeded on generation.


Allow option at runtime to disable uvloop

Is your feature request related to a problem? Please describe.
I'm using sanic with papermill, papermill is not compatible for now with uvloop...
since i distribute my package in pip i don't want to ask my user to do
export SANIC_NO_UVLOOP=true
export SANIC_NO_UJSON=true
pip3 install --no-binary :all: MYMODULE

It should be only simple install since for now only one way exist to install otherwise the module is broken .
Describe the solution you'd like
allow us to disable uvloop usage at runtime like Sanic(no_uvloop=True)
then it will let my users install my module like other one !
pip3 install MYMODULE
Additional context
Add any other context or sample code about the feature request here.

For now my poor solutions is to import this before sanic
import os

class FixUvloop():

    def __init__(self):
        try :
            import uvloop
            with open(uvloop.__file__, 'w') as fp:
                fp.write("raise ImportError\n")
                pass
        except ImportError:
            pass

A+ for creative hacks!
Have you tried?
asyncio.set_event_loop_policy(None)

@ahopkins ahah your solution is way more reliable and works thanks <3

I agree we do need a better mechanism to not install uvloop at all if your project doesn't need/want it. But that is not easy to do. The python extras mechanism doesn't allow for negative dependencies, so the environment variables are for now the only way to do it.
We can probably close this thread now.

@ashleysommer Thanks for the answer, for now Sanic is a part of our module : https://github.com/jupyter-naas/naas
So i cannot force all my users to use env vars before install .
I was asking to allow us to disable it at runtime, since install it with is "okay".
I'm okay to install an "useless" dependency if users don't have to do custom install for one of my dependency.
I'm already doing it with asyncio.set_event_loop_policy(None).
But it's a Hack solution, who create edge case since that not done inside Sanic.
Can you allow us to disable it at runtime in Sanic, like i suggest Sanic(no_uvloop=True) ?

Can you allow us to disable it at runtime in Sanic, like i suggest Sanic(no_uvloop=True) ?

That is probably reasonable.  I would suggest that it belongs inside the Sanic.run() method since it applies to the server. And, I think that it would probably do the same thing: reset the policy.
As @ashleysommer mentioned, it is still not ideal because it leaves a fairly weighty dependency hanging around.
Perhaps another solution is to build out the docker images more, and provide some builds with and without uvloop.

@ahopkins yes in Sanic.run it can be good too :)

Minor note, please avoid negated flag names like no_something=True and use descriptive default values instead, could be async_backend="auto" (uvloop, asyncio, trio)

@Tronic the no_ options predate the handoff of sanic to the current crew of maintainers, but generally I agree.


feat(acvm)!: add eth_contract_from_vk to `SmartContract

chore: deprecate eth_contract_from_cs
Related issue(s)
Resolves #102
Description
Summary of changes
eth_contract_from_cs is now deprecated and eth_contract_from_vk has been added which takes a &[u8] rather than a Circuit.
Dependency additions / changes
(If applicable.)
Test additions / changes
(If applicable.)
Checklist

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt with default settings.
[x] I have linked this PR to the issue(s) that it resolves.
[x] I have reviewed the changes on GitHub, line by line.
[x] I have ensured all changes are covered in the description.

Additional context
(If applicable.)

A deprecation shouldn't be breaking. It's just a warning that something will have a breaking change in the future.

Adding a deprecation shouldn't be breaking. It's just a warning that something will have a breaking change in the future.

Whoops, got a bit trigger happy with all the breaking changes recently :sweat_smile:


Envoy Load Balancer: Connection reset by peer

Is anyone else having issues with the Envoy load balancer present in the Docker Compose deployment? I am not able to talk to the manager service's API on port 9410 (or any of the other services the load balancer is acting as a reverse proxy for) through the load balancer. The manager's API is not the problem since I can curl the API from the host if I map the manager's API port to a local port, and I can curl the API port from within the load balancer container.
I've tried debugging Envoy to see what the issue is but even with it's log level set to debug I'm not seeing anything obvious. Is there a reason why such a complex reverse proxy solution, or even a reverse proxy at all, is being used as part of the default Docker Compose config? I see the need for something like Envoy in a large-scale production deployment of MaEVe in Kubernetes for example, but not in a Docker Compose style deployment.

If you see that Envoy isn't responding, you may find that the restarting that specific docker container resolves your problems (docker compose down lb && docker compose up lb -d). The use of Envoy in the provided docker compose file makes it possible to test the independent scaling of the gateway and manager components, but it isn't required.

I've pushed a change to the docker-compose.yaml file that delays Envoy startup until after the gateway & manager components are running which I hope addresses the issue.

@subnova I just tested your updates to docker-compose.yml and unfortunately I'm still getting errors when using curl to hit the API endpoints.


Update version base

Background / Context

Issue / Requirement / Reason for change

Solution / Feature Overview

Implementation Details

Other Information
Also includes the seed format version update

/retest

/test all


Code model structures ID using Enum

Would be nice to have unique id's for model structures. This would allow for easily creation of time series models via the id and identification of the structure given an id.
Somethign like:
from enum import IntEnum

class Rfunc(IntEnum):
    Exponential = 0
    Gamma = 1
    Hantush = 2

class Recharge(IntEnum):
    Linear = 0
    FlexModel = 1

class Wells(Flag):
    No = 0
    Schoonhoven = 1
    Bergambacht = 2

def make_model(code):
    pass

code = f"{Rfunc.Exponential}{Recharge.Linear}{Wells.Schoonhoven}" # 101

I think it will be difficult to implement something sufficiently generic that works in all/most cases?

What if you have Recharge, River and Wells?
How do you indicate multiple wells in a WellModel?
If you want to indicate both rfunc/stress, you need two numbers for each stress, which already makes it much harder to interpret the code?

My feeling is this is best developed in a specific project. Let me know what you think!

I agree that it is difficult and probably annoying to build. I thought maybe each stress and oseries could get a unique Enum ID from which the model can be built. But I think this would be too cumbersome.

https://www.youtube.com/watch?v=TAMbq0iRUsA


ddcutil: Install udev rool to allow group-based monitor brightness setting

ddcutil and its GUI ddcui allow adjusting the hardware settings (e.g. brightness) of external monitors.
This works with root everywhere, but a better way (on NixOS) is to install the upstream udev rule (see here) so that users of an i2c group can adjust monitor brighness without having to be root.
It would be cool for somebody to try that udev rule, and up in NixOS accordingly.

We could create a hardware.ddcutil.enable option to install the rule but It's fairly trivial to do manually. I don't have an ad-hoc group for i2c but I'm using wheel, in any case it's just
services.udev.extraRules = ''
  # allow admin use of i2c devices
  ACTION=="add", KERNEL=="i2c-[0-9]*", GROUP="wheel", MODE="666"
'';

Still relevant, I think. So, @nh2 how do you propose the udev rule should be installed?

It seems other distros have an i2c group and then install a udev rule granting permission to that group. That seems fairly straightforward and implementable in nixos.
I think this would also enable folks to use gddccontrol/ddccontrol without root, by adding themselves to the i2c that would then exist. The group/rule could be behind hardware.i2c.enable, which could also add i2c-dev to system modules so that dependency would also be taken care of.
cc: @lovesegfault

It seems other distros have an i2c group and then install a udev rule granting permission to that group. That seems fairly straightforward and implementable in nixos.
I think this would also enable folks to use gddccontrol/ddccontrol without root, by adding themselves to the i2c that would then exist. The group/rule could be behind hardware.i2c.enable, which could also add i2c-dev to system modules so that dependency would also be taken care of.
cc: @lovesegfault

Fixed by #111996.
It should be enough to do hardware.i2c.enable = true to use ddcutil/ddcui on a desktop computer.


Error when type value is array not string

When type value is array like:
  "street": {
    "description": "street",
    "type": [
      "string",
      "null"
    ]
  },

library returns "Manatee.Json.JsonValueIncorrectTypeException: Cannot access value of type Array as type String" when JsonSchemaFactory.Load is executed

Originally created at 2016-04-01 13:54:19+00:00 (UTC) by itmind as a(n) major issue.

I'm building some unit tests here as well.

4.0.0-alpha released for your testing enjoyment!
https://www.nuget.org/packages/Manatee.Json/4.0.0-alpha

BUG FIX: #14 - Error when type value is array not string
Added MultiSchema to support arrays in "type" property.
BUG FIX: #15 - JsonSchemaFactory.Load incorrect parse type when enum is a type of other type
Enum schema now populated using EnumSchemaValue instead of JsonSchemaTypeDefinition.
Implemented equality operators on all schema types.
Commit includes refactor with breaking changes.  Version has been bumped to 4.0.
→ <<cset 0a0eab28040e>>

Looks great, Greg! We'll retest it ASAP (should have some results tomorrow at the latest) :-)

Yes, it would be great to have it very soon so we can test it :-)

I have successfully resolved this. I'm going to commit to the repo so that everyone can test.
As far as publishing, I'm going to wait until I have the other issues resolved.

I am currently working this and other issues.  I will release as soon as the bugs are fixed.  I don't have an estimate, I'm sorry.

Yes, this looks like a serious issue. We need that ASAP :-) (+1)

When do you plan to release 4.0?

Seems to me that null could be understood, but I see why you'd want to specify it explicitly.
Working on this now.  Will be part of 4.0 release.  (Coming soon!)

Yeah, I faced this problem as well. I vote for fixing it as soon as possible as I need it for production code.

Considering the following excerpt from the validation spec, I think oneOf is most correct.

String values MUST be one of the seven primitive types defined by the core specification.

Why weird? Lets think about person who must have address ("type": "object") and could have correspondence address ("type": ["object", "null"]). I suppose a good example.

That last example feels weird.  It's like saying, "If it's an object, these are the properties.  But it might not be an object, in which case, the properties don't matter."  I realize it's valid, but it's strange.  I'm having to work some things around to code for it.
Library contracts shouldn't change.  Should be released soon.

"street": "Abbey" -> OK
"street": "null" -> OK
"street": "" -> BAD

I think onOf schema
One more valid sample:
{
  "type": "object",
  "properties": {
    "home": {
      "type": [
        "object",
        "null"
      ],
      "properties": {
        "street": {
          "type": "string"
        }
      }
    }
  }
}

Thanks for the report.  I've never seen schema type as an array before.  I'll need to make sure it's valid when I get back to a computer this weekend.
Does the array imply an anyOf or oneOf schema?


Design: Landing page designs before signup

[ ] Design X - @vyydesigns @avronr

[ ] View 1: Routes
[ ] View 2: Discover [Hotels, destinations, flora & fauna etc.]
[ ] View 3: Plan

[ ] View 1: Routes
[ ] View 2: Discover [Hotels, destinations, flora & fauna etc.]
[ ] View 3: Plan


gh-862 - Refactoring logging to not string concat

Codecov Report

Merging #867 into develop will increase coverage by 0.95%.
The diff coverage is 20.93%.

@@             Coverage Diff              @@
##             develop    #867      +/-   ##
============================================
+ Coverage      62.74%   63.7%   +0.95%
- Complexity       828     880      +52
============================================
  Files            440     440
  Lines          12654   12795     +141
  Branches        1476    1509      +33
============================================
+ Hits            7940    8151     +211
+ Misses          4130    4048      -82
- Partials         584     596      +12

Impacted Files
Coverage Δ
Complexity Δ

.../accumulostore/retriever/AccumuloSetRetriever.java
64.7% <0%> (ø)
0 <0> (ø)
:arrow_down:

...accumulostore/retriever/AccumuloItemRetriever.java
64.93% <0%> (ø)
0 <0> (ø)
:arrow_down:

...java/uk/gov/gchq/gaffer/hbasestore/HBaseStore.java
59.45% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ain/java/uk/gov/gchq/gaffer/data/element/Edge.java
86.07% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../export/resultcache/GafferResultCacheExporter.java
76.25% <0%> (ø)
0 <0> (ø)
:arrow_down:

...n/java/uk/gov/gchq/gaffer/data/element/Entity.java
87.5% <0%> (ø)
0 <0> (ø)
:arrow_down:

...tion/mapper/AbstractAddElementsFromHdfsMapper.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...k/gov/gchq/gaffer/accumulostore/AccumuloStore.java
66.05% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ava/uk/gov/gchq/gaffer/graph/hook/Log4jLogger.java
60% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ed/operation/jcs/cache/NamedOperationJCSCache.java
52.83% <0%> (ø)
13 <0> (ø)
:arrow_down:

... and 21 more

Continue to review full report at Codecov.

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


Improve the visibility of the format label in thumbnail view

Hi, thanks for the nice work!
I would like to improve the appearance of the format label with this PR, in the latest version of elFinder, it look like this:

And I improved a bit to make it looks like the following:

Alternatively, I think we can also hide that label since windows 10 does have such a label.

Thanks for contributing!


New Feature: Choose which objects to have attachments converted

Not all objects support files, like EmailMessage and EmailTemplate. The conversion will fail when trying to creating a File shared to those kinds of records.
This feature allows the admin to configure which objects they want to be included in the conversion process rather than all Attachments that exist in the org.

Also see https://github.com/DouglasCAyers/sfdc-convert-attachments-to-chatter-files/issues/40

Also see #47


[HUDI-3272] If mode==ignore && tableExists, do not execute write logic and sync hive

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
If mode==ignore && tableExists, do not execute write logic and sync hive
Brief change log

If mode==ignore && tableExists, do not execute write logic and sync hive

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@manojpec : Can you review this patch.

@codope Hello, I have added test cases. Can you have a look?


Mu crashing

I'm getting errors with liv, it seems like mu crashes when it starts "housekeeping" with particular emails.
This is the error message:
Sep 27 10:48:29 proqqul liv[1568395]: 10:48:29.063 [info] Start housekeeping...
Sep 27 10:48:29 proqqul liv[1568453]: ** (mu:1568453): CRITICAL **: 10:48:29.827: failed to write output '(:docid 355 :subject "Re: Generators/Relations of Generalized Algebraic Theories" :message-id<EMAIL_ADDRESS>:path "/home/liv/Maildir/cur/1606775279.M253014P28199.proqqul,S=13292,W=13543:2,RST" :maildir "/" :priority normal :from (("Eigil Rischel" . "ayegill@gmail.com")) :to (("Owen Lynch" . "root@owenlynch.org")) :references<EMAIL_ADDRESS>:in-reply-to<EMAIL_ADDRESS>:date (24517 29153 0) :size 13292 :flags (replied seen trashed) :thread (:path "307" :level 0 :date "1606904386" :date-tstamp (24519 27202 0) :root t :has-child t :thread-subject t))'
Sep 27 10:48:29 proqqul .mu-wrapped[1568453]: failed to write output '(:docid 355 :subject "Re: Generators/Relations of Generalized Algebraic Theories" :message-id<EMAIL_ADDRESS>:path "/home/liv/Maildir/cur/1606775279.M253014P28199.proqqul,S=13292,W=13543:2,RST" :maildir "/" :priority normal :from (("Eigil Rischel" . "ayegill@gmail.com")) :to (("Owen Lynch" . "root@owenlynch.org")) :references<EMAIL_ADDRESS>:in-reply-to<EMAIL_ADDRESS>:date (24517 29153 0) :size 13292 :flags (replied seen trashed) :thread (:path "307" :level 0 :date "1606904386" :date-tstamp (24519 27202 0) :root t :has-child t :thread-subject t))'

If possible, can you include more error messages, I want to identify the line of the crash. You can hide the mail content.

I see what is going on. I guess you are using latest mu (1.6+), and I am still using the 1.4 branch. 1.6 apparently introduces some incompatibilities. Well I guess now is a good time for me to upgrade as well because why not?
Let me try to smooth it out. I'll get back to you tomorrow.

If you want to try you can check out the derek branch for both mc and liv. It should work with mu  latest. I want to do more testing before push to yhe main branch

Updates for both mc and liv now merged to main, minimum requirement of mu raised to 1.6
Closing

Thank you so much!!


Crate doesn't seem to build on x86_64-unknown-linux-musl

Build failed, waiting for other jobs to finish...
error: failed to run custom build command for `libz-sys v1.0.5`
Process didn't exit successfully: `/path/to/crate/target/debug/build/libz-sys-b246b9f3989628d7/build-script-build` (exit code: 101)
--- stdout
OPT_LEVEL = Some("0")
PROFILE = Some("debug")
TARGET = Some("x86_64-unknown-linux-musl")
debug=true opt-level=0
HOST = Some("x86_64-unknown-linux-gnu")
TARGET = Some("x86_64-unknown-linux-musl")
TARGET = Some("x86_64-unknown-linux-musl")
HOST = Some("x86_64-unknown-linux-gnu")
CC_x86_64-unknown-linux-musl = None
CC_x86_64_unknown_linux_musl = None
TARGET_CC = None
CC = None
HOST = Some("x86_64-unknown-linux-gnu")
TARGET = Some("x86_64-unknown-linux-musl")
HOST = Some("x86_64-unknown-linux-gnu")
CFLAGS_x86_64-unknown-linux-musl = None
CFLAGS_x86_64_unknown_linux_musl = None
TARGET_CFLAGS = None
CFLAGS = None
running: "./configure"
Compiler error reporting is too harsh for ./configure (perhaps remove -Werror).
** ./configure aborting.

--- stderr
thread '<main>' panicked at 'failed to run successfully: exit code: 1', /path/to/home/.cargo/registry/src/github.com-1ecc6299db9ec823/libz-sys-1.0.5/build.rs:126
note: Run with `RUST_BACKTRACE=1` for a backtrace.

I'm sorry, musl wasn't installed.


Make PeopleCard and CabinetMember clickable

Make these two components clickable the whole card, not just title text

PeopleCard
CabinetMember

I can help with this tonight.


masking problem when output length is different from input

Hi, dear all,
I have two inputs. One is with length of 50, the other one is with the length of 50. I will concatenate them along the time axis. The output length should be same as the input 1 (50). When I use keras and its extension Seq2seq, I define the input length is the summation (50+50) and output is 50.
However, when I use masking layer, it has the error:
ValueError: GpuElemwise. Input dimension mis-match. Input 1 (indices start at 0) has shape[1] == 100, but the output's size on that axis is 50.
If I remove the masking layer, everything is OK. Could anyone tell if the masking will see the output and detect the length of the output? Should I make the input and output exactly same long?
Thank you.

Hello,
I don't think I understand your question. But it seems you are referring to a problem with https://github.com/farizrahman4u/seq2seq. (project which imho, in its current state, is not faithful to Graves Seq2Seq : it is attempting to solve a much harder problem, and had (last time I checked) several blunders). I recommend not using it and sticking with the tensorflow Seq2Seq implementation, or rolling your own Seq2Seq implementation in Keras which is a good week-end project.
"If your issue is an implementation question, please ask your question on StackOverflow or join the Keras Slack channel and ask there instead of filing a GitHub issue."


💡 [Adding Resources] -

[x ] I am from GSSoC23.
[x ] I would like to be assigned this task.

fill [x] to check the boxes.

Section
Description

Concise Description
I want to add a new section of Cybersecurity

Solution Proposed
I want to add some resources to it also

Additional Context
Resources include- Books, Videos

Code of Conduct
By submitting this issue, you agree to follow our Code of Conduct.

[x ] I follow the Contributing Guidelines of this project.
[ x] I have checked the current issues for duplicate problems.
[ x] I have read the readme of this repository.

@kartikkc @Astrodevil @bhartik021 Please assign me this task under GSSOC'23 . Thank You

@sangdeepganvir assigned


GH-37072: [C++] GetArrayView: allow non-nullable fields to be null if parent is null

Rationale for this change
The layout constraint checks in GetArrayView are too strict when they don't allow  non-nullable STRUCT fields to be null since STRUCT fields can legally have any value when the parent Array describes the entry as not valid.
Reproduction script
import pyarrow as pa

struct_type = pa.struct([pa.field("x", pa.int32(), nullable=False)])
nulls = pa.nulls(5, struct_type)

# The following is an error:
nulls.cast(struct_type)

What changes are included in this PR?
Allow the execution of GetArrayView with nulls in non-nullable fields iff the parent Array is 100% null. To cover all cases, the checks would have to be done value by value, but that would be expensive. This PR covers the cases where STRUCT arrays are created by MakeArrayOfNull. Kernels producing STRUCTs with a few nulls might be wiser to set validity bits to 1 on non-nullable fields even when the parent is null.
Are these changes tested?
By existing tests and manual reproduction script.

Closes: #37072

I'm honestly not sure this is the right solution. If a field is not nullable, then ideally it shouldn't have any nulls even if it is shadowed by an all-nulls parent array.
Other implementations may also not like receiving such data over IPC or the C Data Interface.
Also, in any case, this should come with unit tests!

If a field is not nullable, then ideally it shouldn't have any nulls even if it is shadowed by an all-nulls parent array.

But what value should a field of a struct have when the entire struct is a logical null? Isn't that completely undefined and a waste of time to be populated?

It's undefined but it needs to be populated anyway :-) (we can't leave memory uninitialized as it can be serialized through IPC, for example)
MakeArrayOfNull already assigns a buffer full of zeros to all array buffers.

But what you are asking for here requires a validity buffer set to 0xff on the children and data buffers set to 0 or some type-specific notion of default value.

Right, it would add a bit of complexity, hopefully not too much. Most types would probably be ok with 0xff for all buffer data. Perhaps (list/binary/dense union) offsets and run-ends would better be zero, though?

Other implementations may also not like receiving such data over IPC or the C Data Interface.

Practically I think implementations are not great about using the nullability flag, instead relying mostly on the physical presence of nulls, but this is a chicken-and-egg issue.

If a field is not nullable, then ideally it shouldn't have any nulls even if it is shadowed by an all-nulls parent array.

This sounds reasonable, but we should clarify this more generally? It sounds like we aren't precise enough with that field nullability means for the physical data.

This sounds reasonable, but we should clarify this more generally? It sounds like we aren't precise enough with that field nullability means for the physical data.

Hmm, it seems like this could be discussed on the mailing-list. Perhaps we don't want to impose undue constraints here.

Right, it would add a bit of complexity, hopefully not too much. Most types would probably be ok with 0xff for all buffer data. Perhaps (list/binary/dense union) offsets and run-ends would better be zero, though?

If we were to got that route we would have to define the "default value" for each type similar to how protobuf 3 is allowed to omit data and default values are assumed.

Boolean - false
Number - 0.0
List - empty list (offsets would have to be 0 and not 0xffffffff)
Unions - the default value of the first type

This sounds reasonable, but we should clarify this more generally? It sounds like we aren't precise enough with that field nullability means for the physical data.

Hmm, it seems like this could be discussed on the mailing-list. Perhaps we don't want to impose undue constraints here.

I vote in favor of not imposing constraints. This is why I don't think the way to go here is "fixing" MakeArrayOfNull but relaxing the constraint checking that's failing on per-spec valid data.

We don't need to officially define "default values", just make sure we don't produce invalid data (such as non-monotonic offsets). This is exactly the idea behind ArrayBuilder::AppendEmptyValue:
https://github.com/apache/arrow/blob/d330d1b36c87ea43bfd16e1bc43a4ff9876abb4b/cpp/src/arrow/array/builder_base.h#L156-L161
But this discussion can also regard other implementations, I think it would be useful to discuss on the ML.

My only insight here is that nullability in nested types and the need of fast paths are causing non-trivial performance degradation in parquet serde. So if restricting some edge cases helps here, likely it’ll help in other code paths as well.

But this discussion can also regard other implementations, I think it would be useful to discuss on the ML.

The discussion is needed if we decide to put the definition of empty value (a better terminology than "default value" I used above) on the spec. My current position is that validation should be relaxed and not assume other implementations are producing non-null fields inside null structs.
For instance, I believe that a compute kernel, in the official implementation or other implementations, should be allowed to skip producing empty values inside of null structs. This can be a very meaningful improvement for structs with a large number of fields.

If we follow your position, other implementations will be exposed to such data. So they will be affected anyway.

All I'm saying is that we be liberal in data that we receive (relaxed checks). We can still be strict in MakeArrayOfNull but we shouldn't rely on everyone producing non-null values in struct fields.
Starting a whole specification refinement discussion takes much more time than I thought would be required from me to fix this issue. If the relaxed checks is not an acceptable solution, I can close the PR and move on.

My only insight here is that nullability in nested types and the need of fast paths are causing non-trivial performance degradation in parquet serde. So if restricting some edge cases helps here, likely it’ll help in other code paths as well.

@alippai on the Arrow->Parquet or on the Parquet->Arrow path?

This was list specific and for the parquet->arrow path: https://github.com/apache/arrow/issues/34510
I’ve read this issue again and as I understand both of the solutions would serialize into exactly the same parquet? Not sure which version the parsing will yield

Closing this in favor of #38252


Set hasFragileUserData to yes in AndroidManifest.xml

Describe your suggested feature
This will allow user to optionally retain all data in Android/data/ after uninstall.
TachiyomiAZ have this enabled:

Other details
https://github.com/az4521/TachiyomiAZ/blob/95e5651ad99378c5c24c0a23d8711106b50e80b6/app/src/main/AndroidManifest.xml#L31
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an official extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to version 0.15.3.
[X] I will fill out all of the requested information in this form.

Why? I see 0 benefit and people now potentially have orphaned data they can't remove

people now potentially have orphaned data they can't remove

They can, either by file manager on older android versions, or with file manager with shizuku support, or by installing and removing again.

I see 0 benefit

This folder can probably be big if you have many manga, and if you for example only have mobile data which have limited traffic you've probably want to retain this so mihon will not redownload this.

Very highly unlikely people will uninstall the app just to reinstall it again in couple of minutes

If they don't plan to use it anymore after uninstalling, why they will select to keep app data?


Tests the actual output of util::var_dump.

Tests the actual output of util::var_dump, using PHPUnit's output testing functionality.

Coverage increased (+0.25%) to 76.73% when pulling 8d49484c3ba7e69a2c12fba07962f3a4c5b4dfe2 on phpexpertsinc:master into a4ff07ebb66fbcb91d4151c21b9197a40da72d9f on brandonwamboldt:master.

Coverage increased (+0.5%) to 76.98% when pulling d1b7bb9b3bb2dfc185519ffe33766e8a34e9eb49 on phpexpertsinc:master into a4ff07ebb66fbcb91d4151c21b9197a40da72d9f on brandonwamboldt:master.

Yeah I screwed this PR up pretty good. Redoing ;p


It installs fail to laravel 5.3.24

composer require dingo/api:1.0.x@dev

Result:
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - dingo/api 1.0.x-dev requires dingo/blueprint 0.2.* -> satisfiable by dingo/blueprint[v0.2.0].
    - dingo/api v1.0.0-beta4 requires dingo/blueprint 0.2.* -> satisfiable by dingo/blueprint[v0.2.0].
    - dingo/api v1.0.0-beta5 requires dingo/blueprint 0.2.* -> satisfiable by dingo/blueprint[v0.2.0].
    - dingo/api v1.0.0-beta6 requires dingo/blueprint 0.2.* -> satisfiable by dingo/blueprint[v0.2.0].
    - Conclusion: don't install dingo/blueprint v0.2.0
    - Conclusion: don't install dingo/api v1.0.0-beta3
    - Installation request for doctrine/annotations == <IP_ADDRESS> -> satisfiable by doctrine/annotations[v1.3.0].
    - Conclusion: remove laravel/framework v5.3.24
    - Installation request for phpdocumentor/reflection-docblock == <IP_ADDRESS> -> satisfiable by phpdocumentor/reflection-docblock[3.1.1].
    - Conclusion: don't install laravel/framework v5.3.24
    - dingo/api v1.0.0-beta1 requires illuminate/routing 5.1.* -> satisfiable by illuminate/routing[v5.1.1, v5.1.13, v5.1.16, v5.1.2, v5.1.20, v5.1.22, v5.1.25, v5.1.28, v5.1.30, v5.1.31, v5.1.41, v5.1.6, v5.1.8].
    - dingo/api v1.0.0-beta2 requires illuminate/routing 5.1.* -> satisfiable by illuminate/routing[v5.1.1, v5.1.13, v5.1.16, v5.1.2, v5.1.20, v5.1.22, v5.1.25, v5.1.28, v5.1.30, v5.1.31, v5.1.41, v5.1.6, v5.1.8].
    - don't install illuminate/routing v5.1.1|don't install laravel/framework v5.3.24
    - don't install illuminate/routing v5.1.13|don't install laravel/framework v5.3.24
    - don't install illuminate/routing v5.1.16|don't install laravel/framework v5.3.24
    - don't install illuminate/routing v5.1.2|don't install laravel/framework v5.3.24
    - don't install illuminate/routing v5.1.20|don't install laravel/framework v5.3.24
    - don't install illuminate/routing v5.1.22|don't install laravel/framework v5.3.24
    - don't install illuminate/routing v5.1.25|don't install laravel/framework v5.3.24
    - don't install illuminate/routing v5.1.28|don't install laravel/framework v5.3.24
    - don't install illuminate/routing v5.1.30|don't install laravel/framework v5.3.24
    - don't install illuminate/routing v5.1.31|don't install laravel/framework v5.3.24
    - don't install illuminate/routing v5.1.41|don't install laravel/framework v5.3.24
    - don't install illuminate/routing v5.1.6|don't install laravel/framework v5.3.24
    - don't install illuminate/routing v5.1.8|don't install laravel/framework v5.3.24
    - Installation request for laravel/framework == <IP_ADDRESS> -> satisfiable by laravel/framework[v5.3.24].
    - Installation request for dingo/api 1.0.x@dev -> satisfiable by dingo/api[1.0.x-dev, v1.0.0-beta1, v1.0.0-beta2, v1.0.0-beta3, v1.0.0-beta4, v1.0.0-beta5, v1.0.0-beta6].

Installation failed, reverting ./composer.json to its original content.

Same here with Laravel 5.3.26 please Fix it ASAP

I've created a PR for this issue.
In the meantime, you guys could use the same workaround I'm using right now - create your own dingo/api fork, change the composer.json like I did here https://github.com/dingo/api/pull/1273/files
And require your own fork in your project's composer.json - https://getcomposer.org/doc/05-repositories.md#loading-a-package-from-a-vcs-repository
(Note: don't use my fork as I'll delete it once the changes are merged and I'll be able to continue using official dingo/api repo)
This should be good enough for development, I wouldn't feel good about putting this into production though.

This should be resolved by https://github.com/dingo/api/commit/1f022fa2155907d7f8ccc92944579d18a19ecb2a, the dependency isn't needed in the api package, and the blueprint package has been updated accordingly.

1f022fa
Using version ^0.10.0 for dingo/api
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for dingo/api ^0.10.0 -> satisfiable by dingo/api[v0.10.0].
    - Conclusion: remove laravel/framework v5.3.28
    - Conclusion: don't install laravel/framework v5.3.28
    - dingo/api v0.10.0 requires illuminate/routing 5.1.* -> satisfiable by illuminate/routing[v5.1.1, v5.1.13, v5.1.16, v5.1.2, v5.1.20, v5.1.22, v5.1.25, v5.1.28, v5.1.30, v5.1.31, v5.1.41, v5.1.6, v5.1.8].
    - don't install illuminate/routing v5.1.1|don't install laravel/framework v5.3.28
    - don't install illuminate/routing v5.1.13|don't install laravel/framework v5.3.28
    - don't install illuminate/routing v5.1.16|don't install laravel/framework v5.3.28
    - don't install illuminate/routing v5.1.2|don't install laravel/framework v5.3.28
    - don't install illuminate/routing v5.1.20|don't install laravel/framework v5.3.28
    - don't install illuminate/routing v5.1.22|don't install laravel/framework v5.3.28
    - don't install illuminate/routing v5.1.25|don't install laravel/framework v5.3.28
    - don't install illuminate/routing v5.1.28|don't install laravel/framework v5.3.28
    - don't install illuminate/routing v5.1.30|don't install laravel/framework v5.3.28
    - don't install illuminate/routing v5.1.31|don't install laravel/framework v5.3.28
    - don't install illuminate/routing v5.1.41|don't install laravel/framework v5.3.28
    - don't install illuminate/routing v5.1.6|don't install laravel/framework v5.3.28
    - don't install illuminate/routing v5.1.8|don't install laravel/framework v5.3.28
    - Installation request for laravel/framework == <IP_ADDRESS> -> satisfiable by laravel/framework[v5.3.28].

Installation failed, reverting ./composer.json to its original content.

Just to confirm - after update everything works on my side as well!


Ajax calls skipping .then method JavaScript SharePoint Online

Category

[x] Bug

Hello All,
We are getting information from a 3rd party API.
Below is the code that communicates with the API
function getDistfromSAP(root)
{

var deferred = $.Deferred();

   $.ajax({
   async:false,
   url: root,
   dataType: 'jsonp',
   method: 'GET'

    }).then(function(data) {

    deferred.resolve(data);

   }).fail(function(jqXHR, textStatus, err) {

        console.log("Failed to get JSON");
        deferred.reject(err);
    });

    return deferred.promise();

}

Calling the function
var root = "Web API URL"

getDistfromSAP(root).then(function(response){

console.log(response);

var resp = JSON.parse(response.GetQuoteDetailsResult);

},function(error){

console.log(error);

});

When the function is called for the first time ,.then() part is skipped and the code proceeds further with empty value in the response array "resp" in our example
It then enters the .then() part of the code and required data is received from the API. However the challenge is that it breaks the code if .then() is not executed in the first call resulting in empty values being returned.
Read a lot of articles online pertaining to callbacks , fetch() , However they did not work for our scenario.
Would appreciate if anyone could let me know how this needs to be handled.
Please let me know if you need any other details

Have you verified this works outside of SPO & it isn't an issue explicitly with SPO?
Also, it would help to get a lot more context, like seeing the data you're submitting, the URL, the actual request from your NETWORK tab in the browser to see if it's being sent, the response, etc.
In general, need a lot more context and a repro of the issue to better assist.


Unable to set server http port in config file?

Am I missing a way to set the server http port in the config yaml file? (Equivalent to the --http-port flag that can be passed to ord server?)
I tried adding an http_port entry (that I just made up!) to the yaml file, just as a shot in the dark, but that's clearly not a valid entry:
 $ ord --config signet.yaml server
error: failed to deserialize config file `signet.yaml`
because: unknown field `http_port`, expected one of `bitcoin_data_dir`, `bitcoin_rpc_limit`,
`bitcoin_rpc_password`, `bitcoin_rpc_url`, `bitcoin_rpc_username`, `chain`,
`commit_interval`, `config`, `config_dir`, `cookie_file`, `data_dir`, `first_inscription_height`,
`height_limit`, `hidden`, `index`, `index_cache_size`, `index_runes`, `index_sats`,
`index_spent_sats`, `index_transactions`, `integration_test`, `no_index_inscriptions`,
`server_password`, `server_url`, `server_username`

 $ cat /home/so7ow/.local/share/ord/signet.yaml
http_port: 81
index_runes: true
chain: signet

 $ ord --version
ord 0.18.5

 $ cat /etc/os-release
PRETTY_NAME="Ubuntu 22.04.4 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.4 LTS (Jammy Jellyfish)"

If there is no way to do it currently, could we get that added?

#3796
If you want to add some other server setting to the yaml file you can take that PR as a template to add the others as well :)

I'd love to but I'm GitHub and Rust illiterate. You don't want me breaking your app.


User not able to view Plan

Describe the bug
I have a user that has all necessary permissions- project admin and Allow 'View/Edit Analytics Permissions' in Azure DevOps project. He holds Stakeholder license and gets an error while trying to view a Plan i created. If i give him 'Basic' license he is able to view the Plan.
Can you please help me
Browser
Choose a browser
[ ] Chrome
Version
[ ] Azure DevOps/VSTS
Screenshots
Access to data from Analytics OData endpoint is not available for all users.

One of my colleagues has the same issue. The full error message is;
"VS403527: Access to data from the Analytics OData endpoint is not available for all users. Either you have not been authenticated, or you do not have a paid license or subscription for this account."
They have a 'Basic' license assigned. I created the plan using my Visual Studio Enterprise licensed account.
This is in Azure DevOps Service with version <IP_ADDRESS>6795967 of the Delivery Plans extension.


More lobby tips

Purpose
Add more lobby tips that will trigger on certain lobby titles. In the future these tips can be removed if we can incorporate it as part of Chobby UI.

Test Steps
In a new lobby boss yourself then rename the lobby to contain noob or rotato (or both) e.g.
$rename Noob Rotato

Or
$rename Noob
$rename Rotato

Nice ideas. I'll come back to this later and incorporate them.

Updated based on @geekingfrog feedback

Not sure how relevant this PR still is considering the addition of votable Teiserver commands and pending changes to boss mode?

I guess it could be adjusted to use ! instead of $. Setting to draft

Have updated to use ! commands instead of $
          "Useful commands for Noob lobbies",
          @splitter,
          "To restrict this lobby to players who are new, use either:",
          "!maxchevlevel <chevlevel>",
          "!maxratinglevel <rating>",
          "",
          "To ensure new players are distributed evenly across teams:",
          "!balancealgorithm split_one_chevs",
          ""
        ]


Implement basic CAN functionality

some info here: https://discourse.odriverobotics.com/t/can-interface-available-for-testing/1448

Has anyone tested the communication rate achievable with CAN vs serial?

Have you tested the ros package communication rate with the odrive while using CAN?

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: Wetmelon:x: jutavaYou have signed the CLA already but the status is still pending? Let us recheck it.

For the record: merged as part of #410


Rearchitect RFC

Here would be my rough draft detailing the architecture.
@eriknelson @rikai Please feel free to go ham on things that need to be added as well as your thoughts/suggestions on the architecture.

@thefirstofthe300 thanks! Will take a look later today.

@thefirstofthe300 looks like #95 supersedes this? Should this get closed?

Yeah. This should probably be closed. Still learning how to use Git. :stuck_out_tongue:


XIVY-8503 remove static page image

we now produce it on the fly with a build job

@ivy-rew merge?


Pull Request Creator can merge own PR

It´s currently possible that the creator can merge the PR when a review approved the changes.
That should not be possible

In the last meeting it was decided that this implementation would be too complicated and complex.
Therefore, the current situation will be kept.


Add 1.28 section for BR Releases

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/retest

/hold

/retest

/retest


[d3d11] Vindictus: Wrong Aspect Ratio in Multiple Screens Setup

When playing Vindictus in DirecX 11 mode in a setup with more than one screen, if one of those is connected through DisplayPort and the second one through HDMI then the aspect ratio of only the DisplayPort screen will be used, regardless of which screen is being used as the main one.
In my case, my main screen is 16:9 connected through HDMI, with another side monitor, also 16:9, used vertically and connected through DisplayPort. This causes Vindictus to default its resolution to windowed 1024x768 when started with dxvk, and to only show vertical resolutions when choosing a custom one, as shown here:

It's worth noting that this doesn't happen when playing in DirectX 9 mode.
Software information

Vindictus on Windows 11

System information

GPU: NVIDIA GeForce RTX 4090
Driver: 552.12
Wine/Proton Version: NA (Windows)
DXVK version: v2.3.1

Apitrace file(s)

https://mega.nz/file/jYdkkBiT#69EOZzST9pNPPucEy9aZkg0ZmKi2k0c7Jgeee1AAFo8

Log files
No Proton or Wine log, being on Windows, however I can provide the dxvk log:

Vindictus_x64_d3d11.log

We currently don't really support rotated displays, in part because I have no idea how DXGI reports display modes for them and if apps have to be aware of the rotation.
I'm also not sure what the expectation here is though, the game is not aware of multi-monitor setups and only queries display 0, and I don't know if changing the "main" display at an OS level has any effect at all on the way EnumDisplayMonitors / EnumDisplaySettings report things. If not, we can't really do all that much.


Error with composite key

Tried to load test database with DomDatabaseWrapper.
Database was created with KeePass v. 2.34 for Windows and secured with composite key: password+key file.
Code looks like:
  Credentials credentials = new KdbxCreds(password.getBytes(), keyInputStream);
  DomDatabaseWrapper.load(credentials, databaseInputStream);

Meantime, if I remove from database composite key (i.e. only password based encryption) - I'm able to read database correctly using:
  Credentials credentials = new KdbxCreds(password.getBytes());
  DomDatabaseWrapper.load(credentials, databaseInputStream);

That's odd. There's a specific test for this use case in DomDatabaseWrapperTest, does the test work for you?
@Test
public void inspectKeyfileDatabase() throws IOException {
    InputStream keyFileInputStream = getClass().getClassLoader().getResourceAsStream("KeyFileDatabase.key");
    Credentials credentials = new KdbxCreds("123".getBytes(), keyFileInputStream);
    InputStream inputStream = getClass().getClassLoader().getResourceAsStream("KeyFileDatabase.kdbx");
    DomDatabaseWrapper database = new DomDatabaseWrapper(new KdbxStreamFormat(), credentials, inputStream);

    database.save(new StreamFormat.None(), new Credentials.None(), System.out);
}

I agree - that's odd. Test unit with test database passes, but with my database fails with trace:
java.lang.IllegalStateException: Inconsistent stream start bytes. This usually means the credentials were wromng.
                                                                                at org.linguafranca.pwdb.kdbx.stream_3_1.KdbxSerializer.checkStartBytes(KdbxSerializer.java:129)
                                                                                at org.linguafranca.pwdb.kdbx.stream_3_1.KdbxSerializer.createUnencryptedInputStream(KdbxSerializer.java:88)
                                                                                at org.linguafranca.pwdb.kdbx.stream_3_1.KdbxStreamFormat.load(KdbxStreamFormat.java:37)
                                                                                at org.linguafranca.pwdb.kdbx.dom.DomDatabaseWrapper.<init>(DomDatabaseWrapper.java:53)
                                                                                at org.linguafranca.pwdb.kdbx.dom.DomDatabaseWrapper.load(DomDatabaseWrapper.java:58)

Here's test database and test database key file
Password is 123456

That database opens with password 123456, and does not require the key file. So when I try to open it with the key file as well as the password it fails, as expected. Am I misunderstanding something?

Hmmm... in my case with native KeePass v. 2.34 (Win x64) I can open database only if I apply both password and key file. But suddenly I realized that I could open it with KeePassJava2 w/o key file (only using password).
So from this point of view looks like correct, but a bit strange... Can't understand whose bug/feature it is? native KeePass?

When I run KeePass 2.34 in a windows emulation it will not open with both password and creds so that is odd. Anyway, glad all sorted for now.  Plus the good news is I got to see that there is a typo in "This usually means the credentials were wromng." :-)


Fix import statement generation on typescript files

On typescript files import statements are generated as follows
import Something from './something'

import statements should like this
import { Something } from './something'

issue reported incorrected. current import form is ok.


Style weight vs. Content weight

Is bumping up the style weight by a factor two the same thing as halving the content weight, or do the absolute values have a meaning, too?

It should be roughly equivalent; the main difference is that halving the content weight will result in lower absolute values for the loss function as opposed to doubling the style weight, and this can sometimes affect the termination logic for the L-BFGS optimizer. I'm pretty sure that they should be equivalent when using Adam.

Cool, thank you!

I found that doubling style has better effect than halving content, though.

What are those weights expressed as? I couldn't quite tell from the defaults of "5e0".and "1e2" if they were scientific notation of decimals (except 5e0 is just 1 so figure not), hex, or radial basis function values. If the latter, I don't have any intuition around what changing these will do. Further examples value would be very welcome! Thanks!

They are just scientific notation, 5e0 is 5.
On Dec 6, 2015 5:17 AM, "Gruff Davies"<EMAIL_ADDRESS>wrote:

What are those weights expressed as? I couldn't quite tell from the
defaults of "5e0".and "1e2" if they were scientific notation of decimals
(except 5e0 is just 1 so figure not), hex, or radial basis function values.
If the latter, I don't have any intuition around what changing these will
do. Further examples value would be very welcome! Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/jcjohnson/neural-style/issues/87#issuecomment-162298059
.

what content weight and style weight you guys often use?


Keycloak Operator 7.0.1 (initial)

Signed-off-by: Sebastian Laskawiec<EMAIL_ADDRESS>Tracking ticket: INTLY-3231
This Pull Request introduces an initial version of Keycloak Operator to the Operatorhub. Keycloak is an OpenSource project for Identity Management.
This is a complementary part to https://github.com/operator-framework/community-operators/pull/889
New Submissions

[x] Has you operator nested directory structure?
[ ] Have you selected the Project Community Operator Submissions in your PR on the right-hand menu bar?
[x] Are you familiar with our contribution guidelines?
[x] Have you packaged and deployed your Operator for Operator Framework?

[x] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Submit operators to both upstream-community-operators and community-operators at once
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Thank you for integrating this @J0zi !

Hey @J0zi ! I've just got a fresh OpenShift 4.2 installation from RHPDS and it seems Keycloak is not there. Could you please give me a hint why is that? It should be there, shouldn't it?


Update data-transform.md

"sort" 的 config 的形式化定义的“type”有误，更正为"sort"

Thanks for your contribution! Could you please help correct the same typo in the English documentation?

Yes, the same typo in the English documentation have been corrected just now, please check the 'pr'.


bf16 & tf32，can they be used together

In the hyperparameter, I saw the bf16 and tf32 are both set to true. However I don't they can be used together?

I have the same question

I was searching for the internet to find the answer. and then I found this thread.
is it possible though? I think the latter arg probably overwrites the former.
if someone could clarity this, I would be grateful.

any new update on this?

By setting bf16, the model weights are saved in bf16, but gradients (computed in half-precision but converted to full-precision for the update) and optimizations (optimizer states) are still done in full 32-bit precision. By additionally setting tf32, you speed up that very calculations at the expense of precision, but since ultimately those weights are saved as bf16 anyway, it probably has no effect on accuracy.


http_index_files not working in http2 mode

Please answer these questions before submitting your issue.

What did you do? If possible, provide a simple script for reproducing the error.

Using HTTP/1.x the index.html will be served, with http2 the request is not working
`<?php
Swoole\Coroutine::set([
'trace_flags' => SWOOLE_TRACE_HTTP2,
'log_level' => 0,
]);
$key_dir = DIR . '/../ssl/';
$http = new swoole_http_server("<IP_ADDRESS>", 9501, SWOOLE_BASE, SWOOLE_SOCK_TCP | SWOOLE_SSL);
$http->set([
'open_http2_protocol' => 1,
'enable_static_handler' => TRUE,
'document_root' => DIR,
'enable_coroutine' => true,
'http_index_files' => ['index.html', 'index.txt'],
'ssl_cert_file' => $key_dir . '/example.com+4.pem',
'ssl_key_file' => $key_dir . '/example.com+4-key.pem',
]);
$http->on('request', function (swoole_http_request $request, swoole_http_response $response) {
$response->header('Access-Control-Allow-Origin', '*');
$response->header('Content-Type', 'text/event-stream');
$response->header('Cache-Control', 'no-cache');
$response->header('X-Accel-Buffering', 'no');
while(1) {
    $response->write("event: ping\n\n");
    $r = $response->write("data: {\"time\": \"" . date(DATE_ISO8601) . "\"}\n\n");
    if(!$r) return;
    co::sleep(1);
}

$response->end(__DIR__);

});
$http->start();
`

What did you expect to see?

index.html should be served using http2

What did you see instead?

Canceled Request

What version of OpenSwoole are you using (show your php --ri openswoole)?

4.10.0

What is your machine environment used (show your uname -a & php -v & gcc -v) ?

21.2.0 Darwin Kernel Version 21.2.0: Sun Nov 28 20:28:41 PST 2021; root:xnu-8019.61.5~1/RELEASE_ARM64_T6000 arm64
PHP 8.1.2 (cli) (built: Jan 21 2022 04:34:05) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.1.2, Copyright (c) Zend Technologies
with Zend OPcache v8.1.2, Copyright (c), by Zend Technologies
Apple clang version 13.0.0 (clang-13<IP_ADDRESS>)
Target: arm64-apple-darwin21.2.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

Fixed and supported at https://github.com/openswoole/swoole-src/pull/137