Probably need to reconsider allowed operations

It seems that it is necessary to re-consider allowed operations for the measurement objects.
Consider this example:
>>> from unitsnet_py import Length
>>> length = Length.from_meters(10) * Length.from_centimeters(30)
>>> length.base_value
3.0
>>> length.centimeters
300.0
>>> length.meters
3.0

We actually multiplied 10 meters by 30 centimeters to obtain 3 squared meters.
However the object length is treated as 3 meters, not squared meters.
Maybe it will be a good idea to ban multiply, divide, mod and pow operations for now.
IMHO, it is necessary to create specialized classes for these operations:
class Multiply(Measure):
    def __init__(self, measure_1: Measure, measure_2: Measure):
        self.measure_1 = measure_1
        self.measure_2 = measure_2

    def convert(self, multiply_unit: MultiplyUnits) -> float:
        return (
            self.measure_1.convert(multiply_unit.unit_1)
            * self.measure_2.convert(multiply_unit.unit_2)
        )

    @property
    def base_value(self) -> float:
        return self.measure_1.base_value * self.measure_2.base_value

class MultiplyUnits:
    def __init__(self, unit_1: Unit, unit_2: Unit):
        self.unit_1 = unit_1
        self.unit_2 = unit_2

Hi @ivanovmg
Thank you for your ideas :)
Currently, yes, this library does not contains any aspect of measurement and complex conversions.
Regrading your suggestion, I think it also could be related to #10 and be solved by UnitsNet conversion formulas, so you will be able to define a formula that multiply of Length unit will become an Area unit.


feature: exec preview

Idea: improve feedback loop by executing code on lint and show the changes in the world immediately

nice idea, really powerful, will implement this step by step


barra de progresso não aparece sempre [BUG]

Describe the bug
As vezes a barra de progresso de treinamento não aparece.
To Reproduce
j.setup -> db_type = SelfSupervised ; utilizando-se o mycelia_bases

Possible scenarios:

Database is too small and thus the training part happens in a shorter period of time than frequency_seconds. Unlikely, but it's a possibility.
Throughput issue (on client or server side, or both). Depending on how busy a pod is, the status response time might degrade. Loosening the threholds for new pods may help improve performance (with a cost trade-off). On client side, network problems (unstable connection, slow connection, etc) can make status responses get sluggish as well.

Closing this issue since both scenarios are not directly related to the SDK per se. Feel free, however, to reopen it if this behavior gets recurrent.


Modal data-te-modal-dismiss not working if no data-te-toggle="modal" on page

Describe the bug
After Upgrading from 1.0.0beta2 to beta3 or rc1 it is no longer possible to dismiss a modal with data-te-modal-dismiss.
As the Modals are shown programatically there is no data-te-toggle button. In the code the enableDismissTrigger function within the modal-component init has been removed. The overall init function just uses the data-te-toggle selector.
As soon as a button with data-te-toggle="modal" exists the dismiss trigger works again.
To Reproduce
follow the examples in documentation. This works fine.
Remove the button with data-te-toggle="modal" and the data-te-target="#id".
Reload the page.
Open the page programatically. Modal will be shown. But closing based on the dismiss-trigger not working.
Expected behavior
A modal should be closable from inside with data-te-modal-dismiss either if it is toggle by button or opened programatically.
Actual behavior
data-te-modal-dismiss only working if not using programatically modal opening.
Show your code
The following code is used (ES) to open the modal:
window.openModal = (id) => {
const el = document.getElementById(id)
if (typeof el != 'undefined' && el !== null) {
let modal = Modal.getInstance(el)
if (modal === null) {
modal = new Modal(el, {})
}
modal.show()
}
}
Desktop (please complete the following information):

OS: Windows
Browser Chrome One
Version 101.0.4843.43

Make sure it is indeed an issue/bug report
Without the data-te-toggle-button only two click-listeners are registered to the dismiss-button. As soon as the toggle button is on the page 6 listeners are registered including the dismiss.te.modal uidEvent.

Hi, we'll try to fix this for the 1.0.0 release (11.09). It was working without any issues before right?

Hello juujisai. That's correct with beta2 it works fine. Starting with beta3 it stopped working.


Add mailto: to email addresses

Android only knows how to handle emails in a URLSpan with a mailto: in front of the email address. This adds that for email addresses matched by Patterns.EMAIL_ADDRESS.

@laszlourszuly Thanks for letting me know. Opened a PR importing Patterns.

Now that's what I call a quick response :-) Thanks!


Ops Resolver

what is the purpose of using these ops resolvers in code? what is your intention, Mr.Chris?

@Anugaradhawa The MicroMutableOpsResolver allows for a subset of the operations needed to be loaded, reducing memory usage. See Tensorflow's Get started with microcontrollers.


RN WebView goBack and goForvard not working when inside 1-st slide

But no problem when WebView inside 2-nd ....
RN 4.4 - Android device 4.2

I just changed <Swiper index={to my index}>

Same happening here~ @oky1 does the workaround works?


As an end-user, I want to be able to write to modbus registers

Support for writing to modbus registers from MQTT events.

mhemeryck, have You found a solution for bidirectional modbus<>mqtt gateway?

@pki791 not really; the best setup I currently have is to use modbus between the unipi modules and an MQTT interface which just uses unipi's evok library. See https://blog.mhemeryck.com/2021-11-16/modbusbackup (and related posts) for an overview.


Adding DefaultHttpHeaders to itself creates infinite loop

Example:
public void test() {
    HttpHeaders headers = new DefaultHttpHeaders();
    headers.add("foo", "bar");
    headers.add(headers);
    // This will never end
    headers.forEach(entry -> {});
}

...why would you even do that

@willisblackburn thanks for the report. Will add a guard against this

@willisblackburn - I'm assuming you are using Netty 4.0?


Option to Change font weight

We're using a specific font for our event, but I can't change the font to bold or semi-bold. Is there anything that can be done to fix that?

Hey @SeanFromBoP, I don't really have the best solution, but what I was able to do was uninstall the weights of the fonts that I didn't want. For Example, we used Proxima Nova, and I wanted semi-bold. I uninstalled all of the weights lower than semi-bold and then it worked. After the event, I just reinstalled the weights. I hope that helps.


feat: add possibility to listen to react model open/request close

Summary:
At Reanimated we currently facing an issue that we're not able to listing to keyboard height changes in modal. This is due to the fact that android dialogs are attached next to the view tree, so we need to start observing insets on them separately, thus we need to listen to dialog open. To my knowledge, currently, there is no way to detect/listen when modal (dialog) is opened in a way that we can have access to its window (which we need to observe insets). The only hack that I found is listening to all events and act on “showModal”, which seems like hacky approach. In this PR I added observer that would be notified by ReactModalHostManager and would allow to attach listeners in other places. With this change in other packages we could get an instance and listen for those events. That should allow us to fix https://github.com/software-mansion/react-native-reanimated/pull/5916 and may help with https://github.com/kirillzyusko/react-native-keyboard-controller/issues/369.
Changelog:
[ANDROID] [ADDED] - Allow to listen to RN Modal events in native code.
Test Plan:
Tested on clean project from npx @react-native-community/cli@next init MyApp --version next --skip-install, which installs RN: 0.75.0-rc.5.
Changes in RN Core as in the PR.

Part added to MainApplication.kt onCreate:
ModalEventManager.addModalListener(object: ModalCallback {
      override fun onModalOpen(dialog: DialogInterface?, viewTag: Int) {
        println("Open")
        (dialog as Dialog).setOnDismissListener({
          println("Close")
        })
      }

      override fun onModalRequestClose(dialog: DialogInterface?, viewTag: Int) {
        println("onRequestClose")
      }
    })
  }

App.ts
import * as React from 'react';
import {
  TextInput,
  View,
  Button,
  Modal,
  SafeAreaView,
  StyleSheet,
} from 'react-native';

function MyModal({ visible, hide }) {
  const [isVisible, setIsVisible] = React.useState(false);

  return (
    <Modal transparent visible={visible} onDismiss={hide} onRequestClose={hide} onLayout={(layout) => console.log('layout', layout)}>
      <View style={styles.modalContainer}>
        <View style={styles.modalContent}>
          <Button onPress={hide} title="Close Modal" />
          <TextInput style={styles.textInput} placeholder="Inside a modal" />
          <Button onPress={() => setIsVisible(visible => !visible)} title="Toggle Component With Animated Keyboard" />
        </View>
      </View>
    </Modal>
  );
}

export default function EmptyExample() {
  const [visible, setVisible] = React.useState(false);

  return (
    <>
      <SafeAreaView style={styles.container}>
        <Button onPress={() => setVisible(true)} title="Open Modal" />
        <TextInput style={styles.textInput} placeholder="Outside a modal" />
      </SafeAreaView>
      <MyModal visible={visible} hide={() => setVisible(false)} />
    </>
  );
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    justifyContent: 'space-between',
    backgroundColor: '#ecf0f1',
    paddingHorizontal: 8,
    paddingBottom: 50,
    paddingTop: 8,
  },
  modalContainer: {
    ...StyleSheet.absoluteFillObject,
    flex: 1,
    justifyContent: 'center',
    backgroundColor: 'rgba(0,0,0,0.2)',
  },
  modalContent: {
    justifyContent: 'space-between',
    backgroundColor: 'white',
    borderRadius: 24,
    padding: 24,
    minHeight: 256,
  },
  textInput: {
    paddingHorizontal: 8,
    paddingVertical: 4,
    borderRadius: 8,
    backgroundColor: 'silver',
    fontSize: 18,
    textAlign: 'center',
  },
});

Small bump on this. That could be really useful for react-native-edge-to-edge.

@maciekstosio Is this change still needed? If so, I would like to look into how we can move forward with this.


Unable to remove service from proxy configuration

It seems that removing a service from proxy configuration is not possible. There is no explicit API method for removing a service, and configuring a service with empty path merely changes the service frontend to match any request as follows:
frontend http-env-echo-fe
        bind *:80
        bind *:443
        option http-server-close
        acl url_http-env-echo path_beg
        use_backend http-env-echo-be if url_http-env-echo

backend http-env-echo-be

The option to send a request that will remove a service will be added in the next release in a day or two.

The release contains the option to remove a service from the proxy config. Please check https://github.com/vfarcic/docker-flow-proxy#remove for details.
Can you please test it and let me know if it matches your scenario?

Unable to test it in my environment right now. The description of the functionality looks good, and the code looks reasonable.


System.InvalidCastException: Unable to cast object of type 'System.String'

Steps to reproduce
public class person_profile
{
public int approval_id {get;set;}
public string city_name {get;set;}

public virtual lib_approval lib_approval {get;set;} => has int as PK
public virtual lib_city lib_city {get;set;} =>has string as PK
}

Problem occurs when accessing lib_city.city_name..
 public class person_profileDTO
    {
        public string lib_approval_name { get; set; }
        public string lib_city_city_name { get; set; }

        public static System.Linq.Expressions.Expression<Func<person_profile, person_profileDTO>> SELECT =
            x => new person_profileDTO
            {
                lib_approval_name = x.lib_approval.name,
                lib_city_city_name = x.lib_city.city_name,

            };

    }
}

using this method still throws same error:
   db.person_profile  .Include(x => x.lib_city).Select(
                    x => new
                    {
                        person_profile_id = x.person_profile_id,

                        lib_city_city_name = x.lib_city.city_name,

                    }
                    );,

The issue
When retrieving values of related tables for values with string keys, exception is thrown. For integer values there is no problem.
If you are seeing an exception, include the full exceptions details (message and stack trace).
Exception message:  An unhandled exception has occurred while executing the request
System.InvalidCastException: Unable to cast object of type 'System.String' to type 'DeskApp.DataLayer.person_profile'.
   at lambda_method(Closure , ValueBuffer )
   at System.Linq.Enumerable.<GroupJoinIterator>d__40`4.MoveNext()
   at System.Linq.Enumerable.<SelectManyIterator>d__22`3.MoveNext()
   at System.Linq.Enumerable.<JoinIterator>d__37`4.MoveNext()
Stack trace:

Further technical details
EF Core version: 1.0.0-rc2-final
Operating system: Windows 10
Visual Studio version: 2015
Other details about my project setup:

EF Team Triage: This issue is lacking enough information for us to be able to effectively triage it. In particular, it is missing the following information requested in the new issue template. Can you please provide this information?

Steps to reproduce

Ideally include a complete code listing that we can run to reproduce the issue.
Alternatively, you can provide a project/solution that we can run.
BTW we're not just doing this to be mean 😄... we get a lot traffic on this project and it takes time to attempt to reproduce an issue based on fragments of information. In addition, our attempt is often unsuccessful as the exact conditions required to hit the issue are often not explicitly included in the code provided. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we ask that folks give us a self contained way to reproduce an issue.
For a guide on submitting good bug reports, read_ Painless Bug Tracking.

You should also upgrade to the RTM release, rather than RC2.

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.


QR upload not working on iOS 14.4.2

Don't know if anyone else is having this issue but the buttons to either select a file from the device or open the camera are not working, no matter the browser used, on my iPhone X with iOS 14.4.2.
It's been like that for at least a week now.

Duplicate of #35


fix: Adapter has high qps to apiserver

Provide a description of what has been changed
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[N/A] Tests have been added
[N/A] A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
[N/A] A PR is opened to update the documentation on (repo) (if applicable)
[x] Changelog has been updated and is aligned with our changelog requirements

Fixes https://github.com/kedacore/keda/issues/2914

Looking good, could you please also pass the same clien to the handler on line 106? So we can get rid of the kubeclient completely?
Also, have you been able to test this setup to see if it really helped with the issue?

could you please also pass the same client to the handler on line 106? So we can get rid of the kubeclient completely?
will do later

have you been able to test this setup to see if it really helped with the issue?
I test the image locally, only the first request will hit the apiserver, after there are only list from informer cache(guaranteed by list-watch)

Cool :)
Then please also fix DCO and update the Changelog, that's a great improvement!

@zroubalik Can I just skip the test case in this PR? Since this one only mitigate the burden to Apiserver, will not affect the performance/logic in Keda side

@zroubalik Can I just skip the test case in this PR? Since this one only mitigate the burden to Apiserver, will not affect the performance/logic in Keda side

@bamboo12366 yeah, no need for test in this case.

/run-e2e

/run-e2e

/run-e2e

I'm nos sure how, but it seems that these changes are breaking e2e tests. They have failed 3 times in a row when in main branch they are working. Maybe e2e tests are based on this wrong approach, I'll take a look deeper asap

/run-e2e

/run-e2e mysql*

/run-e2e argo*

Hey @bamboo12366
We plan to do a release during next week. I have solved the problems with e2e tests (they were in other places, not in your changes), could you rebase your branch and update the changelog? 🙏
I'll trigger e2e test again and once they pass, we can merge it to include this improvement in next release

@JorTurFer I update CHANLOG and resolve conflicts accordingly, could you please help me review again to see if everything fit?

/run-e2e

I'm not sure about what commit breaks the DCO check @bamboo12366 but at least 1 commit is not signed.
Could you squash your branch for just pushing your commits (signed)?

sorry I am not that familiar with git. I already merge the rebase and merge the main into current branch, I don't know how to squash commit now, could you advise more? @JorTurFer

sorry I am not that familiar with git. I already merge the rebase and merge the main into current branch, I don't know how to squash commit now, could you advise more? @JorTurFer

Sure, you can do this:
git checkout yourBranch
git reset $(git merge-base main $(git branch --show-current))
git add -A
git commit -sm "one commit on yourBranch"

Squash the branch is not mandatory, but it's the easiest way to ensure all commits are signed (because only 1 needs to be signed xD)

sorry I am not that familiar with git. I already merge the rebase and merge the main into current branch, I don't know how to squash commit now, could you advise more? @JorTurFer

Sure, you can do this:
git checkout yourBranch
git reset $(git merge-base main $(git branch --show-current))
git add -A
git commit -sm "one commit on yourBranch"

Squash the branch is not mandatory, but it's the easiest way to ensure all commits are signed (because only 1 needs to be signed xD)

Thanks A Lot!!


Master

Hi @gplarsen, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.azure.com.

TTYL, AZPRBOT;

Hey Paul,
All internal FTEs should submit their PRs in the private version of this repo. Let's sync up on doing that.
Thanks,
Dave

@gplarsen, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, AZPRBOT;


Dont want to display jitsi page after call hangup

This Issue tracker is only for reporting bugs and tracking code related issues.
Before posting, please make sure you check community.jitsi.org to see if the same or similar bugs have already been discussed. General questions, installation help, and feature requests can also be posted to community.jitsi.org.
Description
When i hang up any call , it displays default blue colored page (Close/Welcome page) with " Start New Meeting " Option.
Setting up different config variables but still no luck.
Current behavior

Expected Behavior
I Dont want that page to display.
Possible Solution

Steps to reproduce

Environment details
*Application is in Angular 6
 config : {                      startWithAudioMuted : false,                    startWithVideoMuted : false ,                    enableWelcomePage: true,                    enableClosePage: true,                    url:""                   },       isUserInteractionEnabled : true,       interfaceConfigOverwrite: {         DEFAULT_REMOTE_DISPLAY_NAME :this.fromdisplay,         DEFAULT_LOCAL_DISPLAY_NAME : this.ologgedInUser.entity.display,         DISPLAY_WELCOME_PAGE_CONTENT: false,         GENERATE_ROOMNAMES_ON_WELCOME_PAGE: false,         APP_NAME: 'Test Meet',         NATIVE_APP_NAME: 'Test Meet',         SHOW_WATERMARK_FOR_GUESTS: false,         SHOW_JITSI_WATERMARK: false,         CLOSE_PAGE_GUEST_HINT: false,         RECENT_LIST_ENABLED: false       },

Please for problems and question use the community forum before opening an issue.
You need to enable in the config: enableClosePage: true,
https://github.com/jitsi/jitsi-meet/blob/master/config.js#L237
This will redirect you to a static close page.

There is no such option, it is in the code https://github.com/jitsi/jitsi-meet/blob/b7b43e8d9c582910aa043a522fbe1b19500c2b44/conference.js#L226
It is always static/close.html


Add support for HTTPS calls via pd during testnet join

The pd testnet join flow should support HTTPS endpoints. Right now, it assumes all connections are HTTP, but we should update the option handling to infer the proper scheme based on a url::Url or similar and make the relevant requests.
We already have a long-running HTTPS endpoints for the testnet deployments: https://{,g}rpc.testnet{,-preview}.penumbra.zone Traffic on 443/TCP to those FQDNs will talk to pd or tm as necessary. Let's use those to build the new option handling, and default client requests to an HTTPS endpoint in future releases.

The option handling update will be done in #2222; otherwise, I think this should be de-prioritized relative to HTTPS calls in pcli (#1887) and TLS support in pd (#1406), since it's the least important thing to have encryption + authentication for (any data an attacker interferes with will result in not joining the chain correctly anyways)

HTTPS support is implemented for pd testnet join:
❯ pd testnet join https://rpc.testnet-preview.penumbra.zone
2023-04-11T19:09:39.139050Z  INFO fetching genesis genesis_url=Url { scheme: "https", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("rpc.testnet-preview.penumbra.zone")), port: None, path: "/genesis", query: None, fragment: None }
2023-04-11T19:09:39.509745Z  INFO fetched genesis
2023-04-11T19:09:39.576903Z  INFO fetched node id node_id=node::Id(c29498fc7921dac2e8f5ecfa1bf681518a8df0f6)
2023-04-11T19:09:39.722752Z  INFO peers=[Tcp { peer_id: Some(node::Id(c29498fc7921dac2e8f5ecfa1bf681518a8df0f6)), host: "rpc.testnet-preview.penumbra.zone", port: 26656 }, Tcp { peer_id: Some(node::Id(97c8dfd33eedcd29432e90bb08d303573408f3c9)), host: "<IP_ADDRESS>", port: 26656 }, Tcp { peer_id: Some(node::Id(ff7efd725cb1808dcbe6822910fbcea6d2eaa589)), host: "<IP_ADDRESS>", port: 26656 }, Tcp { peer_id: Some(node::Id(93abbe37154ff80e1a67106df6310d234d9fefa6)), host: "<IP_ADDRESS>", port: 26656 }]
2023-04-11T19:09:39.726215Z  INFO writing genesis genesis_file_path=/home/conor/.penumbra/testnet_data/node0/tendermint/config/genesis.json
2023-04-11T19:09:39.734678Z  INFO writing tendermint config.toml config_file_path=/home/conor/.penumbra/testnet_data/node0/tendermint/config/config.toml
2023-04-11T19:09:39.734768Z  INFO writing node key file node_key_file_path=/home/conor/.penumbra/testnet_data/node0/tendermint/config/node_key.json
2023-04-11T19:09:39.734856Z  INFO writing validator private key priv_validator_key_file_path=/home/conor/.penumbra/testnet_data/node0/tendermint/config/priv_validator_key.json
2023-04-11T19:09:39.734945Z  INFO writing validator state priv_validator_state_file_path=/home/conor/.penumbra/testnet_data/node0/tendermint/data/priv_validator_state.json
2023-04-11T19:09:39.734985Z  INFO writing validator custody file validator_spend_key_file_path=/home/conor/.penumbra/testnet_data/node0/tendermint/config/validator_custody.json

The changes in #2222 were sufficient to resolve. I'm leaving this ticket open, however, because #2341 has the potential to break the cluster side, so I'll want to retest all the rpc endpoints after making changes. Once we're happy with the proxy config, we should update the default value for pd testnet join to be an HTTPS URL, at which point we can close this ticket.

Currently, the seed node support we added in #1847 doesn't really work: the TendermintAddress that gets interpolated in the seeds = field in the tm config retains the 26657 port from the RPC URL, when it should use the 26656 for the p2p service. Fortunately, because we have 4 nodes in our deployment, they gossip and the addresses are distributed to new joiners well, so we this isn't a huge problem. The problem is more obvious when we update the RPC URL to HTTPS, because then we can no longer reuse the host portion of the address, let alone the port. Instead, we should use the Tendermint RPC to inspect the node's definition of its own listener address:
❯ curl -s https://rpc.testnet-preview.penumbra.zone/net_info | jq .result.listeners
[
  "Listener(@<IP_ADDRESS>:26656)"
]

and interpolate that into the generated config. While doing so, we can break down some of the taller functions in testnet join into smaller pieces, making the code easier to work with.


app macro doesn't work very well with rust-analyzer

i've been writing a decently large embedded system to control a dsp and display a vumeter, however i was not particularly happy with how the rust language server integrates with vscode, so i switched to rust analyzer.
the change has been good, and things in general work a lot better, however there's one annoying problem: if i start typing anything at all inside of the app macro the entire screen turns red as everything following that attribute macro is considered a compiler error.

i've not written a proc macro myself, but here the rust analyzer devs talk about how this problem can be fixed by the macro developer in at least some cases:
https://github.com/rust-analyzer/rust-analyzer/issues/11014
right now i'm considering externing everything out of the macro just so i can avoid this little problem. which both seems fun, but also extremely pointless.
i thought it worth bringing up in case you were not aware of this and it can be fixed. it seems as tho fixing this in rust analyzer may not be possible in the short term as they have a lot of considerations to make.

Yea,  I saw this problem as well, I thought my rust-analyzer was acting up, but it seems to be a more fundamental problem. In general rust analyzer is now better understanding the proc macros, however when there are (syntactic) errors inside the proc macro RTIC + rust analyzer as of today is not able to spot the span of the error (and hence the span will be the entire RTIC module).
I had a quick look at the linked discussion, and it seems that the initial idea was to put the burden on the proc macro implementer (to parse the inner in detail) not sure we want to do that. Right now we just re-emit any tokens that are unknown to RTIC, which  I think is the sensible thing to do atm. In #11014 it was also discussed other potential solutions requiring work on the actual rust-analyzer. So I'm not sure if we want/can do anything about this for the time being.
I agree however that it is less than optimal that the whole module becomes squiggled while typing, its a lot of noise.
/Per

This seems odd, I do not get this problem with rust-analyzer. It correctly highlights the macro error as we would in a build:

The same with code errors:

I'm using this config to reduce the annoyance of this issue:
"rust-analyzer.procMacro.enable": true,
"rust-analyzer.experimental.procAttrMacros": false,
"rust-analyzer.diagnostics.disabled": [
    "macro-error"
],

We should definitely shrink the error span here, that can be done without fixing the whole ecosystem

Closing as the rust analyzer support works quite good now and with the support of having tasks outside the macro eliminates the issue.


allow csi-rbdplugin deploy to master

Describe what this PR does
allow  csi-rbdplugin deploy to master
Provide some context for the reviewer
csi-rbdplugi not deploy on master of kubernetes
we should install ceph plugin on all node, like cni plugin does
Is there anything that requires special attention

@zgfh Thanks for the PR. no need to start daemon set on master by default.
the newly started pod is of no use if the environment is not allowing the application pods deployment on the master node.
you can comment out the added code and also add a comment  if users  want to run the plugin on the master node to  uncomment the toleration part

@zgfh  Can you please fix markdown error ?

@humblec sorry ,updated and rebased

/retest


sched: Implement getpid and gettid correctly

Summary
getpid should return the main thread id
gettid should return the current thread id
See the discussion here: https://github.com/apache/incubator-nuttx/issues/2499
Impact
All caller of getpid
Testing

This line in pthread.h needs to change too:
#define pthread_self() ((pthread_t)getpid())
Please review all uses of getpid() in the OS and make sure that the calls still make sense.  In most cases, getpid() will need to be replaces with gettid() or pthread_self().

This line in pthread.h needs to change too:
#define pthread_self() ((pthread_t)getpid())
Please review all uses of getpid() in the OS and make sure that the calls still make sense. In most cases, getpid() will need to be replaces with gettid() or pthread_self().

Yes, I will review all place and ensure all usage is correct.

There is some naming that should be made consistent too.

Those are POSIX APIs and are required to use a process ID as would be returned by getpid().
gettid() is a non-standard Linux API that really should not be used in portable software.  The standard, portable interfaces are getpid() and pthread_self().

what's the plan for pid_t taking APIs like kill and wait?

Those are POSIX APIs and are required to use a process ID as would be returned by getpid().
gettid() is a non-standard Linux API that really should not be used in portable software.  The standard, portable interfaces are getpid() and pthread_self().

Perhaps we could put the changes on a branch.  I could help out some with the conversion.
However, we should wait until after the next 10.1.x release to give the changes as much time as possible to mature before 10.2.
There should be no real use of the (corrected) getpid within the OS other than at the OS API.  There are several functions that take the pid as a parameter (and usually if the pid is zero, then it should use getpid() to get the pid of the caller).
There is also a lot of naming of variables and structure fields that would need to be corrected.

Perhaps we could put the changes on a branch.  I could help out some with the conversion.
However, we should wait until after the next 10.1.x release to give the changes as much time as possible to mature before 10.2.
There should be no real use of the (corrected) getpid within the OS other than at the OS API.  There are several functions that take the pid as a parameter (and usually if the pid is zero, then it should use getpid() to get the pid of the caller).
There is also a lot of naming of variables and structure fields that would need to be corrected.

Perhaps we could put the changes on a branch. I could help out some with the conversion.

Thanks for helping.

However, we should wait until after the next 10.1.x release to give the changes as much time as possible to mature before 10.2.

Sure, I will create a branch once the new release is out.

There should be no real use of the (corrected) getpid within the OS other than at the OS API. There are several functions that take the pid as a parameter (and usually if the pid is zero, then it should use getpid() to get the pid of the caller).
There is also a lot of naming of variables and structure fields that would need to be corrected.

@patacongo I remove getpid change from PR, so we can merge this one first.


[QUESTION] Component State

@xinjie-zhang, is the only way to define local state for each component using the inline x-data directive? I have a few components with more complex state, example: nested objects, arrays, etc. Is it possible to define this inside the
</template>

<template x-component="customer" x-data="$api && $api.data()" x-init="$api && $api.init()">

</template>

Is there anyway to achieve this?

I got this working by removing the x-data directive altogether:
<template x-component="customer" x-init="$api && $api.init()">

</template>

Is this the preferred way?

I'm working on headless UI (https://github.com/vimeshjs/vimesh-headless), which will implement Tailwind headless UI (https://headlessui.com/) with Alpine.js. There are already some components like listbox, switch, tabs etc. They are my preferred way to develop components.
Let me explain $api and x-data. $api is inherited from x-data. The difference is $api is not directly visible to the child elements. x-data is accessible to all children elements. So if you want properties and methods visible to its descendant, use x-data. If you want to make something only available to current element, use $api. Lets say x-data is "public", $api is "private". In some cases, you want to access invisible $api of its ancestor, you could use $api.$of(type of ancestor). There are many examples in vimesh headless ui. For initialization, x-data use init(). In $api, you could use onMounted(), and onUnmounted() for destroy.
For your case, I would propose :
<template x-component="customer">

</template>

Great! Thanks for the detailed explanation, @xinjie-zhang!


Nix module broke: error: attribute 'system' missing

Hyprland Version
a0cf89029263decba29b0f4216b38576f4418335
Bug or Regression?
Regression
This regression seems to  have been introduced unintentionally by @fufexan in the commits the past days : https://github.com/hyprwm/Hyprland/commits/main/nix
Description
         at /nix/store/fdy2b9n83ksarxhddniyb6wxzh9628g6-source/lib/attrsets.nix:820:11:

          819|         || pred here (elemAt values 1) (head values) then
          820|           head values
             |           ^
          821|         else

       … while evaluating the attribute 'value'

         at /nix/store/fdy2b9n83ksarxhddniyb6wxzh9628g6-source/lib/modules.nix:768:9:

          767|     in warnDeprecation opt //
          768|       { value = builtins.addErrorContext "while evaluating the option `${showOption loc}':" value;
             |         ^
          769|         inherit (res.defsFinal') highestPrio;

       (stack trace truncated; use '--show-trace' to show the full trace)

       error: attribute 'system' missing

       at /nix/store/091n9gc82ygfvjylfl08zidk3msls8lp-source/nix/module.nix:50:62:

           49|
           50|     portalPackage = mkPackageOptionMD inputs.xdph.packages.${pkgs.system} "xdg-desktop-portal-hyprland" {};
             |                                                              ^
           51|
       Did you mean one of mystem, systemc, systemd, syntex or systemfd?

How to reproduce
My flake uses Hyprland: https://github.com/Ramblurr/nixcfg/

https://github.com/Ramblurr/nixcfg/blob/main/flake.nix#L4
https://github.com/Ramblurr/nixcfg/blob/main/profiles/gui-hyprland.nix#L13

Let me know if I can be of more help.
Crash reports, logs, images, videos
No response

Cause was config issue:
nixpkgs.config.allowAliases = false;

Think this can be closed

@eclairevoyant I should've used pkgs.hostPlatform.system. I'll fix it.

Fixed in https://github.com/hyprwm/Hyprland/commit/78fa8adadc146a7efeebf63438c1140662484fba.


Fixed redis url parsing for cluster coordinator

Codecov Report

Merging #210 (3d525cb) into master (3721a13) will decrease coverage by 0.24%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files

@@            Coverage Diff             @@
##           master     #210      +/-   ##
==========================================
- Coverage   33.84%   33.61%   -0.24%
==========================================
  Files          41       41
  Lines        1312     1321       +9
==========================================
  Hits          444      444
- Misses        846      855       +9
  Partials       22       22

Flag
Coverage Δ

unittests
33.61% <0.00%> (-0.24%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

config/services.go
33.07% <0.00%> (-1.22%)
:arrow_down:

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 3721a13...3d525cb. Read the comment docs.


litterally doesn't work (not trying to be rude)

the gun orb shows up but does nothing
i cant equip the rats
"unbalanced mode" is not in the settings

I'm new to ultrakill and modding so that could be an issue


Fix Arch install scripts

This pr fixes a few small problems in the arch install scripts. Our nuc12s are working fine, but this will make things easier when nucs need to be flashed in future.
Changes:

Add a pacman.conf file to ignore keychain when installing packages (we do this in docker since #896 so I don't think there's a good reason not to do it here)
Remove request for missing file docker/home/nubots/.ssh/id_rsa.pub (removed in #1102 )
Add user libraries to /etc/ld.so.conf to allow systemconfiguration to run
Add a blank zsh config file to prevent the prompt when restarting after post-install script

NUbots/NUbook#281, I think mirror list date is relevant here too


add unknown types to the Type enum

one thing this allows is the ability to see more type a parameters is in the $scope.commands.signature | flatten.

closing in favor of #545


gofmt: Anonymous field could be placed in the 2nd column of a struct

What version of Go are you using (go version)?
go version go1.7 windows/amd64
What operating system and processor architecture are you using (go env)?
set GOARCH=amd64
set GOBIN=
set GOEXE=.exe
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=C:\PROJECTS\Go
set GORACE=
set GOROOT=C:\Go
set GOTOOLDIR=C:\Go\pkg\tool\windows_amd64
set CC=gcc
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0
set CXX=g++
set CGO_ENABLED=1

What did you do?

What did you expect to see?

What did you see instead?

I suspect it's being placed in the first column because while it's called an anonymous field, it's actually named rune (as well as being type rune).

I'm going to re-open this. I think the suggestion is quite reasonable and syntactically it makes sense (but @quentinmit has a point, too). Not saying that I will do anything about it yet, but I want to think about this a bit.

Oh, cool. Thanks. There's a whole variety of ways to look at it, so I can understand opinion on where such a thing should go is going to differ for each person. I see the columns as 'name' and 'type', and so it makes logical sense to me that a name-less field should be lined up that way. The other side of the coin is that a nameless type uses the type as the name; ergo -- it's a name and belongs in the first column.
Last but not least, you can write the type twice as a workaround:
type thing {
    rune    rune    // a character
``

@Kroc This is not field with a nameless type, it's a field w/o an explicit field name. That is, rune in this case is the type. The work-around that you describe may work here (embedding of a predefined type w/o methods), but generally this is not correct for any type as the semantics of the struct change.

Excuse my ignorance, I'm extremely new to Go still :S


[Unit of measuring] The deleted unit from 'Unit of measusing' page isn't deleted from the event card

Date Time
06-08-2021 12:00
Priority
High
Pre-requisites
Environment:
OS: [Windows 10]
Browser [Google Chrome, Версія 91.0.4472.77]
Version [ 20210607.1]
Pre-conditions

Go to https://eventsexpress-test.azurewebsites.net
Authorize as the admin
Move to the “Units of measuring” page
Select already existing 'Unit name'
At least 1 event was created

Steps to Reproduce
Steps to reproduce the behavior:

Move to the 'Home' page
Click on the 'View' button in this event and choose the 'Unit name' from the dropdown list below the 'Count' label  (e.g. 'kilometers')
Click on the 'Save' button
Authorize as the admin
Move to 'Unit of measuring' page
Click on the 'Delete' button near the unit '#kilometers'
Authorize as simply user , move to ' Home page' and click on 'View' button in this event
See the short name of the unit below 'Count' label

Expected result
The unit is fully deleted from the event card after it was deleted from the 'Unit of measuring' page.
Actual result
The unit is present in the event card after it was deleted from the 'Unit of measuring' page.
Screenshots

User story and test case links E.g.: "User story #347 Test case #637
Labels to be added "Bug", Priority: Medium, Severity: Major, Type Functional.

The deleted unit must not modify the already created inventory. The deleted unit should not appear in the list when editing inventory


Need to escape app name when used as a drill in honeycomb PU

I'm using the following honeycomb PU:
!PU(honeycomb):base=low;warn=20;crit=40;drill=/ui/user-sessions?filter=app:${app.name}&filter=browserFamily:$name&filter=errorType:Javascript%20error
If app.name has spaces in it, this breaks the drill. Can you make app.name URL friendly like it's done in other parts of the configurator swap?
Thanks,

The drill link might also include filters for session properties that have spaces, like "New York" and those are populated at run time. Can the drill link be "cleaned up" with %20 for the spaces prior to launch? That would solve both the app filter and session property filters...

Do you just need a new swap from Configurator that is uri encoded? If so, that’s an easy task. Originally I thought you were asking for something PowerUps based.

I think it will be in both places. The $name is evaluated at PU run time so that would need to be something that PU escapes. The other filters are built with swaps, so that would be done in the configurator. So both.

Just a configurator issue...


Introduce logic to accommodate in place upgrades.

What this PR does / why we need it:
This is still gated by validateManagement which prevents this logic from ever running.
Upcoming PRs will drop the gating validation and will complete the in place upgrade logic to run pods in the guest cluster.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/retest

/test e2e-aws

/lgtm

needs a rebase to include https://github.com/openshift/hypershift/pull/1212

/lgtm


Using Pytorch, Train a Simple ELM Model

Reference the following Paper.
Use PyTorch and train something super simple, like MNIST on this model

Editing first issue to reflect contribution towards presentation

Issue will be used to finish implementing the presentation template, and writing presentation notes


Low light enhancement

Hi
Could you include support for  some of the models included in this link?
https://github.com/caiyuanhao1998/MST-plus-plus
specially for higher PSNR? Restormer MPRNet MST-L MST++

We should already support Restormer and MPRNet.
About MST and MST++: These are meant to reconstruct for full spectrum from an RGB image, AFAIK. So given an RGB image, they will output an approximation of the light frequency spectrum for each pixel. I guess this could be used for Low-light enhancement, but not directly as is.
So I don't know which specific models you want us to support. @Euanggelos, could you please link them more directly?

We should already support Restormer and MPRNet.

I tried both today from spandrel Readme links, using ChaiNNer:

Restormer: No Low-light model; I did test 4 (3 worked, from 13), but did other things like: deblurring, denoising. I did find restormer.pth, did load, but finish with an error.
MPRNet: No Low-light model, I did test 2 models (from 3) and worked, as expected doing deblurring and denoise. I did find mprnet.pth, but doesn't load as a model.

Maybe both doesn't have Low-Light Enhancement models, and are in the Results on NTIRE 2022 HSI Dataset just to compare older models.

From NTIRE 2024 Challenge on Low Light Enhancement, seems that RetinexFormer and MST++ are nice aditions, because they ranked top-2.
From Readme, RetinexFormer is supported, but models from links says unsupported in ChaiNNer. Maybe ChaiNNer (0.24.1) doesn't have an updated spandrel that support this.

In summary, I didn't find any Low-Light models in ChaiNNer that works.

chainner-bin (Arch AUR): 0.24.1-1 (Last Release)
distro: Arch Linux x86_64
kernel: 6.6.22-1-lts
shell: bash 5.2.26
term: tmux
cpu: Intel i7-4790 (8) @ 3.600GHz
gpu: AMD ATI Radeon RX 470/480/570/570X/580/580X/590

Maybe we need to wait for next update (Spandrel and/or ChaiNNer)?
Is possible to compile Spandrel to get/test newer commits? (I don't know how, not a programmer)


Style/home page

Update landing page design

The footer is not present in the homepage as it is in the design. We already have the footer component.

The searchbar hero section doesn't entirely cover the full height of the screen

The scale of the CTA button looks off for me. I feel the text is too bold. Probably because we haven't added the design font from figma

The footer is not present in the homepage as it is in the design. We already have the footer component.

Ohhkk, I'll add it right away

The searchbar hero section doesn't entirely cover the full height of the screen. The 1st section should have the screen min-height
The scale of the CTA button looks off for me. I feel the text is too bold. Probably because we haven't added the design font from figma

The button does look big but that's the design specification from figma, another reason is he used 1920px screen width to design and that greatly affected the scale of elements and how they turned out

The searchbar hero section doesn't entirely cover the full height of the screen. The 1st section should have the screen min-height
The scale of the CTA button looks off for me. I feel the text is too bold. Probably because we haven't added the design font from figma

The button does look big but that's the design specification from figma, another reason is he used 1920px screen width to design and that greatly affected the scale of elements and how they turned out

I'll send the deployment link to papersych so he examines and compare with the design.

After merge: Some observations on the UI, @Extheoisah let me know if I should create issues for those:

Activating the search bar reveals filters that appear beyond the visible window, requiring scrolling for access.
The arrow for advancing to the next section remains visible even after displaying the filters.

-Initiating a search without input leads to an irregular state where the footer overlays the top, hiding other homepage elements. In this case the URL is  <domain>?search=, therefore even refreshing does not reset the page.

I'm looking it right away

@IgboPharaoh the second issue that I mentioned here hasn't been solved.

I just checked the latest deployment from emmanuel's last pr on staging and the issue was non-existent. Just to be clear, the second issue we are talking about is The arrow for advancing to the next section remains visible even after displaying the filters. right ?

Sorry, my mistake, I meant the third issue, apologies for no being clear

Initiating a search without input leads to an irregular state where the footer overlays the top, hiding other homepage elements. In this case the URL is ?search=, therefore even refreshing does not reset the page.

I posted 2 videos, so I kind of referring to the second video as second issue.

No worries. I set conditions where the footer is only displayed if there is
a homepage or a search query or search results returns a value. But I'll
still test it out in the latest deployment to fix this.
On Mon, 8 Apr 2024, 11:03 am Andreas, @.***> wrote:

I just checked the latest deployment
https://bitcoinsearch-l4wko3rxe-btc-knowledge-base.vercel.app/ from
emmanuel's last pr on staging and the issue was non-existent. Just to be
clear, the second issue we are talking about is The arrow for advancing
to the next section remains visible even after displaying the filters.
right ?
Sorry, my mistake, I meant the third issue, apologies for no being clear
Initiating a search without input leads to an irregular state where the
footer overlays the top, hiding other homepage elements. In this case the
URL is ?search=, therefore even refreshing does not reset the page.
I posted 2 videos, so I kind of referring to the second video as second
issue.
—
Reply to this email directly, view it on GitHub
https://github.com/bitcoinsearch/bitcoinsearch-app/pull/93#issuecomment-2042350178,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUYGRSMMZIKZMFTIQA2AGN3Y4JTQ3AVCNFSM6AAAAABFKPBJXSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBSGM2TAMJXHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

I couldn't replicate the 3rd issue, you can try this deployment link: latest-deployment. This is the format of the link I tried:  https://bitcoinsearch-l4wko3rxe-btc-knowledge-base.vercel.app/?search= and the footer was not present.
Perhaps you can provide more context on this if I am not following through correctly

I couldn't replicate the 3rd issue. you can try this deployment link: latest-deployment. This is the format of the link I tried: https://bitcoinsearch-l4wko3rxe-btc-knowledge-base.vercel.app/?search= and the footer was not present.
Perhaps you can provide more context on this if I am not following through correctly

You are right that right now the footer issue is solved. But, what's the point of allowing that empty search query? Empty search query takes the page into a kind of intermediary state, which seems not useful. Why not disable the button when search is empty in order to avoid that?

I'm happy to know the issue has been fixed.

I don't think it takes it into an intermediary state, it just returns an empty result. We can investigate the query code to see if it returns if the router query is empty

Try searching google, bing or duckduckgo with an  empty query string.


How to set a negative margin percent?

I want to set a negative value to margin by a percent of parent but i can't do this. Is there any work around?

no I don't think it's actually possible


Allow publishing to UDP while reading from TCP

Describe the feature
Hello,
Would it be possible to split the protocols parameter into two: one for publishing and another for reading?
I would like to publish to UDP and read from TCP.
While this works with the default config, it always attempts to read from UDP first before falling back to TCP:
publishing to path 'cam', 1 track with UDP
reading from path 'cam', 1 track with UDP
reading from path 'cam', 1 track with TCP

Setting protocols: tcp prevents publishing to UDP, but allows for default reading from TCP.
Setting protocols: udp allows publishing to UDP, but only allows reading from UDP.

Sorry but this is not technically possible for the moment; if i were you, i'd create two instances of rtsp-simple-server:

one for the publishers, with protocols: udp
one for the readers, with protocols: udp and configured to request streams to the publisher's instance by using source: rtsp://publisher-server-ip:/...


[BUG] Get Error When Using gen() with list_append=True in Guidance Server

The bug
File "/opt/homebrew/lib/python3.9/site-packages/guidance/models/_model.py", line 116, in encode
return self.serialize()
File "/opt/homebrew/lib/python3.9/site-packages/guidance/models/_model.py", line 119, in serialize
proto = self._to_proto()
File "/opt/homebrew/lib/python3.9/site-packages/guidance/models/_model.py", line 105, in _to_proto
return _serialization_pb2.EngineCallResponse(
File "/opt/homebrew/lib/python3.9/site-packages/google/protobuf/internal/python_message.py", line 531, in init
copy.update(field_value)
File<EMAIL_ADDRESS>line 941, in update
self[key] = other[key]
File "/opt/homebrew/lib/python3.9/site-packages/google/protobuf/internal/containers.py", line 431, in setitem
checked_value = self._value_checker.CheckValue(value)
File "/opt/homebrew/lib/python3.9/site-packages/google/protobuf/internal/type_checkers.py", line 211, in CheckValue
raise TypeError(message)
TypeError: [b'The '] has type <class 'list'>, but expected one of: (<class 'bytes'>, <class 'str'>)
To Reproduce
from guidance import select
from guidance import models, gen, with_temperature
llama2 = models.LlamaCpp("http://localhost:8392", api_key="SDFSDF")

query = "Who won the last Kentucky derby and by how much?"
lm = llama2 + f'''\
Q: {query}
Now I will choose to either SEARCH the web or RESPOND.
Choice: {select(["SEARCH", "RESPOND"], name="choice")}
'''
if lm["choice"] == "SEARCH":
    lm += "Here are 3 search queries:\n"
    for i in range(3):
        lm += f'''{i+1}. "{gen(stop='"', name="queries", temperature=1.0, list_append=True)}"\n'''

I came up a way to fix it:
Modify your code of relative functions in guidance.models._model.py.EngineCallResponse like this:
    def _to_proto(self):
        """Converts an EngineCallResponse object to its Protobuf representation.

        Returns:
            engine_response_pb2.EngineCallResponse: The Protobuf equivalent of this object.
        """
        groups = {}
        group_log_probs = {}

        log_probs_list_prefix = "__LOGS_PROBS_LIST_"

        for k, v in self.capture_groups.items():
            if not isinstance(v, list):
                groups[k] = v
                group_log_probs[k] = self.capture_group_log_probs[k]
                continue

            groups[k] = json.dumps([str(value, encoding="utf8") for value in v])

            for i, numeric_value in enumerate(self.capture_group_log_probs[k]):
                group_log_probs["{}_{}_{}".format(log_probs_list_prefix, k, i)] = numeric_value

        return _serialization_pb2.EngineCallResponse(
            new_bytes=self.new_bytes,
            is_generated=self.is_generated,
            new_bytes_prob=self.new_bytes_prob,
            capture_groups=groups,
            capture_group_log_probs=group_log_probs,
            new_token_count=self.new_token_count,
        )

    @staticmethod
    def deserialize(byte_data):
        proto = _serialization_pb2.EngineCallResponse()
        proto.ParseFromString(byte_data)

        groups = {}
        group_log_probs = {}

        log_probs_list_prefix = "__LOGS_PROBS_LIST_"

        for k, v in proto.capture_groups.items():
            groups[k] = [bytes(value, encoding="utf8") for value in json.loads(v)]
            if isinstance(groups[k], list):
                group_log_probs[k] = []
                for i in range(len(groups[k])):
                    key_name = "{}_{}_{}".format(log_probs_list_prefix, k, i)
                    group_log_probs[k].append(proto.capture_group_log_probs[key_name])
            else:
                group_log_probs = proto.capture_group_log_probs[k]

        return EngineCallResponse(
            new_bytes=proto.new_bytes,
            is_generated=proto.is_generated,
            new_bytes_prob=proto.new_bytes_prob,
            capture_groups=groups,
            capture_group_log_probs=group_log_probs,
            new_token_count=proto.new_token_count,
        )

@stonekim thanks for raising the issue and @Dobiichi-Origami thanks for the workaround!
@nking-1 it looks like the issue is that the protocol expects the values of the capture_groups and capture_groups_log_probs dicts to be bytes/strings and floats respectively, while we need to support the union of those and list-ified versions of that. Is this a corner of the codebase you're familiar with/could help out with?

I'm vaguely familiar with this code. I'll take a look today.

Quick update - I looked over this today and made some progress toward reproducing the issue, along with generating new protobuf definitions. I'd say I'm about halfway through understanding the fix for this.
The bug only seems to occur when running a guidance server. I didn't see any issues when running gen with list_append=True otherwise.

Thank you @nking-1 !

Thank you @nking-1 @hudson-ai  !
May I ask if there is an update to fix this issue? We urgently need this feature.

@stonekim see the newly linked PR :)


Update stable version in repo from 0.0.0 to 0.1.4

Think about adding version 0.1.4 to stable repository.
Version: stable=0.0.0 (0 days old) => latest=0.1.4 (799 days old)
Installs: stable=0 (0%), latest=272 (95.77%), total=284
Click to edit

Currently staying at Beta state


Error: ReferenceError: module is not defined

I am coding an application using angular2 and angularfire2.
My app directories as following:

All js file in public/lib is copied from correspond node_modules
In index.html, I wrote a config as below:
System.config({
    map: {
        firebase: '/js/lib/firebase-web.js',
        angularfire2: '/js/lib/angularfire2'
    },
    packages: {
        baseURL: '/js/',
        js: {
            format: 'register',
            defaultExtension: 'js'
        },
        angularfire2: {
            defaultExtension: 'js',
            main: 'angularfire2.js'
        }
    }
});

System.import('/js/main').then(null, console.error.bind(console));

But when I run code on Chrome browser, It output error in Console

My dependencies version:
"angular2": "2.0.0-beta.13",
"firebase": "^2.4.2",
"angularfire2": "2.0.0-alpha.13",
"systemjs": "0.19.25",

There is a module.exports = Firebase; at the end of firebase-web.js and that is something the browser does not understand. And if I delete it, it output another error:

How to slove it ?
Thank you

@jeffbcross , @davideast Please help me :baby_chick:

I have been resolved problem by adding: meta config :joy:
System.config({
    meta: {
        firebase: {
            format: 'global'
        }
    },
    map: {
        firebase: '/js/lib/firebase-web.js',
        angularfire2: '/js/lib/angularfire2'
    },
    packages: {
        baseURL: '/js/',
        js: {
            format: 'register',
            defaultExtension: 'js'
        },
        angularfire2: {
            defaultExtension: 'js',
            main: 'angularfire2.js'
        },
    }
});

@euclid1990 Those tricky System configs :angry:
Glad to see you resolved it. I'm closing the issue, but let me know if you run into any other errors.


Runs in pegging appear to be completely broken

Output below
Hello, you have ran the main function. Starting cribbage...
Enter to continue..

================================
AI:0		(c)HUMAN:0
================================

0	1	2	3	4	5
♣Q	♠8	♣A	♥10	♥4	♠A
Choose 2 card(s) to discard to the crib.
1
0	1	2	3	4
♣Q	♣A	♥10	♥4	♠A
Choose 1 card(s) to discard to the crib.
0
Enter to continue..

♥2 cut.
Enter to continue..

================================
AI:0		(c)HUMAN:0
================================
Count: 0

AI played ♥J
Enter to continue..

================================
AI:0		(c)HUMAN:0
================================
Count: 10

0	1	2	3
♣A	♥10	♥4	♠A
Choose a card to play.
1
HUMAN played ♥10
Enter to continue..

================================
AI:0		(c)HUMAN:0
================================
Count: 20

AI played ♠6
AI: run of 3 for 3!
Enter to continue..

Just had a 6-4-5 not register as a run of 3 (although it did register as a 15)
We need to separate the pegging scoring logic into a subset of the Scoring class and control it there.

apparently fixed with https://github.com/PoeppingT/cribbage-cmd/commit/e761ef3e64508387354e6418f4cd6af38219721f


Error on Import

Used the "Additional Path" Compass trick to use the current head of Compass Recipes, and when I @import "recipes";, the following Compass error occours:
Line 1: File to import not found or unreadable: recipes/form/element/border-color.
This is obviously preventing me from using all of Compass Recipes. If you have a quick fix for me to do that would be great, but it does need to be resoled.

Have you replace compass-recipes stable with the current head ? If yes, do you use @import "recipes/form/element/border-color" ?
Because this recipe have been removed, & there is no trace of "element/border-color" in all files in the current head.
Are you sure the @import "recipes"; add the error ? (I've just try to import all & I've no error)

Yes, I just did a fresh clone into Compass Recipes, @import "recipes", and getting that error.

I just did the same on a fresh zip download
http://cl.ly/1F2c3k1X3R3u230B0023
http://cl.ly/1K3l3X2K1v0v3F3O392p
No problem.
So I guess the "old" stable gem I making the problem. @import "recipes"; include the stable gem...
First thing, to correct the gem bug: (my path assume you are on OSX).
Remove the first line of the file /Library/Ruby/Gems/1.8/gems/compass-recipes-0.2.0/stylesheets/recipes/form/_element.scss.
You will no longer have the error.
But it's weird that the "additional path" have not a higher priority on the gems...
I will take a look...

Or maybe just remove the compass-recipes 0.2 gem :)

Removed the compass-recipes 0.2 gem, but now my config.rb isn't picking up the additional paths:
in config.rb:
Additional Import Paths
additional_import_paths = '~/Development/compass-recipes/stylesheets'
Require any additional compass plugins here.
require 'compass-recipes'
Now it's throwing:
LoadError on line 31 of /System/Library/Frameworks/Ruby.framework/Versions/1.8/usr/lib/ruby/1.8/rubygems/custom_require.rb: no such file to load -- compass-recipes
Am I using additional_import_paths wrong?
BTW, thank you for your help.

Yes you are using it wrong :p
additional_import_paths - Array of Strings - Other paths on your system from which to import sass files. See the add_import_path function for a simpler approach.
So you need
    '~/Development/compass-recipes/stylesheets'
]```

Maybe you just want :
`add_import_path "~/Development/compass-recipes/stylesheets"`

from http://compass-style.org/help/tutorials/configuration-reference/

Ahh, Ruby array, not Sass array. Either way, still not working; same error. Now using add_import_path

I found this in my memory http://groups.google.com/group/compass-users/browse_thread/thread/8e655d6706ca6d2d?pli=1
Maybe using ~/ unix trick does not work (since the ruby process does not know your homepath ?)
Is it working with add_import_path ?

It's not working w/add_import_path and I tried changing to an absolute directory, neither of which did the trick.
Is there a reason the gem isn't updated? Can I help w/that? Can I compile a gem out of the git repo?

It's weird for the import path. I used to import recipes like that...
Do you use compass watchor Compass.app ?
We have planned to release an alpha release (but @davidkaneda have to push something before)
Anyway, you can build the gem easily (everything is almost ready).
Into the repo:
sudo gem build compass-recipes.gemspec
sudo gem install compass-recipes-0{tab}

This should work.

Compass Watch
So when running gem build, got the following:
    undefined method `today' for Date:Class

Hum (I'm not a ruby pro, so I do not know where the Date class is...)...
So just edit the gemspec file & remove the date line. It should works.

That worked. Thank you very much!

0.2.1 has been released on rubygems to fix this.


Employees page is breaking on first date of the month

Describe the bug
The employee page (Menu->HR->employees) breaks on the first date of the month.
To Reproduce
Steps to reproduce the behavior:

change the date in your system to the first day of the month
Open the employees page
See the error

Expected behavior
This should work fine.
Screenshots
NA
Desktop (please complete the following information):
NA
Smartphone (please complete the following information):
NA
Additional context
NA

This issue is fixed refer screenshot


Implement --pre option for pip in PipConfig

This allow install pre-release and development versions of python packages

Looks fine. Thanks!


Include script/style bundles

Is there a possibility to include script and style bundles to the layout?
We build those bundles dynamically together, so they include many different .css/.js-files.
In our non-e-mail layouts we always include them like that:
 @Styles.Render("~/bundles/" + theme + "/styles") @Scripts.Render("~/bundles/scripts") @RenderSection("scripts", false)

Same question.


Updated UV texture

Updated the UV texture, to a high resolution version.
After:

Before:

Thanks!


problem with adding httpheaders

hi
i want the ngautocomplete to sent a custom httpheader while calling the api but it doesnt send the added header, here is the code i use:
public dataService: RemoteData;
constructor(private completerService: CompleterService) {
this.dataService = completerService.remote(some url, 'label', 'label');
let options = new RequestOptions({ headers: new Headers() });
options.headers.set('api-key', 'some credentials');
this.dataService.requestOptions(options);
}
<ng2-completer [(ngModel)]="partTerm"
[ngModelOptions]="{standalone: true}"
[datasource]="dataService"
[clearUnselected]="true"
[inputClass]="'form-control'"
[placeholder]="'search by term'"
[minSearchLength]="2"
[pause]="500"
(selected)="selected($event)">
i'll appreciate it

Looks like they changed the way request headers are passed in.
This worked for me using 1.6.3:
this.dataService = this.completerService.remote(null, 'name', 'name');
let options = new RequestOptions({
  headers: new Headers({
    Authorization: 'token ' + localStorage.getItem('token')
  })
});
(<RemoteData>this.dataService ).requestOptions(options);

This is what it looks line now that i'm using 2.0.8:
this.dataService = this.completerService.remote(null, 'name', 'name');
(<RemoteData>this.dataService).requestOptions({
  headers: {
    Authorization: 'token ' + localStorage.getItem('token')
  }
});

Believe the change occurred with release of v2.0.0
https://github.com/oferh/ng2-completer/blob/master/CHANGELOG.md
Another interesting note... the demo... https://oferh.github.io/ng2-completer/#/native (Remote data with URL Formater and Headers)  doesn't add remote headers as far as i can see.. https://github.com/oferh/ng2-completer/blob/master/demo/native-cmp.ts#L94-L101


Implement Temperature checker

Temperature checker is a core thermostat component.
Input:

Target temperature(s) information
Zones

Output:

Temperature state

Todo:

[x] Finalize Plans/Schedules relation and interaction
[ ] Linear approximation method

Weighed arithmetic mean in combination with linear approximation of target temperature will be used for first versions.


[Schema Registry Avro] Remove old format compatability

Packages impacted by this PR
@azure/schema-registry-avro
Issues associated with this PR
Fixes https://github.com/Azure/azure-sdk-for-js/issues/20063
Describe the problem that is addressed by this PR
The old payload format was planned to be supported until the last beta before the first stable release and since the first stable release is scheduled for May, the compatability should be removed now.
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[x] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[x] Added a changelog (if necessary)

API change check for @azure/schema-registry-avro
API changes are not detected in this pull request for @azure/schema-registry-avro


Remove old version templates in convert data test

Description
Remove old version templates in convert data test which may cause parse failures in E2E test. Now we directly upload embedded templates to remote ACR.
Related issues
Addresses [issue #].
Testing
E2E test
FHIR Team Checklist

[x] Update the title of the PR to be succinct and less than 50 characters
[x] Add a milestone to the PR for the sprint that it is merged (i.e. add S47)
[x] Tag the PR with the type of update: Bug, Dependencies, Enhancement, or New-Feature
[x] Tag the PR with Azure API for FHIR if this will release to the managed service
Review squash-merge requirements

Semver Change (docs)
Skip

When do these tests get run? The comments says they are skipped locally and I see them being skipped in the PR.

When do these tests get run? The comments says they are skipped locally and I see them being skipped in the PR.

Actually these  tests do not run in the OSS pipeline because they need to inject the target ACR info into environment variables (We need the ACR secrets to upload a template, for PaaS testing we have made it work by saving the information in the Key Vault of test FHIR account).
I just made a few changes to add ACR information to account key vault, does that make sense?

  ScriptType: inlineScript

this script and script starting in line 159 looks identical for me. Can they be refactored into one script file with link to it?
Only difference I see is name of ACR container is suffixed with -sql for sql.
Can same ACR container be used for SQL and Cosmos backend? Is there are difference or ACR just can't handle request from different systems?

Refers to: build/jobs/run-tests.yml:62 in 734a0c8. [](commit_id = 734a0c845797049ced5bc15aaa8c5d7da2b40a32, deletion_comment = False)

into

Yes. They are two parts for setting up Cosmos and Sql E2E variables. I feel we can refactor the whole into a set-variable task, but that would take some time to test the pipeline. We can merge this bug fix first and create an item to track this?
Same ACR can be shared by different server, but in the pipeline settings, all resources are deployed with ARM template where each ACR are binded to a single FHIR server. Just like the storage account in export scenario.

  ScriptType: inlineScript

this script and script starting in line 159 looks identical for me. Can they be refactored into one script file with link to it?
Only difference I see is name of ACR container is suffixed with -sql for sql.
Can same ACR container be used for SQL and Cosmos backend? Is there are difference or ACR just can't handle request from different systems?
Refers to: build/jobs/run-tests.yml:62 in 734a0c8. [](commit_id = 734a0c8, deletion_comment = False)

Yes. They are two parts for setting up Cosmos and Sql E2E variables. I feel we can refactor the whole into a set-variable task, but that would take some time to test the pipeline. We can merge this bug fix first and create an item to track this?
Same ACR can be shared by different server, but in the pipeline settings, all resources are deployed with ARM template where each ACR are binded to a single FHIR server. Just like the storage account in export scenario.


Fix/Update dependency/f5totec/f5tovtk installation

Myself and a few others using OpenMPI-4.0.6 were having issues compiling f5totec and f5tovtk which are fixed in this PR by explicitly linking to open-pal.
Additionally, I noticed that the readme and Makefile in the extern directory had not been touched in 6+ years so I updated them. I removed the readme and added more instructions to the installation page in the docs to avoid duplication.
Since AMD/UFconfig are optional, the Makefile currently doesn't care if they fail to build. I did this so that running make should still work even if the user doesn't provide the AMD and UFconfig tarballs. But it also means that the user will not be notified if they do provide the tarballs but the compilation fails. I'm open to suggestions on how to handle this properly from people who know more about writing makefiles than me.

Cane we move the TECIO include/lib information to the Makefile.in.info to be consistent with the other libraries? See here for example.

Can we move the TECIO include/lib information to the Makefile.in.info to be consistent with the other libraries? See here, here, and here for example.

Yeah, good idea, should I use the info from Makefile.in in the extern makefile too?

Installing tecio should be optional and omitting it shouldn't cause any errors.
…
________________________________ From: Alasdair Gray @.> Sent: Wednesday, August 3, 2022 9:19 AM To: smdogroup/tacs @.> Cc: Subscribed @.> Subject: Re: [smdogroup/tacs] Fix/Update dependency/f5totec/f5tovtk installation (PR #121) Can we move the TECIO include/lib information to the Makefile.in.info to be consistent with the other libraries? See herehttps://github.com/timryanb/tacs/blob/conda-build/Makefile.in.info#L61-L64, herehttps://github.com/timryanb/tacs/blob/conda-build/extern/f5totec/Makefile, and herehttps://github.com/timryanb/tacs/blob/conda-build/TACS_Common.mk#L16-L24 for example. Yeah, good idea, should I use the info from Makefile.in in the extern makefile too? — Reply to this email directly, view it on GitHub<#121 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACCBPTSL6WQN4ADKBTFMU2DVXJWWXANCNFSM55NACKGQ. You are receiving this because you are subscribed to this thread.Message ID: @.>

Yep, that should still be the case after these changes, just as it is for AMD

Can we move the TECIO include/lib information to the Makefile.in.info to be consistent with the other libraries? See here, here, and here for example.

Yeah, good idea, should I use the info from Makefile.in in the extern makefile too?

I think it's fine to leave the extern file the way you have it. The way I see it is exposing all of the library links at the Makefile.in level allows users to pass their own external compiled libraries during the tacs compilation procedure. If the user calls make from the extern directory, then I think the assumption is they want compile all of those libraries locally the tars there.


Add stellar scenarios and renderings

Add stellar renderings for recently added scenarios

@bsdorra This one looks like a Stellar bug, are you aware of it? https://modelviewer.dev/fidelity/#khronos-TransmissionRoughnessTest

@elalish if you're referrering to the darker color, then it is expected behavior.
The pathtracing integrator evaluates the albedo two times for these volumes, on enter and on exit. This effectively gives you albedo^2.

@bsdorra @gkjohnson The purpose of these tests is to find rendering inconsistencies and it's found one even between these two path-tracers. I just figured it might be worth bringing up at PBR to decide who has the spec-compliant behavior.

My path tracer is only evaluating color when entering the front face of the surface but looking at the spec again it's probably actually correct to multiply in the color on the way in and out of the volume. I'll put it on the list to make that change:

Base color changes the color of light at the volume boundary.

Thanks!

It looks like three.js is making an update that will bring it more in line with Stellar here too: https://github.com/mrdoob/three.js/pull/25502
I guess that means we'll have to talk to Filament, Babylon, and the gltf-sample-viewer.


Bootstrap from NPM

ready to pull in Bootstrap 5.3 from NPM

solves https://github.com/Office-of-Digital-Services/California-State-Web-Template-Development/issues/28


OPCT-191: Build and publish the CLI image to registry

https://issues.redhat.com/browse/OPCT-191
This PR introduces the CLI container image, publishing it on Quay.io to allow be mirrored by OpenShift CI jobs, decreasing the complexity and logic of the CI steps.
https://quay.io/repository/ocp-cert/opct?tab=tags

Richard, ptal?
/assign @rvanderp3

/assign @faermanj

/lgtm
/approve


Train heat map predictor

Hi, I am using the panoptic dataset, and I am wondering if it is possible to train the heatmap predictor with this dataset?

@sudo-vinnie hi, dude, have you done the cross-dataset evaluation? i find that the model performs well in single datasets. but when i use the panoptic dataset as the trainset while the campus datasets as the testset, the performance is very poor.


hostPort != containerPort

This makes it possible that e.g. the actual hostport is a random one but in the container its always 80

@bopi- one last thing - could you please implement a test for it?

I see that @bsideup has review in hand, but thanks for this @bopi-. Looks like a useful addition!

@bopi- I just applied some minor changes myself and ready to merge as soon as the CI status is green  👍

I just applied some minor changes myself

yeah. naming and caching ... :smiley:

@bopi- merged! Thanks a lot for your contribution 👍
We should have done it this way from the beginning :)

@bsideup thank you! Especially for your super fast responses!
Do you have an Idea when this can be released?

@bopi- it will be released soon since we have a couple of urgent things to be released, probably this weekend

@bopi- it will be released soon since we have a couple of urgent things to be released, probably this weekend

Hi @bsideup,
any news on a new release?

Released in 1.11.2!


[BUG]

Windows Client:
All Steps done. But i cannot starting harvester
Error below:
Traceback (most recent call last):
File "C:\Users\RG\develop\ceres-combineharvester\venv\Lib\site-packages\ceres\server\start_harvester.py", line 74, in
File "C:\Users\RG\develop\ceres-combineharvester\venv\Lib\site-packages\ceres\server\start_harvester.py", line 68, in main
File "C:\Users\RG\develop\ceres-combineharvester\venv\Lib\site-packages\ceres\server\start_harvester.py", line 47, in service_kwargs_for_harvester
File "ceres\harvester\harvester_api.py", line 30, in init
File "ceres\util\ceres_all_coins_config.py", line 33, in get_all_coins_config_constants
File "ceres\util\ceres_all_coins_config.py", line 50, in get_coin_default_constants
File "importlib_init_.py", line 127, in import_module
File "", line 1030, in _gcd_import
File "", line 1007, in _find_and_load
File "", line 984, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'ceres.consensus.all_coins_default_constants.cryptodoge_default_constants'
[8068] Failed to execute script 'start_harvester' due to unhandled exception!

will be fixed.
Thanks


councurrency option documentation clarification

We might have to clearly express the purpose of the concurrency option in the documentation. As of now it looks like this

The concurrency limit applies to promises returned by the mapper function (since the operations the promises in the input array are "watching" are already running and thus cannot be limited).

It took me a while to understand what it actually says (especially the parenthesized text). When I had to explain that to OP in one of my Stackoverflow answers, I used the following words

The concurrency parameter basically limits the number of Promises which can be resolved before resolving more promises. So, in this case, since the limit is 1, it will execute the first promise and as the limit is reached, it will wait till the first Promise resolves, before moving on to the next Promise.

Should the actual documentation be updated?
PS: Please let me know if my text in the answer needs rework :-)

Yes, it needs rework imo.
If you have a promise, it resolves whenever it wants, you cannot really hinder it. A promise is not "executed", an operation that returns/ed a promise is.
Therefore, the concurrency parameter limits the number of promises that are created and awaited before more calls to the mapper function.

@bergus Thanks :-) I updated the answer now, can you please check if it looks okay?

That's better. If you make a PR for the docs and it explains the fact that concurrency refers to the amount of mappers called at once I'll merge it - please note that documentation should probably focus more on 3.0 which will be documented on the website and not on GH :)

@benjamingr Here is the PR. Working on 3.x docs now. Thanks @joncle and @mattdmo for reviewing the text :-)

Closing this because the PR is accepted :)


When you use a wildcard DNS hostname, i.e. *.domain.com, the return

value is \052.domain.com. which needs to be cleaned up. There are a
few other octal values that need to be handled.  Boto faces the same
issue: https://github.com/boto/boto/pull/1216 with a suggested fix
from a gist: https://gist.github.com/meonkeys/4482362#file-route53octals-py-L13
Test code for a single record set along with tests to mirror the gist
were added.

I somehow missed this one. I understand that this is very late to the game, but if you want to rebase and re-submit, I'll get this merged in. Sorry for the radio silence for so long.


Req: add "increased Elemental Damage" filter(s)

Howdy!  Awesome app, wish I'd known about it earlier.
One request: could you add a filter for "increased Elemental Damage with attack skills" or just "increased Elemental Damage"?  Probably the latter, since that's what Jewels, Sceptres, and Shield seems to use.
Thanks!

Hi, @mbourgon .  The 1.21.0 release includes filters for "increased Elemental Damage" and "increased Elemental Damage with Attack Skills".  Note that the two filters do not overlap; the "increased Elemental Damage" does not include increases for attack skills.
Also, as with similar stat filters, this does not look at Abyss Gems.  This is a known issue and I hope to fix it soon.


feat: added supplements to presentation exchange

We've added supplement and attachment features to the presentation receive method.

Peter did format the files with black before committing.


The y-axis always starts with zero

Even I set axis.y.min = 100000. But it still always starts at zero. All I want is set a "false bottom" of the bar chart.
I'm using c3 v0.3.0 and d3 v3.4.13. Please help!!

Hello Masayuki

Hello masayuki

Hello Masayuki
I also faced above problem i tried the 0.4.11 and 0.4.18
to solve that problem i setted zerobased = true
then it's solved
thanks.


[Bug] BinlogSplitReader#pollSplitRecords return finishedSplit when exception occurs

Search before asking

[X] I searched in the issues and found nothing similar.

Flink version
1.18
Flink CDC version
3.0
Database and its version
any
Minimal reproduce step
Current Code
In current BinlogSplitReader#pollSplitRecords, when the currentTaskRunning = false, will return null, which is seen as fininished split.
See:
//com.ververica.cdc.connectors.mysql.source.reader.MySqlSplitReader#pollSplitRecords
return dataIt == null ? finishedSplit() : forRecords(dataIt);

Problem occurs:
However, the currentTaskRunning = false in four situations:

the bounded stream split is finished( later in issue: https://github.com/ververica/flink-cdc-connectors/issues/2867)
the stream split is paused for new scanly tables(See MySqlSplitReader#suspendBinlogReaderIfNeed)
some exception occurs(See executorService.submit)
The BinlogSplitReader#close
Only in the former two situations, the spilt is finished, otherwise problem will occor.

For example, there is an unbounded stream split:

t1, add this unbouned stream split and start a new thread to fetch binlog.
t2, BinlogSplitReader#pollSplitRecords check there is no Exception at first.
t3,  some excetpion occurs in binlogSplitTask(network, data error, and more), set currentTaskRunning = false.
t4, BinlogSplitReader#pollSplitRecords check currentTaskRunning is fasle, so return null, which is seen as fininished split. Then MysqlSourceReader move to next split.

Thus, when the task is not running, we also need to distinguish whether the split is finished more carefully. I have two idea:

add lock(not a good choice)
when the stream split is paused, we also add an END watermark to queue. Only when get an END watermark, BinlogSplitReader#pollSplitRecords return null, otherwise return empty collections.

What did you expect to see?
Only when get an END watermark, BinlogSplitReader#pollSplitRecords return null, otherwise return empty collections.
What did you see instead?
[Bug] BinlogSplitReader#pollSplitRecords return finishedSplit(null) when exception occurs
Anything else?
No response
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

@loserwang1024 Can BinlogSplitReader#pollSplitRecords call checkReadException() address this problem before return null?

@gong , Yes, it can solve this case.But there are also other case, for example,  it still some other problems when add SNAPSHOT_ONLY ablitity :

Closing this issue as it has been migrated to Apache Jira.


Improve IO.forkAll so it returns a composite fiber

Here's my implementation of IO.forkAll that returns a composite fiber (closes #18).
I used foldRight so the accumulated values are in the right order (:: is right associative).

Can you please resolve conflicts? Thanks!

@dennisVDB Thanks for the contribution! 😄

Why are we specializing on List here while everywhere else TraversableOnce is accepted?

That’s true, I kind of just followed the definition given in #18. Should I make a mini-PR that fixes that?
It was a nice experience for a first open source PR!

Why are we specializing on List here while everywhere else TraversableOnce is accepted?

Why are we accepting TraversableOnce everywhere else?

@hrhino rule of least power I guess


🫐 🐛 Namespace stuck in Terminating state due to fluxcd finalizers

What happened?
┌─────────────────────────────────────────────────────────────────── Describe(-/redpanda) ───────────────────────────────────────────────────────────────────┐
│ Name:         redpanda                                                                                │
│ Labels:       kubernetes.io/metadata.name=redpanda                                                                                │
│ Annotations:  <none>                                                                                │
│ Status:       Terminating                                                                                │
│ Conditions:                                                                                │
│   Type                                         Status  LastTransitionTime               Reason                Message                                      │
│   ----                                         ------  ------------------               ------                -------                                      │
│   NamespaceDeletionDiscoveryFailure            False   Tue, 14 May 2024 18:38:28 -0400  ResourcesDiscovered   All resources successfully discovered        │
│   NamespaceDeletionGroupVersionParsingFailure  False   Tue, 14 May 2024 18:38:28 -0400  ParsedGroupVersions   All legacy kube types successfully parsed    │
│   NamespaceDeletionContentFailure              False   Tue, 14 May 2024 18:38:28 -0400  ContentDeleted        All content successfully deleted, may be wai │
│ ting on finalization                                                                                │
│   NamespaceContentRemaining                    True    Tue, 14 May 2024 18:38:28 -0400  SomeResourcesRemain   Some resources are remaining: helmcharts.sou │
│ rce.toolkit.fluxcd.io has 1 resource instances, helmreleases.helm.toolkit.fluxcd.io has 1 resource instances, helmrepositories.source.toolkit.fluxcd.io ha │
│ s 1 resource instances, redpandas.cluster.redpanda.com has 1 resource instances                                                                            │
│   NamespaceFinalizersRemaining                 True    Tue, 14 May 2024 18:38:28 -0400  SomeFinalizersRemain  Some content in the namespace has finalizers │
│  remaining: finalizers.fluxcd.io in 3 resource instances, operator.redpanda.com/finalizer in 1 resource instances                                          │
│                                                                                │
│ No resource quota.                                                                                │
│                                                                                │
│ No LimitRange resource.                                                                                │
│

Terraform destroy output
╷
│ Error: Timed out when waiting for resource "redpandas.cluster.redpanda.com" to be deleted
│
│ Deletion timed out. This can happen when there is a finalizer on a
│ resource. You may need to delete this resource manually with kubectl.
╵
╷
│ Error: Timed out when waiting for resource "helmcharts.source.toolkit.fluxcd.io" to be deleted
│
│ Deletion timed out. This can happen when there is a finalizer on a
│ resource. You may need to delete this resource manually with kubectl.
╵
╷
│ Error: Timed out when waiting for resource "helmrepositories.source.toolkit.fluxcd.io" to be deleted
│
│ Deletion timed out. This can happen when there is a finalizer on a
│ resource. You may need to delete this resource manually with kubectl.
╵
╷
│ Error: context deadline exceeded
│
│
╵
╷
│ Error: Timed out when waiting for resource "helmreleases.helm.toolkit.fluxcd.io" to be deleted
│
│ Deletion timed out. This can happen when there is a finalizer on a
│ resource. You may need to delete this resource manually with kubectl.
╵

What did you expect to happen?
my terraform destroy -auto-approve operations to finish successfully
How can we reproduce it (as minimally and precisely as possible)?. Please include values file.

❯ helm get values redpanda-broker -n redpanda --all
Error: release: not found

Anything else we need to know?
No response
Which are the affected charts?
Redpanda
Chart Version(s)

❯ helm -n redpanda list
NAME	NAMESPACE	REVISION	UPDATED	STATUS	CHART	APP VERSION

Cloud provider

Azure

dup of #1232


Some suggestions.

Didn't write that in the PR:
We often make code in the examples folder BSD0. See for example https://github.com/toitware/toit-ds18b20/tree/main/examples
After all: we want people to copy that code, and it's easier if they don't need to worry about giving attributions for these snippets.
That said: it's fully up to you.


Distance between two events on same day

I know PASS had limitations of 600 miles distance between two events on the same day. Members had a difference of opinion on that.  Should we consider something similar?

It was unenforceable in Canada, and frankly I don't see the point of an arbitrary distance limit. For the amount of effort it goes to putting together a free event, it would make sense for people in areas nearby to work together. If there's something they were trying to avoid, let's think of different ways to avoid them.

I would be against such a rule for a couple of reasons. For example the distance between Milano and Zürich is roughly 300km (186mi), a bit less than going to Pordenone, yet the local communities speak different languages and many prefer to attend sessions (and network) in local language. So, while two events on the same day in Milan and Zürich would not compete, the same would not be true with Milan and Pordenone. Also, what I've experienced in the last few years speaking at Azure Bootcamps, running the same day in different major Italian cities, is more people attending across them (which just makes sense since they don't have to travel as much). It is more a burden for speakers that may need to choose which one to attend but that's it.

What about distance by Country?

I think as an initial "rule" we can simply make sure that events on the same day know about each other & the organizers talk to each other.
I agree most attendees are unlikely to drive much more than an hour or two to go to a Saturday event.
One of the concerns I've heard in the past isn't about attendees but rather about speakers. There was a complaint a few years ago that one of the bigger, more established events attracted all the good speakers and the smaller event on the same day struggled to get speakers.
However--speakers sometimes fly great distances to speak at events, particularly bigger events. If organizers are able to talk it out, they should be able to help each other, rather than fighting with each other.
If "make the organizers talk it out" proves to be ineffective, then I'd vote for revisiting this based on how things go.

Agree with what @amtwo  said. Another concern is the sponsors. In the USA there are some sponsors who have national/global presence that goes to most events. Then you have a few local sponsors. If we have two events in close proximity as we will struggle with sponsors and what Andy said about speakers.

My biggest concern would be sponsors for sure. I think that's the most inelastic of the three groups (Sponsors, speakers, attendees).
Speakers is a valid concern, but I also feel like part of the goal of these events is to help encourage and grow local speakers. As for attendees, I agree this is likely a non-concern. My understanding of SQL Saturday Baltimore and DC is that they had very little attendee overlap, despite being very close in proximity.

I have always disliked this rule, simple because it limits accessibility and the ability of organizers to reach the community. I believe this came about, as mentioned above, because of organizers not wanting to compete for speakers and sponsors being able to support multiple events.
My view has been that all of us our adults, and we can work out differences, or we will find that two events in the same city aren't sustainable for these, or other, reasons.
My guidance would be that organizers within a geography, say 100km, coordinate with each other.
I also hope this is a point of concern in 2021 and we have live events.

I agree with @amtwo  and @rabryst on this issue. The distance between London and Paris is a touch over 200 miles for example but they would not be competing for sponsors or attendees I would surmise.
Communication between the events will be important - Maybe a calendar would be useful for organisers to be able to organise

Brainstorm🧠🌩:  What if there was a mechanism so that when an event date is entered, it automagically says "Here's the list of other events +/- a day of your event! Y'all should talk!" ... it should probably be also enable some sort of identical ping to the existing event organizers. That should make it easy for events held on the same day or same weekend to take note and decide if there's a problem. Paris & London organizers might decide to ignore the message, while Cincinnati & Cleveland might decide they need to talk & work together.
Event Date would need to be captured as some sort of structured/queryable metadata to enable that.
The alert mechanism could be as simple as a @ mention in GitHub.

I like that idea. Sharing information and allowing decentralized resolution are good qualities to embrace and support.

I like that idea from @amtwo to help event organizers know what's happening around them.
This is a fringe case that may or may not need consideration, but in more remote areas, like the Great Plains and Mountain West in the US, limiting events on the same weekend, due to distance, may prevent some people from being able to attend.  As an example, let's say Denver CO and Casper WY want to run on the same weekend or weekends close to each other.  If Casper's event was rejected in favor of Denver, those who live in Rapid City, SD may not be left out, because it's a longer drive to Denver than to Casper.


p11

-------Please read and delete this text before creating the issue-------
To report a good issue, please verify that you validate these points:

[x] you have read the documentation about command line options and advanced use cases
[x] upgrade and test with the last version
[x] specify the version you are using
[x] include steps you followed to reproduce the problem
[x] if the problem occurs when using a git-tfs command, please provide the output of the command when using the --debug command line option (providing a gist could be a clever idea ;))

And don't forget:

The fastest way to get an issue fixed is to submit a PR that fixes it.

The slowest way to get it fixed is to hope someone else will fix it.

It seems spam. If not, please explain the purpose. Closing while waiting more information...


Add Lumber (https://github.com/ForestAdmin/lumber)

Thanks, but please add your resource at the bottom of the current list, or it's not fair to the ones that were there before you ;)

@fzaninotto Done :)

Thanks!


Building a project where all glbs are externally loaded will failed because of shader stripping

Hi,
We have a project where all content is exclusively downloaded externally, stored in the persistent folder and then loaded in game. We get an error in this published application where it can't find the shaders it needs to create the glb it just loaded. because it's been stripped during compilation since Unity did not find any usage for it  ( that's my understanding of the situation, I may be wrong here)
Is there a guide as to how to deal with this properly? right now, I am tempering with putting a glb in the resources, it seems to work, but is it the best way or not, I am not sure, there could be more implications that I am not foreseeing.
Bye,
Jean

Hello,
maybe you try to add "Assets/UnityGLTF/Runtime/Shaders/ShaderRetention/ShaderRetention.unity" scene to your application build (just have it on the "Scenes In Build" list).
I'm also loading glb files from "internet" and it works correctly for me.
BR,
Marcin

Hi,
uhm, so that's the purpose of this scene! why not simply put the shaders in a Resources folder? that would be automated then...
Let me close this issue, but I would definitely consider improving this to avoid frustrations when devs encounter this.
Bye,
Jean


Via instruction is wrong

The first instruction after a via point has to be recalculated in PathMerger. Otherwise we could get e.g. 'continue' when we need 'turn around' like here. Or fix this via merging all edges into one Path object and creating instructions out of that.
And not sure if the following is a bug: the via point should not insert a duplicate point in the pointlist, instead we need an explicit skip of FinishInstructions when adding to the point list. BUT keep in mind that we will have to tweak the code in Instruction.createJson

also include u-turn instructions for start points with direction (once #434 gets merged)

I will close here as this is a duplicate at #1064 and the issue will be fixed via #1073.


Latest Version of remark-toc Uses ESM Modules

Summary
The latest version of remark-toc uses ESM modules. I'm unable to upgrade to it, I think Gridsome needs to update itself.
// From CommonJS
var remarkToc = require('remark-toc')

// To ESM
import remarkToc from 'remark-toc'

Heads-up that this is also rippling through the remark ecosystem as other packages also update to ESM.

Yeah it's a huge pain. Kludge I've been using: https://www.npmjs.com/package/jiti


Add additional icons to snapcraft build

Added addtional desktop icons for snapcraft build

This pull request is now managed with CodeReviewHub. Refresh.