Search Notebook: Fixing Latex

Latex in the Genetic Algorithm section was not compiling. After some probing I found that the issue was the limits code. I worked around it. The end result, in my opinion, is not very pretty, but it is understood and gets the job done.

Good detective work!


refactor: refactor from BufReader to BufRead

Which issue does this PR close?

Rationale for this change
Enhance API flexibility.
In our use case, we have a SyncIoBridge which implements the Read and BufRead traits, and it works well with arrow::csv::reader::infer_reader_schema.
        let reader = SyncIoBridge::new(decoded);

        let (schema, _records_read) = arrow::csv::reader::infer_reader_schema(
            reader,
            self.delimiter,
            self.schema_infer_max_record,
            self.has_header,
        )

However, the  ValueIter::new only accepts the type of BufReader type argument. If changing BufReader<Read> to BufRead, We can call the API without the BufReader wrapped outside.
        let mut reader = BufReader::new(SyncIoBridge::new(decoded));
        let iter = ValueIter::new(&mut reader, self.schema_infer_max_record);

What changes are included in this PR?
Refactor infer_json_scheam's reader type from BufReader to BufRead

Are there any user-facing changes?

The Clippy seems failed with:
error: ambiguous glob re-exports
  --> arrow/src/compute/kernels/mod.rs:31:13
   |
31 |     pub use arrow_ord::comparison::*;
   |             ^^^^^^^^^^^^^^^^^^^^^^^^ the name `contains_utf8` in the value namespace is first re-exported here
32 |     pub use arrow_string::like::*;
   |             --------------------- but the name `contains_utf8` in the value namespace is also re-exported here
   |
   = note: `-D ambiguous-glob-reexports` implied by `-D warnings`

error: could not compile `arrow` (lib) due to previous error

Where did you see the clippy error? It was not detected by CI.

Thank you


QPUConnection breaks abstraction with QVM/QPU equivalence

The current implementation of the QPUConnection object breaks the intended symmetry between QPU and QVM. It declares certain metrics inside the Connection object that is oustide the scope of a connection container. Ramsey, T1, etc. should use a connection object and not be defined inside a connection object. The current abstraction seems to break this separation of concerns.
Any thoughts @willzeng @ncrubin @stevenheidel

See my PR and also @pkaralekas 's upcoming PR which cleans up QPUConnection.

I think this is fixed now thanks to Peter, can we close?

@jotterbach I think Peter and I improved this with our commits last week, we can reopen if we want to make further changes.


Automatically keep boards alive when Run is active

Fixes #1200
Instead of relying on data updates causing boards to get kept alive (which didn't seem to be working correctly anyway), this PR changes things so any time MobiFlight is in Run mode boards will get kept alive, regardless of whether data is getting sent.

MobiFlightModule.cs: no longer handles any keep alive times and simply exposes a method to set the power save state of the module to either on or off. On devices without updated firmware this will simply update the firmware's lastCommand time. On devices with updated firmware this will update the firmware's lastCommand time if false was provided, otherwise it does nothing.
MobiFlightCache.cs: exposes a new method to keep all connected modules awake, and manages the last time a keep awake message was sent.
ExecutionManager.cs: Calls the new method on MobiFlightCache every tick when Run is active.

This change stands alone and does not need a matching firmware update even though it's sending a setPowerSaveMode event that the current firmware doesn't support. This is because unsupported commands received by the firmware still set the lastCommand variable to the current time, which is sufficient to prevent power save mode from activating.
Having said that, I'll still open a companion change to the firmware to respect the setPowerSaveMode event and remove all the existing code for setting the last command value based on some (but not all) events.
Here's a sample of the log messages using the updated firmware, with debug enabled and tweaked sleep times so I'm not sitting around for 15 minutes waiting:
Debug	11/10/2023 14:39:43	MobiFlightModule.OnDebugPrint(): MobiFlight RaspiPico.debug: Firmware -> Power saving mode on
Debug	11/10/2023 14:41:15	MobiFlightModule.SetPowerSaveMode(): Setting power save for MobiFlight RaspiPico (COM13) to False
Debug	11/10/2023 14:41:15	MobiFlightModule.OnDebugPrint(): MobiFlight RaspiPico.debug: Firmware -> Disabling power saving mode via request
Debug	11/10/2023 14:41:15	MobiFlightModule.OnDebugPrint(): MobiFlight RaspiPico.debug: Firmware -> Power saving mode off

This is a draft pull request. DO NOT MERGE. For draft/testing purposes I've set the keep alive interval to 1 minute. The real release should probably maintain the 5 minutes that's been in the code forever.

Instead of relying on data updates causing boards to get kept alive (which didn't seem to be working correctly anyway), this PR changes things so any time MobiFlight is in Run mode boards will get kept alive, regardless of whether data is getting sent.

This used to work for a long time, but I still haven't figured out why it broke. All the caching logic still makes sense and yet apparently either:

not every 5 mins there was a forced data sent
or the arduino code somehow did not behave correctly

I find it a bit unsatisfactory to now know what the real reason for the current bug is.
This approach however still mean an improvement because @neilenns was saying that we are saving quite some memory with changing the logic on the firmware side of things. And the KeepAwake-message only happens anyway when no other messages are sent, so it doesn't mean an overhead for the serial bus.
@neilenns please help to understand why we were seeing the issue, so that we can make sure we are not doing the same mistake again in the future.
@neilenns please rename the method name or convince me that we should only half implement it and call it all the time with the false

@DocMoebiuz I couldn't figure out why the existing code didn't work either. I looked at it for a bit and decided instead of trying to debug it deeply it would be better to just go with a simpler solution overall.

Memory saving for AVR's is upper 2 digit bytes.

Memory saving for AVR's is upper 2 digit bytes.

If only I got $1 for every byte I save! 😂

@DocMoebiuz I couldn't figure out why the existing code didn't work either. I looked at it for a bit and decided instead of trying to debug it deeply it would be better to just go with a simpler solution overall.

What insights did you have so far? Maybe if we talk about it together then we can pin down the error.

Ok I tweaked this a bit, now with maybe 20% more awesome. I made a firmware change to actually support requesting the modules go to sleep, and the connector now sends a "go to sleep" message to all connected modules when Stop is activated.
The firmware still does keep its own internal 15 minute timeout so even if the connector doesn't send the "go to sleep" message the boards will eventually sleep on their own.
Advantage to this approach: all the method names make sense 🤣

What insights did you have so far? Maybe if we talk about it together then we can pin down the error.

None, I didn't dig too deeply since I'd already decided it was easier to just go with a simpler keep awake model.

@DocMoebiuz I spent some time in the debugger and I don't think there's anything in the desktop app that would directly cause #1200.
While the old code was a bit more complicated (it depends on each different device type doing a check for keep alive and then resending its data if keep alive is needed), it seems in my debugging to be fine. I couldn't reproduce or find, via code inspection, any way that the keep alive check would return false when it should really return true, and the if statements for each of the device types are straightforward.
Only things I can think of:

Mobiflight didn't think the sim was connected. In that situation the keep alive code will never get called because no data is getting sent regularly to the connected modules.
There were no outputs in the loaded config. I hit this in my initial debugging, where I had an old test file loaded that only had inputs. With the previous keep alive code you must have some sort of output configured otherwise your board will go to sleep.
The data did get re-sent to the firmware but for some reason the firmware didn't reset the timer. I can't think of any reason why that would happen though.

@DocMoebiuz I spent some time in the debugger and I don't think there's anything in the desktop app that would directly cause #1200.
While the old code was a bit more complicated (it depends on each different device type doing a check for keep alive and then resending its data if keep alive is needed), it seems in my debugging to be fine. I couldn't reproduce or find, via code inspection, any way that the keep alive check would return false when it should really return true, and the if statements for each of the device types are straightforward.
Only things I can think of:

Mobiflight didn't think the sim was connected. In that situation the keep alive code will never get called because no data is getting sent regularly to the connected modules.
There were no outputs in the loaded config. I hit this in my initial debugging, where I had an old test file loaded that only had inputs. With the previous keep alive code you must have some sort of output configured otherwise your board will go to sleep.
The data did get re-sent to the firmware but for some reason the firmware didn't reset the timer. I can't think of any reason why that would happen though.

Thanks for taking the time and digging into it. Still weird that we can't solve the root cause. Never satisfying.

Start/Stop moved to ExecutionManager.Start/Stop. Ready for rereview!

It'd be great to do this https://github.com/MobiFlight/MobiFlight-Connector/issues/1406 right away

It'd be great to do this #1406 right away

I can look at it later this morning and open a PR for it.


Use specific config types for (de)serialization, not just ConfigDict

BREAKING
FEATURE
This is for discussion only at this point.

This change is

What did we decide on this PR? Can we close this out? If we want to do this in the future let's file an issue and then point to the closed out issue.

I think the approach in this PR is valuable but I don't have bandwidth to revisit it currently.  Closing for now & mentioning on tensorflow/tfjs#1082.  I'll revive the effort when possible.


Push Notifications

This PR is a bit messy. My apologies.
Among laying the foundation for push notifications this PR adds the Dagger dependency injection framework and configures Robolectric to use it to inject fakes for tests.

@colinta said it was good to 🚢


Creating a new project is crashing when voxa is installed globally

Seems like the package maybe doesn't include the typescript interaction.json template ?
$ npm install --global https://github.com/VoxaAI/voxa-cli
/home/armonge/.nvm/versions/node/v10.15.3/bin/voxa -> /home/armonge/.nvm/versions/node/v10.15.3/lib/node_modules/voxa-cli/bin/voxa.js
+<EMAIL_ADDRESS>added 441 packages from 422 contributors in 78.905s
$ voxa
Usage: voxa create

Options:
  -v, --version          output the version number
  -h, --help             output usage information

Commands:
  interaction [options]  create a interaction.json
  init                   create a new interaction.json
  create                 Create a Voxa app
$ voxa create
? Please enter the name of your app sample
? Please enter your name/company armonge
? Which platform will the app use? Amazon Alexa
? Choose a language Typescript
? Will you use Voxa-CLI to generate the interacion model and publishing information? Yes
? Will you use the Canfulfill intent? No
? Will you use DynamoDB to save user's info? No
? Will you use Account Linking? Yes
? What analytics will you be using for this app?(Hit Enter to skip this) Dashbot
{ [Error: ENOENT: no such file or directory, stat '/home/armonge/.nvm/versions/node/v10.15.3/lib/node_modules/voxa-cli/templates/typescript/interaction.json']
  errno: -2,
  code: 'ENOENT',
  syscall: 'stat',
  path:
   '/home/armonge/.nvm/versions/node/v10.15.3/lib/node_modules/voxa-cli/templates/typescript/interaction.json' }
Something went wrong, try again

Good catch, .npmignore is causing this. I'll fix it. Thanks.


Newest game version causes Lure to crash on bootup

Frame one new issue

working on it

still crashes as of newest lure update

sorry forgot to update but it doesn't crash anymore--at least for me
one thing that was crashing my game was Lure + LucysTools & NeosQOL, so I'd definitely check those off first if you have them installed @heikadog


KeyExchangeFailed

I'm connecting to an external SFTP Server. I can connect using WinSCP which gives me the following information:
File transfer protocol = SFTP-6
Cryptographic protocol = SSH-2
SSH implementation = CerberusFTPServer_12.0 FIPS
Encryption algorithm = AES-256 SDCTR (AES-NI accelerated)
Server host key algorithm = ssh-rsa 2048

Trying to connect I get the following error
Renci.SshNet.Common.SshConnectionException: 'The connection was closed by the server: Key exchange failed: Could not agree on key exchange parameters (KeyExchangeFailed).

I'm using the latest version of Neon.SSH.NET which is 2.11.1 at this time.
Thank you.

Hey Marcus,
Here is an example of the code, I've hidden the credentials:
        var connectionInfo = new ConnectionInfo("example.com", "someuserhere", new PasswordAuthenticationMethod("someuserhere", "apassword"));

        using (var client = new SftpClient(connectionInfo))
        {
            client.Connect();

            var yy = client.ListDirectory("/");
        }

The code is throwing an exception on client.Connect(); but using the same credentials in WinSCP works.
If I use the same code above for different SFTP Server then it connects fine.

CLOSING


"SHF Cheapest period start" entity updated at exactly at start time - automations did not trigger

Hello and thank you for the great work!
I have been using "SHF Cheapest period start" to trigger my house heatings to turn on when cheap hours are starting.  So far this has been working fine, but today cheap hours were starting at 00:00 and "SHF Cheapest perior start helper" was updated at excatly the same time.  This caused my automations not to trigger.  Is there a way to force update of "SHF Cheapest period start" earlier to prevent this or something else that should be changed?

This happened again yesterday, SHF Cheapest period start helper was not updated until 00:00.  This might be to late if cheapest period starts at the same time.  I wonder why this update is not happening soon after new Nordpool prices are out.

There have been some problems with this "SHF Cheapest period" sensor. At least one known bug currently is that the calculation takes into account all prices starting from "today 0:00:00". In some cases it could be that the cheapest n hours determined by this system are actually in the past and therefore the sensor does not get updated accordingly. The sensor recalculates values when "tomorrow" changes to "today" and this might be why you are seeing this behaviour.
One extra problem with this is that the next cheapest n hours could be during the afternoon, next night or during tomorrow evening. And you cannot predict that. Therefore, I am working on a solution that would calculate cheapest n hours between today 15:00 and tomorrow 15:00 as usually most of the cheapest hours are during the night but cheapest period could start at 23:00 in some cases (this scheme would take advantage of cheapest n hours spanning on two separate days). And it would also ensure that there is at least 1 "cheapest n hour period" during 24 hours. Time span starting from 15 o'clock was selected as the prices are usually releaset roughly at 14 and the new values should be available soon after that.
Does this sound reasonable? I am going to push updated code soon as it has been running on my instance

I have now pushed update to the main repository. It contains updated code which tries to ensure that there is one "cheapest period" between today 15 o'clock and tomorrow 15 o'clock.
Feel free to test this and report back if it works.

Thanks, I will try this out and report asap.

This seems to working fine with version 0.1.9, so I will close the issue.  Thank you for the quick fix!


Remove name from splitChunks settings (#4769)

Pulled my fork of react-scripts in and ran yarn build --stats && webpack-bundle-analyzer build/bundle-stats.json. Can read about the details of this change in https://github.com/facebook/create-react-app/issues/4769, which also has a more thorough example of the effects of this webpack config change.
The source for the below screenshot has one dynamic import import("zxcvbn")
Before:

After:

Notice that 0.b6c8f3e4.chunk.js is split out of the vendor chunk, which contains zxcvbn.

Replaced by d55525651fb57e6ca6fb79c1d5c61601c129cbbd


Query: Fix #5481 - RC2 Deadlock problem with CountAsync() that does n…

…ot happen with Count()

Replace IX-Async GroupJoin with async/await impl.

@maumar @smitpatel

:shipit:


Can't Find Setter For Property While Using lombok

Hi,
I'm using java-object-diff of version 0.95, it works quite well finding the differences between objects.
But then I try to merge the different properties and apply to database I found the log as follows:
14:33:30.184 [main] DEBUG de.danielbechler.diff.introspection.PropertyAccessor - No setter found for property 'id'
14:33:30.786 [main] INFO de.danielbechler.diff.introspection.PropertyAccessor - Couldn't set new value '1' for property 'id'
I thought it might be caused by the lombok (version 1.18.20), so I add setters/getters to the class, and it works.
So I wonder is there any plans to support lombok? THX

It's fixed when I remove annotation @Accessors(chain = true)
Compile before you run your application


Wired GC system - R and L trigger not working properly

Hardware : DS4\DS3 gamepads; Wii system; BlueRetroDIY (without  SDCARD)
Software : GC Controller Test
R and L trigger not working properly. When pressed to maximum position BlueRetro only sends ANALOG values and digital values never passed to Wii\GameCube system. Maximum analog value is 208 (both on DS3 and DS4 gamepads) insted of 255.

Mapping config -> GameCube Merge analog & digital trigger resolve the problem.


Document is not correct anymore?

Please use the Discussion board if you want to get some help. Please use issues to report bugs.

Description

When config iOS environment by the document，I found that many methods are not corresponding to the code
For example，following codes which are copied from document are not compiled successfully, the function call itself forever?
requestSubscription = async (sku: string, offerToken: string?) => {
    try {
      await requestSubscription(
        {sku},
        ...(offerToken && {subscriptionOffers: [{sku, offerToken}]}),
      );
    } catch (err) {
      console.warn(err.code, err.message);
    }
  };

Expected Behavior

Screenshots

Environment:

react-native-iap:12.4.9
react-native:0.62.2
Platforms (iOS, Android, emulator, simulator, device):iOS

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Press '...'
Error '...' is shown

[Optional] Additional Context

try this
const requestSubscriptionPurchase = async (offerToken, sku) => {     const subscriptionRequest = {       subscriptionOffers: [         {           sku,           offerToken,         },       ],     };     try {       if (Platform.OS == "android") {         await RNIap.requestSubscription(subscriptionRequest);       } else {         RNIap.requestSubscription(product?.productId, null);       }     } catch (error) {       console.log(error);     }   };

@Norcy, that code is correct and it would not call itself recursively. the method names are overloading the ones coming from the library. You can see them working in the Example app. Having said that I see how it can be confusing. I have updated the names and explicitly added the imports to avoid confusion: https://github.com/dooboolab/react-native-iap/pull/2182
@UFFETKK that would not work as for iOS you also need the subscriptionRequest object instead of individual parameters.


First attempt to add GRPC instrumentation

@adriancole Thanks for the quick turn-around, I will move the sample out of the core project.

yeah, after thinking about this, I can make this a test scope dependency,
as it is during the tests that the failures manifest. Still not a great
solution, but I am not sure what else would make more sense?

I agree it is a problem with grpc as it strictly depends on guava. You can
try dependencyManagement section to fix the version, but in practice sleuth
or boot bom will set the dependency versions. optional or provided scope I
think can also help make the build work but not interfere with versions
used by apps.

Sample Project has been pulled out of this repo and is now here: https://github.com/tkvangorder/sleuth-grpc-example. @adriancole Once/if this PR is merged, I can work with you on migrating it to openzipkin.
I moved the "forcing of the guava version" to test scope.
I cleaned up the two failing tests.
I still need to add some testing.

Pretty close, just fighting with my integration test. It runs fine if I run just those three tests, but fails when running the whole suite. I am thinking that something is dirtying the trace context or I am not understanding how the reporter works. The spans of the interceptors are not being stored in the reporter when I run the suite..I will dig in some more tomorrow.

@adriancole @marcingrzejszczak Looks like all tests are now passing, but something has changed in the pipeline :
#!/bin/bash -eo pipefail
mkdir -p $CIRCLE_TEST_REPORTS/junit/
find . -type f -regex ".*/target/.*-reports/.*" -exec cp {} $CIRCLE_TEST_REPORTS/junit/ \;
bash <(curl -s https://codecov.io/bash)
/dev/fd/63: line 1: html: No such file or directory
/dev/fd/63: line 2: syntax error near unexpected token `<'
'
Exited with code 2

@adriancole @marcingrzejszczak I did a little refactoring and added some documentation around the implementation details. How do you both feel about the dependency on grpc-spring-boot-starter? This can work for now but it feels like it would be better if there were a spring-sanctioned gRPC project: spring-grpc or spring-cloud-grpc?

A gRPC sanctioned Spring-Boot starter would also be awesome: https://github.com/grpc-ecosystem could be a good home. I know there are other starters hanging around so it would be good to have a community maintained one that gRPC is willing to put their name on

I might think https://github.com/spring-cloud/spring-cloud-gcp

I think after Google open sourced gRPC they handed things over to the community, so I am not sure if it makes sense to put it under the GCP umbrella? I am a relative newbie to gRPC, so take my opinion with a grain of salt.
Regardless of home, having an agreed upon home for Spring + gRPC would be great. In addition to https://github.com/LogNet/grpc-spring-boot-starter, there is also an interesting reactive effort https://github.com/salesforce/reactive-grpc

From what I recall, the boot team prefers the starter to be external until there is some Framework integration (which there isn't, but there is a jira issue).

I'm merging what we have here ATM and we can think of changing the starter to sth else if necessary.

Another repo has done this.
https://github.com/yidongnan/grpc-spring-boot-starter
This repo seems to provide a lot of features.

@yishibai02 This PR relies on grpc-spring-boot-starter being on the classpath and will add tracing interceptors to the servers (by introducing a brave's server tracing interceptor) and to clients (by introducing a spring-aware ManagedChannelBuilder and brave's client tracing interceptor)

I see this repo also supports Spring Sleuth to trace application.
GrpcServerAutoConfiguration.java#L147-L159
GrpcClientAutoConfiguration.java#L158-L170
I think these commit be better placed in the third-party starter repo.

@yishibai02. Thanks for the clarification!
The repo's have the same name but are very different in the implementation details.
For now, this PR introduces auto-configuration only when the LogNet/grpc-spring-boot-starter is on the classpath.
There is still work to be done to introduce an "official" project for using grpc with spring/spring boot. Perhaps yidongnan/grpc-spring-boot-starter might be a good place to start as a contribution towards that goal?


[Suggestion] Total item count

Trying out this mod hoping it will improve performance over ScannerFix, but I do like the total item count it shows when scanning multiple items. Could this be added here?

Can you explain this in more detail?

Here it is in the mod's thumbnail:

The "Count: 24" is what I mean.

So, I looked at this a while ago and decided the effort is not worth it


Token dashboard - not showing 2nd time staked keep ?

Description:
I got a token grant of 300k, my ETH account - 0x27D4876C8dD7abF6e40aAC6F9Fe0942e01fBF5cE.
I'm https://dashboard.test.keep.network dashboard.
Initially I staked 200k (out of the 300k Grant) and here is my Etherscan txn - https://ropsten.etherscan.io/tx/0x6f073fe20ec00905545c49e0d2e9d6164a3c5b3e767a670b889191ef9761d82f - After this 200k staking, the dashboard said i have staked 66%, This looks all good.
After couple of days, I later staked additional remaining 100k keep. Etherscan txn -
https://ropsten.etherscan.io/tx/0xeed3e07c171a19f6a0efa0213032f737ff66420610dab6931958a81e5f44ca4b - this is bit funky to me. Token dashboard everywhere still says - I've only staked 66% /200k and not 300k.
My available keep is also 0 which means the 100k is staked but not showing in dashboard.
screenshots below showing only 200k staked.

Screenshots showing I have no available keep.

Steps to reproduce the behavior:

Initially stake 100k to 200k KEEP in token dashboard.
Add additional stake of 100k KEEP.

Expected behavior:
I expected it to show me that I've 300k (100%) staked in dashboard.
Environment details:
OS: Windows 10 Enterprise, 64 bit
Web Browser: Google Chrome, Version 84.0.4147.125 (Official Build) (64-bit)
Wallet: Metamask (Chrome extension), Version 8.0.7

Thanks for reporting it. I'll transfer this issue to keep-core, that's where KEEP token dashboard code lives.

Thanks for posting an issue!
I think you should also commit a top-up. Could you please check if you have any records in the Available top-ups on the Delegate Tokens page?

yes - I did indeed have the 100k in the topup and after commiting it, it shows the new KEEP stake. this is good.
One suggestion though - it was not clear to me until you pointed it out saying I have to do a top up step to complete the staking process. can we have some comment or pop up during staking process to notify the user to do this step. If not, it can get confusing.
Thanks !

Improved in https://github.com/keep-network/keep-core/pull/2434 and https://github.com/keep-network/keep-core/pull/2307.


nm, ovs: Properly delete ovs-port profiles

When an interface that is connected to an OVS bridge is removed, there
is a need to remove also the ovs-port profile which implicitly exists.
This change removes the ovs-port profile toghether with the enslaved
interface.
This PR replaces #511 .

There still seem to be a problem left.
When deleting the OVS bridge, the port device is also removed, but not the profile.
We need to support the deletion of an inactive profile for such cases.

The only test that did not ran yet is the py37 unit test.
This is passing locally and the py36 and py38 are passing, therefore, merging as is to save time.


instance.container is not a function

I'm trying to upgrade to Ember 2.5 from 2.0 and getting this: TypeError: instance.container is not a function
Sample initializer:
App.initializer({
name: 'appInitializers',
initialize: function(app) {
app.inject('controller', 'test', 'test:main');
app.inject('component', 'test', 'test:main');
app.inject('route', 'test', 'test:main');
app.inject('adapter', 'test', 'test:main');
app.inject('serializer', 'test', 'test:main');
},
});
wondering what I'm doing wrong. Thanks for any help.

@vivian-ho hi thanks for raising, appears to be linked to #33. I'll let you know when I've investigated.

@vivian-ho Hi Vivian, what version of ember-cli-bugsnag are you using?
It looks like this should not be a problem since 0.1.1 (see #35).
https://github.com/twokul/ember-cli-bugsnag/pull/35
Can you confirm that you are seeing this on v0.1.5, you may need to delete your node_modules folder  and clear your cache with npm.

I'm not using ember-cli-bugsnag.  And I deleted node_modules and cleared the cache every time there's a change in the package.json.  If this is not related to ember-cli we can close the case.  Thanks for your investigation.

@vivian-ho are you using something else instead?

Not sure what caused it.

Ok thanks for checking in 👍


Make FlexibleSpaceBar title offset more flexible.

Customize left padding of FlexibleSpaceBar title by titleStartOffset parameter.

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.
What to do if you already signed the CLA
Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.

I signed it!

Hi @showang , I am also looking to implement such change, but as I see, now your PR has a merge issue, can you check it and update it? I didn't notice your PR first so I implemented it myself also, but would make sense if you still plan to go forward, I will close my PR in favor of yours.

First of all: I apologize for how long it's taken to get to this PR. It looks like it's been about 4 months which is a record, but not a good one.
I've moved the change to a new PR: https://github.com/flutter/flutter/pull/27623 which accomplishes the same thing in terms of a newtitlePadding parameter.  I'm closing this PR in favor of the new one. With any luck the new one will land later this week.
So thank you for the contribution and, again, sorry that it's taken so long to move it along.


'NoneType' object has no attribute 'train'

I met the error 'NoneType' object has no attribute 'train' when I run this command "python tools/run_net.py --config-file ./projects/ngp_stylization/ngp_pair_base.py".
I've already had nvcc 11.5, g++ 10.4 and full requirements.
My GPU is RTX4090.
Please Help me !
Thanks.

Looks like the dataset didn't load successfully


fix: increase ec2 resource detector timeout

Fixes #708

Thanks for the fix! 🙌 any estimate on when a new version of the package will be released?


NPE while preparing racksandhost detail through RackawareEnsemblePlacementPolicyImpl

Describe the bug
We are trying to configure rack-aware policy, when starting pulsar we get this error at RackAwareEnsemblePlacementPolicyImpl and then it end up using default-rack as per the logs.
18:10:36.083 [pulsar-ordered-OrderedExecutor-0-0] ERROR org.apache.bookkeeper.client.RackawareEnsemblePlacementPolicyImpl - Failed to initialize DNS Resolver org.apache.pulsar.zookeeper.ZkBookieRackAffinityMapping, used default subnet resolver : java.lang.RuntimeException: java.lang.NullPointerException java.lang.NullPointerException
18:10:36.087 [pulsar-ordered-OrderedExecutor-0-0] INFO  org.apache.bookkeeper.client.RackawareEnsemblePlacementPolicyImpl - Initialize rackaware ensemble placement policy @ <Bookie:<IP_ADDRESS>:0> @ /default-rack : org.apache.bookkeeper.client.TopologyAwareEnsemblePlacementPolicy$DefaultResolver.

To Reproduce
Steps to reproduce the behavior:

Enable bookkeeperClientRackawarePolicyEnabled in pulsar broker.conf
Configure ensemblePlacementPolicy=org.apache.bookkeeper.client.RackawareEnsemblePlacementPolicy in bookkeeper.conf
configure racks
start zookeeper, bookkeeper  & pulsar

Expected behavior
Expected RackawareEnsemblePlacementPolicyImpl to initialise racks with given rack info and its bookies as per the implementation of ZKBookieRackAffinityMapping.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: iOS
Pulsar 2.8.0
Bookkeeper 4.14.1

Additional context
While troubleshooting we found, during initialisation of RackAwareEnsemblePlacementPolicy it creates new instance of dnsResolver as below at line https://github.com/apache/bookkeeper/blob/97818f5123999396e66f5246420d3c7e3d25f53d/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/RackawareEnsemblePlacementPolicyImpl.java#L262
            dnsResolver = ReflectionUtils.newInstance(dnsResolverName, DNSToSwitchMapping.class);
            if (dnsResolver instanceof Configurable) {
                ((Configurable) dnsResolver).setConf(conf);
            }

setConf is then triggered on ZKBookieRackAffinityMapping which throws NullPointer at https://github.com/apache/pulsar/blob/89ac98e4af363b09f2fe8e309539b0e35243aaee/pulsar-zookeeper-utils/src/main/java/org/apache/pulsar/zookeeper/ZkBookieRackAffinityMapping.java#L97 while prepare rack with its host information.

Same proble in https://github.com/apache/bookkeeper/pull/2788.

Can be fixed by https://github.com/apache/bookkeeper/pull/2788.
If it's still relevant, feel free to open a new issue.


RDBES code review: RS_AggregationLevel

RS_AggregationLevel: https://vocab.ices.dk/?ref=1607
Could this be implemeted to AGTYP (https://vocab.ices.dk/?ref=178) ?

A more elaborated CodeType description is needed, and the code descriptions should be changed to "By haul" and "By trip".
The RS should be removed.

Do not merge with AGTYP.


field multi-validation conditions

Hi,
I was very impressed how fast you guys are implementing new stuff and therefore I have question about validating with conditions (maybe I missed it in feature list, or maybe you already have it on your plans).
I have message like this:
message GetDataRequest {
  string version = 1 [
    (validate.rules).string = {
      min_len: 1
      max_len: 10
      pattern: "^([A-Z]{5}-*)$"
      not_in: ["abc", "qwe", "asdf"]
    }
  ];
}

and I can do multi-validations without any problems, this works just fine.
But is it possible to write rules that conjuncted with "OR" and not "AND"?
something like this:
message GetDataRequest {
  string version = 1 [
    (validate.rules).string = {
      OR: {
        AND: {
          max_len: 10, pattern: "^([A-Z]{5}-*)$"
        }
        AND: {
          min_len: 11, pattern: "^([A-Z]{7}-*)$"
        }
      }
    }
  ];
}

where it would allow 1 of 2 cases (if given version is less then 10 it should match one pattern and if given version is longer then 10 it should match another pattern).

This isn't possible without using some kind of more expressive language. See discussion on #128.


[SPARK-30381][ML] Refactor GBT to reuse treePoints for all trees

What changes were proposed in this pull request?
Make GBT reuse splits for all trees:
1, reuse splits/treePoints for all trees:
existing impl will find feature splits and transform input vectors to treePoints for each tree; while other famous impls like XGBoost/lightGBM will build a global splits/binned features and reuse them for all trees;
Note: the sampling rate in existing impl to build splits is not the param subsamplingRate but the output of RandomForest.samplesFractionForFindSplits which depends on maxBins and numExamples.
Note II: Existing impl do not guarantee that splits among iteration are the same, so this may cause difference in convergence.
2, do not cache input vectors:
existing impl will cached the input twice: 1,input: RDD[Instance] is used to compute/update prediction and errors; 2, at each iteration, input is transformed to bagged points, the bagged points will be cached during this iteration;
In this PR,input: RDD[Instance] is no longer cached, since it is only used three times: 1, compute metadata; 2, find splits; 3, transformed to treeePoints;
Instead, the treePoints RDD[TreePoint] is cached, at each iter, it is convert to bagged points by attach extra labelWithCounts: RDD[(Double, Int)] containing residuals/sampleCount information, this rdd is relative small (like cached norms in KMeans);
To compute/update prediction and errors, new prediction method on binned features are added in Node
Why are the changes needed?
for perfermance improvement:
1,40%~50% faster than existing impl
2,save 30%~50% RAM
Does this PR introduce any user-facing change?
No
How was this patch tested?
existing testsuites & several manual tests in REPL

Test build #116152 has started for PR 27103 at commit f7c5476.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20943/
Test PASSed.

testcode:
import org.apache.spark.ml.regression._
import org.apache.spark.storage.StorageLevel

var df = spark.read.format("libsvm").load("/data1/Datasets/a9a/a9a")

(0 until 8).foreach{ _ => df = df.union(df) }
df.persist(StorageLevel.MEMORY_AND_DISK)

df.count
df.count
df.count

val gbt = new GBTRegressor().setMaxIter(10)
val gbtm = gbt.fit(df)

val start = System.currentTimeMillis; val gbtm = gbt.fit(df); val end = System.currentTimeMillis; end - start

gbtm.evaluateEachIteration(df, "squared")

val gbt2 = new GBTRegressor().setMaxIter(10).setSubsamplingRate(0.8)

val start = System.currentTimeMillis; val gbtm2 = gbt2.fit(df); val end = System.currentTimeMillis; end - start

gbtm2.evaluateEachIteration(df, "squared")

result:
about 48% faster than existing impl

dur_gbt(new)
dur_gbt2(new)
dur_gbt(old)
dur_gbt2(old)

197777
188205
133214
134787

loss(new)
loss2(new)
loss(old)
loss2(old)

0.4283679582338368
0.42678864636469305
0.4283679582338368
0.42678864636469305

The convergences of gbt should be the same, since splits are always built on the whole dataset and should be the same.
The convergences of gbt2 do not have to be the same. However in above tests they happen to be the same, maybe due to that the input df is generated by repeate a9a 1024 times.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20945/
Test PASSed.

Test build #116155 has started for PR 27103 at commit 04bdf37.

Test build #116152 has finished for PR 27103 at commit f7c5476.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/116152/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20948/
Test PASSed.

Test build #116157 has started for PR 27103 at commit 092e927.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20949/
Test PASSed.

Test build #116158 has started for PR 27103 at commit 0b194eb.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20950/
Test PASSed.

Test build #116155 has finished for PR 27103 at commit 04bdf37.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/116155/
Test PASSed.

Test build #116157 has finished for PR 27103 at commit 092e927.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/116157/
Test PASSed.

Test build #116158 has finished for PR 27103 at commit 0b194eb.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/116158/
Test PASSed.

Test build #116203 has started for PR 27103 at commit f99dd66.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20995/
Test PASSed.

Test build #116203 has finished for PR 27103 at commit f99dd66.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/116203/
Test PASSed.

Test build #116238 has started for PR 27103 at commit 26feb5a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/21030/
Test PASSed.

Test build #116238 has finished for PR 27103 at commit 26feb5a.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
    logDebug(s\"isMulticlass = $
    logDebug(s\"isMulticlass = $

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/116238/
Test PASSed.

Merged to master, thanks @srowen for reviewing!


The procedure for sending the encrypted files

The GitHub page says, "Completed applications (please encrypt before transmission) should be sent to<EMAIL_ADDRESS>How do we send the passwords? Should we send the encrypted documents and then the passwords in a separate email? Or can we send the encrypted PDFs and passwords in the same email?
Thank you.

Hello @czopluoglu - yes, keep it simple and send the passwords in a subsequent email.   Easiest if you encrypt from a zip application, so that you only need to encrypt one time (and the same on this side).  Given that there's no student/classified information, this is just an added measure of protection for your benefit and good practices.


.useQuery of the same tRPC procedure share the same key even if the transform is different

The issue
As the useAsyncData key defined by trpc-nuxt useQuery is only based on the procedure path and params, the query returns the same result across pages no matter if there is a transform applied or not.
Reproduction
Suppose you have two pages pages/a.vue and page/b.vue:
pages/a.vue:
<template>

    {{ todoList }}
    <NuxtLink to="/b">Go to b</NuxtLink>

</template>

pages/b.vue:
<template>

    {{ data }}
    <NuxtLink to="/a">Go to a</NuxtLink>

</template>

When navigating from /a to /b, the result is reused as the two queries share the same key.
NOTE: It may not be helpful for solving this specific issue, but somehow, when navigating from /a to /b, the error I got was raised in page a stating that the computed value todoList raised an error because data is a string and not an array (so the result of the query in page b). It is quite mysterious, but it may be due to the way Nuxt handle data fetching cache and navigation. 🤔 I would have rather expected a bug such as page b displaying the raw data from page a in the
 Anyway....
Possible solution
There may be other solutions, but I would propose to allow a parameter in useQuery that would be a key suffix. This way, pages/b.vue could be :
  const { data } = await useTrpcClient().todo.getText.useQuery('joined', undefined, {
    transform: data => data.map(({ label }) => label).join(', ')
  })

so it does not share the same key with pages/a.vue.
Otherwise, exposing the key param from useAsyncData would be enough to allow to fix this issue.

Hey, starting with v0.10.9 you can pass a custom queryKey option that overrides the generated key:
const { $client } = useNuxtApp()

$client.todo.getTodos.useQuery(undefined, {
  queryKey: 'some-key'
})

Feel free to open another issue if needed!


feat: redesign linear elements 🎉

https://user-images.githubusercontent.com/11256141/182552158-5dd836f7-da70-4dc4-a599-937da3193c80.mp4

Looking good!

[ ] It seems you've already added handlers for when hovering over the arrow points. I think we should add a hover state for the background as well, which will require adding it to appState in some form.
[ ] the line editor is visibly identical right now. Not sure what we wanna do about it long term, but we should differentiate somewhat. Maybe a different colors + size (slightly bigger?) for the line points when inside the editor?

Tweaked the design for hover and editing point states
https://user-images.githubusercontent.com/11256141/182328161-687bfd08-5be6-4736-82d6-cc1125d1752f.mp4

Tweaked the design for hover and editing point states
arrow-redesign.mp4

Looks good! Or at least on brand (I'm sure @maielo won't like the big size still 😅 but we can tweak later).

some UX issues:

[ ] clicking on the point shouldn't remove the hover effect

[ ] there's a flicker of the hovered point on pointerup (in the gif below it sometimes doesn't show for some reason — frame skip or something)

[ ] after dragging on pointerup, and moving the cursor away, the hover effect remains

[ ] box-selecting a single linear element should set the state.selectedLinearElement
[ ] not necessarily a bug, but when the element is already selected without state.selectedLinearElement (which shouldn't happen after we fix all the bugs, but you can never be sure), doing some actions such as opening a contextmenu on it doesn't set the state.selectedLinearElement (which happens if you open contextmenu while the element isn't already selected). May affect other cases.
[ ] I'm thinking that opening a contextmenu on a locked element (or locking it) shouldn't show the points since 1) the element isn't editable 2) it's not super noticeable. I'd consider rendering the normal bounding box when the element is locked.

not necessarily a bug, but when the element is already selected without state.selectedLinearElement (which shouldn't happen after we fix all the bugs, but you can never be sure), doing some actions such as opening a contextmenu on it doesn't set the state.selectedLinearElement (which happens if you open contextmenu while the element isn't already selected). May affect other cases.

Setting selectedLinearElement when opening context menu was fixed yesterday, its not working for you ?

I'm thinking that opening a contextmenu on a locked element (or locking it) shouldn't show the points since 1) the element isn't editable 2) it's not super noticeable. I'd consider rendering the normal bounding box when the element is locked.

Showing bounding box is confusing since we are not showing it for arrows at all, I think its better not to show anything instead.
Even showing bounding box for other shapes doesn't help in IMO, I would rather want to improve the locking behaviour overall and may be show a lock icon when locked elements are clicked which user can use to unlock, but thats a separate discussion

Setting selectedLinearElement when opening context menu was fixed yesterday, its not working for you ?

I meant when the element is already selected without selectedLinearElement set. It shouldn't happen, but it can, so opening a contextmenu on an arrow should make sure it's set. That said, I guess we can ignore this for now and handle it during the selection handling rewrite.

Showing bounding box is confusing since we are not showing it for arrows at all, I think its better not to show anything instead (pushed this change). Even showing bounding box for other shapes doesn't help in IMO, I would rather want to improve the locking behaviour overall and may be show a lock icon when locked elements are clicked which user can use to unlock, but thats a separate discussion

We're showing arrow's bounding box when multiple elements selected so why is bad to show it when a locked arrow is selected?
Whatever we end up doing with locking is unrelated, as we will still not want to show the non-editable points when a locked element is selected.

One other case I found:

[ ] select 2 elements and then deselect the 2nd one, leaving the arrow selected. selectedLinearElement state isn't set.


refactor(pgs): more lax csp

After discussing with @antoniomika we decided to keep as-is.  From IRC:

antoniomika: Currently we have it set where if you're using a custom domain, the CSP doesn't get set. Which honestly, I think is the right way of doing this. That way, anyone using a pico domain has to abide by a strict csp (which would prevent people using us directly for scripting), but they can with their own domain.


Add a textobject for LaTeX objects

Expressions are poorly defined, but some delimiters:

=
\\
\item
\begin{...}
\end{...}
&
%
\label

Closed in a0a245a.


The next up-coming days of this website. (ATP Website/Backup Plan)

Since we are gonna be very busy from March 28th to April 1st, and then again from April 10 to 16th, lack of updates (or bug fixes, background updates, display updates, and responding to our support email<EMAIL_ADDRESS>) will occur in those days. Bugs are recommended to be reported to our support email so we can try to fix them once we are available. The website will still be up during these times, thank you for your patience.
-The AT Products Team.

Note that this backup plan will be used for any dates where we can't be available due to any reason at all. This note for these dates will be closed after April 16th. The next dates will use a different format, like Dates, why, and what will still be up. Once again, thank you for your patience.

March 28th is tomorrow, meaning that we will hold development until April 1st. The support email<EMAIL_ADDRESS>will still be up though there may be a huge delay in replies.

For ProtDos or CodingHome, please contact<EMAIL_ADDRESS>for support during this week.

This backup plan is closed, even during 4-10 to 4-16. Once again, thank you for your patience.


add support for Oracle Cloud Storage

What would you like to be added:
support for Oracle Cloud Storage
Why is this needed:
Oracle Cloud has a free-tier plan that comes with 10 GB of alway-free Object Storage. It's an excellent option for people want to try JuiceFS on the cloud.

Hi @tigerinus, it seems like the API of Oracle Cloud Storage is S3 compatible, you could use S3 storage type directly.

@xiaogaozi - That sounds great. But how exactly do I choose in the JuiceFS console in order to get integrated with Oracle Cloud?

The issue of JuiceFS console belongs to JuiceFS Hosted Service, not the community version of JuiceFS.
I'll close this issue and create an internal issue for JuiceFS console, thanks for your feedback.

Based on https://docs.oracle.com/en-us/iaas/Content/Object/Tasks/s3compatibleapi.htm, we still need to parse the region from endpoint for oracle object storage.
We can support it using S3 SDK with some parsing rules.


latest version can't be used

Issue details

go get latest version failed
Steps to reproduce

go get github.com/pulumi/pulumi-alicloud/sdk/v3@latest

Expected: go get success
Actual: go<EMAIL_ADDRESS>reading github.com/pulumi/pulumi-alicloud/sdk/sdk/go.mod at revision sdk/v3.18.0: unknown revision sdk/v3.18.0
Possible cause

@kordenlu Thanks for reporting this. The issue was due to a bug in our GH actions. The fix has been merged here: https://github.com/pulumi/pulumi-alicloud/pull/257
I've also fixed the tags on 3.17.0 and 3.18.0.


Consolidate Chargeback.initialize

What
Put all the initialization of Chargeback instance into one method.
Why?
Previously the initialization was indirect. We created hashes and then we passed them to ChargebackX.new. That was harded to comprehend by new comers. It was written this way as intialization depends on @options, and we did not have it passed to the instance.
However, main reason i am doing this is that I need @options in the instance for other reason. Thus, once we have @options in the instance, there is no excuse for the indirection. Thus, as I need @options in the instance, I drop the indirection as well.
Also, previously we calculated extra_fields together with key. Potentially dropping unneeded extra_fields.
@miq-bot add_label chargeback, refactoring, euwe/no
@miq-bot assign @gtanzillo

:+1: LGTM

Backported to Euwe via https://github.com/ManageIQ/manageiq/pull/13419


Should kill all processes even though bundle path has been removed

What happened?
When container exits，we should kill all processes even though bundle path has been removed, otherwise maybe it will cause process leak.
async fn should_kill_all_on_exit(bundle_path: &str) -> bool {
    match read_spec(bundle_path).await {
        Ok(spec) => has_shared_pid_namespace(&spec),
        Err(e) => {
            error!(
                "failed to read spec when call should_kill_all_on_exit: {}",
                e
            );
            false
        }
    }
}

containerd shim: https://github.com/containerd/containerd/commit/a687d3a36dac0c75adcef6318a1891034b4cc284#diff-3cd530e1b2d5464c8186bf3bf63e8be48df4107e853ee107751f862c7b503658R549-R554
What did you expect to happen?
When the bundle path has been removed concurrently，we must guarantee other processes can still been killed.
How can we reproduce it (as minimally and precisely as possible)?
create pod and delete it continuously,  sometimes we will find process leak.
Anything else we need to know?
No response
Dev environment
No response

Yes. Would you like to fix it?

@Burning1020 have submitted a pr


Updated readme github buttons

Added a 'open in cloudshell' feature into the Readme

build this please

Why is CI falling?

test this please

@swathiswaminathan please rebase

Rebased. please test this.

/lgtm

@maxwelllincoln you need to make sure the person does a rebase and not a merge. Please and thank you

You get stuff like this
https://github.com/GoogleCloudPlatform/gke-enterprise-demo/commit/511981144d3aa4a5bcc44a552f0b69d92d79847c


max_line_length property

The max_line_length property would be nice.

+1

+1

this would help?
https://github.com/mustardamus/line-length-break

After thinking a little bit about its implementation i wonder how this should be implemented?
The editorconfig claims:

[max_line_length f]orces hard line wrapping after the amount of characters specified

This means at any point in the editing process the editorconfig-plugin should insert a hard-wrap? I currently can't imagine that this might be any helpful, since it takes a additional operation (deleting the inserted newline) to fix a wrong wrap.
I don't know any file format which would tolerate any random hard-wraps.
The second question would be when should the break being introduced, at save of the document or on the fly, while editing the document?
Even though i currently don't see any practical use of a hard wrap over a soft wrap, and i'd expect this rule to be detected by a linter or being done by a "smart" plugin estimating the correct position of a hard wrap i expect to implement this in a few weeks, in order to fulfill the editorconfig-specs.

@florianb Isn't this the equivalent of setting:
atom.config.set('editor.preferredLineLength', <max_line_length>)
atom.config.set('editor.softWrapAtPreferredLineLength', true)

At least according to this page a soft wrap at the set value is acceptable, it looks like the other page only lists a hard wrap as acceptable though, so it seems there is some confusion there.
Personally I would say the forced soft wrap is the more reasonable choice, for the difficulties and potentials to introduce confusion that you outline above for forcing an arbitrary line break at the set value.

@Arcanemagus - thank you very much, the link is very helpful. These config-settings seem to be the responsible settings (i had them in mind, too) - i have to check how they are implemented and how we may realize a per editor-config. 🍻

max_line_length

🎺 The implementation of max_line_length is pending. I would appreciate any help testing it before we're releasing it: #123 (instructions can be found in the PR-description).

The current implementation of max_line_length is questioned by a raising number of users - i appreciate any contribution to #143.


Non-nullable shared_ptr

What
Use shared_ptrs that are not nullable, where appropriate.
Why
Segfaults. See GSL for docs/implementation.

The fact that we're aliasing all the shared pointers means that introducing this would not neccesarily require a rewrite of plugins, so doing this post-v1 is still feasible. - Dr David Feltell.


supportsSession() -> isSessionSupported()

Posted in anticipation of https://github.com/immersive-web/webxr/pull/841 landing. I'll take this out of draft once that happens.
Changes navigator.xr.supportsSession() to navigator.xr.isSessionSupported() and made the method resolve with a boolean to indicate success or failure. This matches the spec change made to do the same in response to feedback from the TAG.
This PR also introduces a small shim that assists developers using the previous signature by redirecting the call to the right place and outputting a warning to the console informing them of the new method name and behavior. That can eventually be removed once the API has been out for a bit.

LGTM FWIW

Likely when does Chrome Canary switch from .supportsSession() to .isSessionSupported()? Until it does the user code may need to be aware of the polyfill like
if (navigator.xr.isSessionSupported) {
  ...
} else if (navigator.xr.supportsSession) {
  ...
}

Chrome canary should already have the new method, but we're also keeping the old function around for a little while with a console message warning developers that it's been deprecated and what they should replace it with.

Thanks for the comment! That sounds nice.


simpletext draw vertical scoll (test)

test

Nice 👍

YesOn Mar 20, 2017 01:14, Alexandre Janniaux<EMAIL_ADDRESS>wrote:@alexandre-janniaux commented on this pull request.
In src/Nazara/Utility/SimpleTextDrawer.cpp:

	m_bounds.Set(Rectf(std::floor(m_workingBounds.x), std::floor(m_workingBounds.y), std::ceil(m_workingBounds.width), std::ceil(m_workingBounds.height)));

   if (reupdate)

   {

       index += m_lines.front().glyphCount;

       ClearGlyphs();

       GenerateGlyphs(m_text, index);

       std::cout << "\nREUPDATE : " << index;

It seems strange to have an utility class which prints things, doesn't it ?
—You are receiving this because you commented.Reply to this email directly, view it on GitHub, or mute the thread.


Language injection: Python

Hey :)
I am not even sure if this is the correct place to ask for this but in lack of a better idea I will try :)
I want to use python code inside my latex using the pythontex package (see here for example).
Should this work out of the box?
Is it something that should be incorporated into the latex grammar?
I use tree-sitter through neovims nvim-treesitter plugin.
Ty for any help.

Thanks for the question. The language injection queries are not located inside this repository. Instead, they are part of the nvim-treesitter repository: https://github.com/nvim-treesitter/nvim-treesitter/tree/master/queries/latex
I will push a new version of the grammar soon, which will handle verbatim content correctly (this requires an external scanner). Afterwards, the language injection queries need to be updated to account for the common locations that contain code written in other languages (lstlisting, minted, pythontex, ...).

Thank you very much for your answer. I will try to sum up what I got from it.
I checked the injections file in the nvim-treesitter repository. Its only line is about comments. For my use case there should be a line/section concerning python code injection.
You will update the grammar provided by this repository.
Did you plan that update independent of my issue?
Does it make sense to open an issue in nvim-treesitter before you updated the grammar?

Did you plan that update independent of my issue?

Yes, there is some code on the develop branch: https://github.com/latex-lsp/tree-sitter-latex/tree/develop
There is one bug that needs to be fixed with the WASM target (somehow the external scanner crashes in the WASM runtime; otherwise, it works just fine). Once that issue is sorted out and the latest contributions are integrated, I will merge it into master.

Does it make sense to open an issue in nvim-treesitter before you updated the grammar?

There will be quite a lot of changes so I recommend waiting a little bit so there is no duplicate work.

got it. Thank you very much!

Sorry for the long wait. The new grammar has just been merged to the master branch. If there are any issues with language injection, please open a new issue.

Hey thank you very much for letting me know about the update.
Syntax highlighting for my injected python code still does not work but I didn't expect that anyway.
For what I understand there is still the need to update https://github.com/nvim-treesitter/nvim-treesitter/blob/master/queries/latex/injections.scm.
I have read a minimal amount of tree-sitter documentation here
https://tree-sitter.github.io/tree-sitter/using-parsers#pattern-matching-with-queries
and here
https://tree-sitter.github.io/tree-sitter/creating-parsers#the-grammar-dsl
Your grammar mentions the pycode environment of pythontex explicitly in
https://github.com/latex-lsp/tree-sitter-latex/blob/c454b3207f0e5d8a6fb82bcedcaf1db07a805024/grammar.js#L474-L499
so an injection rule in the nvim-treesitter repo would need to use those definitions?

Note that the new parser is a breaking update; it will take time to get nvim-treesitter updated to use it. (Right now it's still locked at the old version, so no surprise that it isn't working.)

Yes. Someone has to sit down and Do The Work.

@clason  Wow! I was just blown away by the fact that you are Cristian Clason ;)
I am currently reading your articles together with T. Valkonen on generalized PDPS algorithms.
I would really like to "Do The Work" but I am afraid it would take me very long as I still have very limited knowledge of tree-sitter. I would have tried to update a single injection file with a small query. but overhauling the complete queries seems to be more than I can handle atm.
Anyway, thank you once more for your answers. Really made my day to have met you at Github!

What a small world :) I'm happy you find the articles interesting!
(I'm trying to do the work, but my knowledge of tree-sitter is quite limited as well, so it's slow going.)


Add Incident tab on entity page & list incidents on lineage entity side pane

Incident should be overlayed in card overview as well, similar to how we are show DQ tests cc @karanh37

@karanh37 have we addressed this in lineage view?

No, this is still pending.


Need a ListAtists Command

There is no way to list the artists in the database so you can present the user with a list of artist to choose from

Maybe  The music database -> count group artist?
Or  The music database -> list {TYPE} {FILTER} [group {GROUPTYPE}], like list album group albumartist?

list command   list artist delivers a list of the artists.
or just expose the list command without any forced searches in it.
On Mon, Aug 7, 2023 at 12:24 PM Rafael Estevam @.***>
wrote:

I peeked at the MPD protocol documentation
https://mpd.readthedocs.io/en/latest/protocol.html
And did not find any command for list artists
I found only stats -> artists
Which command you refer to?
—
Reply to this email directly, view it on GitHub
https://github.com/RafaelEstevamReis/SimpleMPD/issues/1#issuecomment-1668199902,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACCB6YPVMSXTLXR7RNI7MKTXUEJEPANCNFSM6AAAAAA3HE4REE
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'll try some tests when I got home

or just expose the list command
Is better to expose the "list" (or GroupCount)

But I'll try booth then we test it

I cannot test (yet - still at work) but can you take a look at these (4ad15f2) changes?

I managed to implement:
List(expression)
and
List(Tags.Artist)
But any of the filters worked (like I did for search and Find)
Check it out d4ef23c

This is awesome thank you.    I have been traveling so I will start using this as soon as possible!  Thank you very much!


rollapply for large data using sparklyr

Can you please help with this problem:
https://stackoverflow.com/questions/46024271/rollapply-for-large-data-using-sparklyr

Followed up on https://stackoverflow.com/questions/46024271/rollapply-for-large-data-using-sparklyr/46064211#46064211, closing this one.

@jairajgupta if your VaR calculation is univariate in the sense that you're calculating it for each symbol, you may be able to use spark_apply() assuming that you can collect each symbol's data onto one node. Here's a snippet (not working yet) that should get you started:
var_sp <- lmanagers_sp %>%
  spark_apply(function(df) {
    require(dplyr)
    df %>%
      arrange(variable, date) %>%
      mutate(var=zoo::rollapply(
        value, 10,
        FUN=function(x) PerformanceAnalytics::VaR(x, p=.95, method="modified",align = "right"), partial=T))
  }, group_by = "variable")


VS Code Multi-root Capabile Setting Sync

Hi,
I hope all is going well people love your extension and it fills a key gap :)
We had a quick glance at your code and it looks like it's multi-root ready with no additional changes.  But given the polarity of the extension (the the fact I keep recommending it as @code :)) I'd love you to check as well (details in this guide).

Request: if you agree it's all looking good we would love you to add multi-root ready to your package.json i.e. "keywords": ["multi-root ready"].  If you upgrade to VSCE 1.32.0 this can be added to any existing keywords and will not be counted against the maximum of 5 allowed keywords.

Happy Coding!
Sean
VS Code Team
cc: @bpasero

Sean,

it's multi-root ready with no additional changes

Thanks !
I will release new version in upcoming days with this keyword.
Can you tell me why this keyword is required for new extensions to work.


Patch for feature request #88

Here is a backward compatible patch for the feature request #88.
A small test case can be found here: org.dozer.functional_tests.CollectErrorsTest

@ArtunSubasi ; can you rebase with latest to force travis to retest

@ArtunSubasi ; any update?

Closing due to inactivity.


docs(main): support arm64 release docs

@utam0k   In fact, I have an idea. The document does not need to be changed. When we officially release it, can we use the robot to automatically submit the commit without having to modify the document every time?

@utam0k In fact, I have an idea. The document does not need to be changed. When we officially release it, can we use the robot to automatically submit the commit without having to modify the document every time?
using GitHub Action: peter-evans/create-pull-request@v5

I appreciate your suggestion 🙏 But how about this in the case of this PR?
https://github.com/containers/youki/pull/2510/files#r1391017650

Codecov Report

Merging #2510 (357c512) into main (9ae5660) will not change coverage.
Report is 9 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2510   +/-   ##
=======================================
  Coverage   65.87%   65.87%
=======================================
  Files         133      133
  Lines       16868    16868
=======================================
  Hits        11111    11111
  Misses       5757     5757

@utam0k In fact, I have an idea. The document does not need to be changed. When we officially release it, can we use the robot to automatically submit the commit without having to modify the document every time?
using GitHub Action: peter-evans/create-pull-request@v5

I appreciate your suggestion 🙏 But how about this in the case of this PR? I'd like to avoid the increase of load for the maintainers. https://github.com/containers/youki/pull/2510/files#r1391017650

https://github.com/cuisongliu/youki/pull/42 This PR is my release test.
GitHub Action automatically generates generator documentation when a new version is released.

But this PR depends on https://github.com/containers/youki/pull/2498

https://github.com/cuisongliu/youki/pull/42 This PR is my release test.

GitHub Action automatically generates generator documentation when a new version is released.
It looks great! May I ask you to provide the workflow?

cuisongliu#42 This PR is my release test.

GitHub Action automatically generates generator documentation when a new version is released.

It looks great! May I ask you to provide the workflow?

This PR already includes the submission of the workflow.   https://github.com/containers/youki/pull/2510/files#diff-87db21a973eed4fef5f32b267aa60fcee5cbdf03c67fafdc2a9b553bb0b15f34R98

this pr is ready @utam0k

https://github.com/cuisongliu/youki/actions/runs/6925964479
https://github.com/cuisongliu/youki/releases
auto update version is ready @utam0k

Manually trigger CI and automatically submit PR

release and create a tag


reflect: fix deep equal bug on comparing func types

deepEqual function only compare nil values for func types, need also compare func pointer address.
bug example:
https://play.golang.org/p/TKKxpYIzs9W

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.
What to do if you already signed the CLA
Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.

I signed it!

the we need change the language specification, how it can be logically sound that you can do f1 := func() {}; fn2 := f1 but not f2 == f1

The language spec already covers this:

Slice, map, and function values are not comparable. However, as a special case, a slice, map, or function value may be compared to the predeclared identifier nil.

So this code won't even compile:
var a func()
var b func()
var c = a == b

deepEqual is in kind of a tough spot here, because it's being asked to do something - compare two values of function type - when the language spec declares such comparisons to be illegal. Not just having an undefined result, but it's a question you're not allowed to ask. deepEqual chose false as the answer. Another possible answer is to panic, but that has its own issues.
What would you have deepEqual do instead? Comparing the function address isn't enough - some functions might be closures.  And due to the way shared library linking is done, identical references to the same function in different packages might have different function addresses (one would be the address of a trampoline that jumps to the other).

ok, it's way more complicated. close this now.


ClangImporter: Adopt for clang change 2b4323722 and unbreak the testsuite.

This sets the NeededByPCHOrCompilationUsesPCH language option when
emitting a brigding PCH.
rdar://problem/30384801

@swift-ci please test

Background story:
In order to prevent nonrecoverable module-out-of-date errors in projects that mix PCH and clang modules, clang 2b4323722bc8896bae27d197e99efffd507c0d73 introduces a NeededByPCHOrCompilationUsesPCH language option which causes clang modules imported by PCH which goes into the module hash, thus isolating them from rebuilds triggered by other clang modules.
ClangImporter doesn't set the correct frontend action when generating the PCH (it's always fsyntax-only) so this patch works around that by manually setting NeededByPCHOrCompilationUsesPCH.

(I merged the PR just to unblock the bots).

Oh hey, sorry I missed this when it was filed. Looks good to me!


Reduce OAR staging and production ECS task count

OAR production is only serving read-only traffic to legacy embedded maps
OAR staging  is only serving read-only traffic
Recommend reducing the desired task count in both tfvars files to 8, manually editing the services through the console to run 8 tasks  and monitoring production latency to see if 8 is sufficient.

Labeled client action needed to get approval for this task.

CAN pending 2/1 sprintly meeting with OSH

@jwalgran @obrienad , given that this could significantly help our AWS cost, I'd like to up the priority. @jwalgran , can you take a screen cast of how you do this instead of scheduling paired time?

Mariel confirmed OS Hub sorted this.


Added Suleka_Helmini.md file to Students folder and added name to CONTRIBUTING.md

#1 #2

resolve conflicts plz


Add common HDL language extensions

Add support for detecting files of the common hardware description
languages of verilog and VHDL.

can you link a reference, please?


trial.sh does not support compiling with Dynamics.

Using this works for me:
~/clean/bin/clm -h 100m -s 200M -dynamics -I .  -I ~/clean/lib/Dynamics -I ~/clean/lib/StdEnv -I ~/git/FP_2011_Uitwerkingen/ -I ~/clean/lib/Gast -I ~/clean/lib/Generics -I ~/clean/lib/MersenneTwister -I ~/clean/lib/StdLib  StdDynSetTest

Didn't know "-dynamics" was a thing.

Fixed in a72cf47


Make more_core_extensions direct dependency

Since the gem's RE_EMAIL constant is used in app/models/user.rb, the
gem should be a direct dependency.
To precede https://github.com/ManageIQ/manageiq/pull/12901 so that, once a new release of the gem is cut, the version specified here can be bumped to coordinate the moving of test code from app to gem.
@Fryguy hope I am understanding this process correctly :grin:
@miq-bot assign @Fryguy

Hawkward.
I count two other places a direct require of the extensions gem is used within the application, too.

I agree with making more_core_extensions a direct depedency, but I am against the = operator.  This tight locking makes it really hard to fluidly update versions across repos.  Note that manageiq-gems-pending already has "~>3.1.0" ([1]), and I think even that is too strict considering we own that gem.  I would prefer we did "~>3.1", personally.
@jrafanie @bdunne Thoughts on the above?

@Fryguy yeah, I'm fine with that. My thoughts were that if this gets merged as is, I would effectively upgrade the version as part of https://github.com/ManageIQ/manageiq/pull/12901 once a new version was released, either by relaxing the constraint or removing it. I was trying to find some way to make it clear what was going on across repositories.


Ensure root scope types have no namespace set

We need to be stricter in ensuring that no namespace is set for root-scoped types. Otherwise, someone could make it through policy checks under a namespace at one layer, then get escalated when another layer ignores the namespace.
These three places were mishandling root-scoped objects:

GetAPIRequestInfo sets the request namespace to "default"
BeforeCreate/BeforeUpdate sets ObjectMeta.Namespace to "" but ignores a namespace set in the context
The etcd key-building functions for root-scoped objects ignore any namespace in the passed context rather than erroring

This PR:

[x] Adds error return value to root key function to allow indicating errors when the context has an invalid namespace for the type
[x] Returns errors early in BeforeCreate/BeforeUpdate when an invalid namespace is set in the context or in the object for an unnamespaced type
[x] Fixes rest tests to use generated names in minion and namespace tests (were getting AlreadyExists errors which was preventing testing the namespace validation conditions)
[x] Fixes rest test error checking
[x] Makes minion and node tests use ClusterScope()
[x] Changes GetAPIRequestInfo to stop defaulting requestInfo.Namespace to default for root-scoped types

Assigning to @lavalamp, feel free to reassign.

I can review this in the morning to save @lavalamp time.

@derekwaynecarr I need to look at the integration tests before you take a look

@derekwaynecarr rebased and updated, PTAL

I will review in morning
Sent from my iPhone

On Apr 13, 2015, at 4:26 PM, Jordan Liggitt<EMAIL_ADDRESS>wrote:
@derekwaynecarr rebased and updated, PTAL
—
Reply to this email directly or view it on GitHub.

Not a huge fan of the change to EtcdKeyRootFunc to return an error.  Feel like that may be overzealous, but I understand your desire to protect all layers.  More concerned about the namespace context when dealing with the namespace resource itself and if it impacted policy.  We can catch up tomorrow to discuss those concerns.

If EtcdKeyRootFunc wasn't used to list, I'd be less concerned.

What do we want to do with this PR?

What indeed. Once pre-v1beta3-style urls go away, I'll be a lot more comfortable without this level of checking. Do you know when we plan to remove v1beta1/2?


hi, how to render a view with a model?

server:
ctx.render( 'index', { title: 'blog', globalVariable: { body: 'bla', foot: 'bla' } } );
view:

but the result is that:
[object Object]

I think you should create an issue in the repo who provide ctx.render method.


[TextServer] Fix orphan StringNames and cleanup.

Move static feature set and number system StringNames to the singleton instance to avoid orphan StringNames.
Remove hb_ prefix from the custom bitmap font functions to avoid potential conflicts with the HarfBuzz.
Cleanup commented debug code from MSDF font implementation.
Update numbering system data to CLDR 39.

Fixes #53688

Thanks!


stream edit toggler: Update select_tab when changing hash."

Followup to #27637.
This fixes a bug in #26717 (not merged yet) where the hash would flash to #personal before #subscribers. Sometimes setup_toggler is called after the hash change, and it needs to know which tab to show.
Ideally in the future we'd clean up this code so that the hash changing function isn't called multiple times.

Merged, thanks @evykassirer!


To get RSAPublicKey from string

How to get RSAPublicKey  from a public key string?

Not part of this library for sure. But I made a gist a while ago sharing the methods we use for running the tests on this SDK. You might find it useful https://gist.github.com/lbalmaceda/9a0c7890c2965826c04119dcfb1a5469


Add system bundle for package com.sun.net.httpserver

Add system bundle for package com.sun.net.httpserver
https://docs.oracle.com/en/java/javase/11/docs/api/jdk.httpserver/com/sun/net/httpserver/package-summary.html

Hi @nedtwigg,
Do you have an ETA for this publication?
No pressure here I'd just like to now if I can wait for you to publish or if I need to add a system bundle locally in my project for now?
Anyway thanks for this project that soften one of the rough edges when building with Tycho.

I just published to bintray, and it's in their queue to be included in jcenter. Once that gets approved (normally 24 hrs, but it is NYE...) then it'll go to mavencentral next. Here's where it's living in the meantime: https://bintray.com/diffplug/opensource/extension.com.sun.net.httpserver

Released on jcenter and mavencentral. Closed by https://github.com/diffplug/osgiX/commit/473d2ebbe27b2bb3a00cab59b1a7c8c3a74225da

Released on jcenter and mavencentral. Closed by https://github.com/diffplug/osgiX/commit/473d2ebbe27b2bb3a00cab59b1a7c8c3a74225da

Thanks !!

Thanks !!


Update scalatest to 3.1.0-RC3

Updates org.scalatest:scalatest from 3.1.0-RC2 to 3.1.0-RC3.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalatest", artifactId = "scalatest" } ]

labels: semver-pre-release

Already done as part of another MR.


Allow for topical tweets and replies

It'd be neat to be able to include a prompt/subject for our tweet/reply to guide ChatGPT to write a more specific tweet.
Maybe in the tweet box, I could say something like "cheer for Argentina after their world cup victory"
This could translate in ChatGPT to something like this:

Write a tweet about "cheer for Argentina after their world cup victory"

I kinda stopped active development of this extension for now but will put it into a ToDo list.
Also, if somebody wants to do a PR for this, I'll gladly merge it.

This feature is now released with v2 version.


password entered correctly still wrong

Could be related to onedrive or maybe not

Is the machine maybe Azure AD-joined? I haven't tested it for such setup, so that might not work correctly yet. Not sure what you mean with that it could be related to OneDrive.


No good way to bind one frontend to multiple ports

Cookbook version
5.0.0.
Chef-client version
12.21.4, chef local mode (zero)
Platform Details
Ubuntu 16.04
Scenario:
Need to bind one frontend to more than one port on the same IP.
Steps to Reproduce:
bind = {
  "<IP_ADDRESS>": "80",
  "<IP_ADDRESS>": "8080",
  "<IP_ADDRESS>": "443 ssl crt /etc/haproxy/ssl/foo.pem"
}

haproxy_frontend "testing" do
    action :create
    bind bind
end

If the value of the bind property is a Hash, the key is supposed to be the IP and the value is the port (with possibly additional options, like ssl … in my case). But one IP (ie. one key) cannot be used more than once.
If haproxy should answer requests on all IPs/devices, one can go hacky like so:
bind = {
  "*": "80",
  "<IP_ADDRESS>": "443 ssl crt /etc/haproxy/ssl/foo.pem"
}

But this doesn't work for more than two binds.
Expected Result:
Maybe the value of the bind property should/could be an array as well. An array of simple strings:
bind = [
  "<IP_ADDRESS>:80",
  "<IP_ADDRESS>:8080",
  "<IP_ADDRESS>:443 ssl crt /etc/haproxy/ssl/foo.pem"
]

And then the haproxy_frontend resource should create a frontend … configuration in haproxy.cfg with multiple bind … lines.
Actual Result:

bind = {
  "<IP_ADDRESS>": "80",
  "<IP_ADDRESS>": "8080",
  "<IP_ADDRESS>": "443 ssl crt /etc/haproxy/ssl/foo.pem"
}

Result: Only one bind line is written (with port 443, as it's the last entry).

bind = [
  "<IP_ADDRESS>:80",
  "<IP_ADDRESS>:8080",
  "<IP_ADDRESS>:443 ssl crt /etc/haproxy/ssl/foo.pem"
]

Chef generates an error, as the value of the property needs to be a string or a hash.

@alexs77 would you be willing to put in a pull request?

This is getting to the point of, everything needing to be a hash. I generally dislike having haproxy listening on multiple ports for the same backend, but know it's needed in some circumstances.
If you fancy creating a PR on this one, that'd be great.

I think the solution in #268 should work


Backport wave changes

Signed-off-by: Ian Chen<EMAIL_ADDRESS>🦟 Bug fix
Summary
Backport some minor changes from #555:

removed unused variable
fixed variable names
adjusted wave amplitude and camera z pos

Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

This pull request has been mentioned on Gazebo Community. There might be relevant details there:
https://community.gazebosim.org/t/new-ignition-releases-2022-03-01-citadel-edifice-fortress/1313/1


PMM: Install PMM

https://docs.google.com/document/d/1eiK6xA7eqYmYC3sPg2FNDeHEgHiFOG6c57lkVUeKbl8/edit
https://powerofus.force.com/s/article/PMM-Install

We made inline edits and resolved all comments but one. @barbarachristensen could you check on the final comment and make a call on this one?

@coriobriensfdo  this is cleaned up and ready for you!

@coriobriensfdo this is cleaned up and ready for you!

Looks great...back to you! :)

@Billflorio want me to voice record this?


HFQ junk sell error

This and some other junk from HFQ throw errors upon pressing sell junk button.

Are you sure this happens on the latest version?
It should have been fixed 3 months ago:
https://github.com/shagu/ShaguTweaks/commit/c3e5cebc29cae4d5bf8980771e4da6a7ddaf2ed9

Uhm, I'll recheck and report.

My bad, I had older version.


Add InterceptorFactory

Interceptors are being accidentally misused by users.
The issue is that an Interceptor can be re-used between
multiple PeerConnections. Interceptors were designed to only be
single PeerConnection aware, so state is being corrupted.
Instead we are now going to provide InterceptorFactories. The default
API of pion/webrtc will now be safe to use between PeerConnections.
Relates to pion/webrtc#1956

Do people prefer that Option gets executed once per factory or interceptor?
I opted for once per Interceptor

You can customize each one
You could track interceptor creation

It could be wasteful resource wise, but I don't believe enough to matter.


gcloud option

skaffold run -p gcloud-build

LGTM

@Avi-Robusta can you add a sentence to the docs here: https://docs.robusta.dev/master/developer-guide/platform/dev-setup.html#installing-robusta-in-cluster-from-source
e.g. "You can use skaffold run -p gcloud-build for faster builds"
Then go ahead and merge?

Awesome, merged and deleted the branch.
Thanks, Avi!


checking SSLOffloading header set by environment

Fixes #2399
Summary
Checking Offloading header for popups when opening

Good point! let me try to explain it. Most important part is that both settings are dependent on Server configuration.
When Webserver configure with SSL certificate and request sends as https then ASP.net Framework pick that and give HttpRequest.IsSecureConnection true, which is good for one server configuration.
Unfortunately, when SSL offloading occurs ASP.net does'nt have any clue that this request is secure, but luckily we can set http header at load balancer end, so that when SSL offloading occurs http based request sends to backend servers with SSLOffloading header. And then any application in backend server needs to check that header to response(i.e send content) as secure request


feat(tickers.stock): use yahooquery instead of yfinance

yahooquery seems like a saner yahoo finance API interface. Unfortunately it currently doesn't support fetching pre/post market data.
I opened a PR to add it and hopefully once/if  https://github.com/dpguthrie/yahooquery/pull/281 is merged I can swap over to using yahooquery.

closes #58

yahooquery is dead, closing


Visual glitch in stacked line chart (bridges end and start points)

Version
5.1.2
Reproduction link
https://jsfiddle.net/b1kcr42a/1/
Steps to reproduce
Create a stacked chart with several data series. Sometimes there will be a visual glicth that appears to bridge the end and start points of a series in the chart. Sometime this will occur for many series.

What is expected?
The stacked chart draws without issues
What is actually happening?
A visual glitch that appears to bridge the end and start points of a series

This problem occurs in 5.1.2 but not in 5.0.0 - I havent not tested version in between.
5.1.2

5.0.0

I have now tested on 5.1.1 and the issue is not present.

The issue reported by #15372 may be the same as this issue. It seems related to ecomfe/zrender#760.

Note that, this error case needs us to specify some very small values, such as 0.1 or 0.2 in this case. If we increase it slightly, this issue will disappear.


There is an issue with the Reminder Repository

It seems that there is an issue with the conversion of the DateTime object when validating expiration date.
Below is the 'created_at' object in the reminder collection
"created_at" : { "date" : "2014-08-03 23:44:43", "timezone_type" : 3,
 "timezone" :"America/New_York"}}

and when I dump $reminder['created_at'] variable I get 2014.
		// Convert the DateTime object that got saved to MongoDB
		if (is_array($reminder['created_at']))
		{
			$reminder['created_at'] = $reminder['created_at']['date'] + $reminder['created_at']['timezone'];

            var_dump($reminder['created_at']);
		}

int(2014)

when these changes will be released


Add "type" field to feature properties

Before making a pull request

[x] Make sure all tests run
[x] Update the changelog appropriately

Description
https://internal.cida.usgs.gov/jira/browse/NHGF-82
The feature property now displays a type field to indicate the catchment association.
After making a pull request

[x] If appropriate, put the link to the PR in the JIRA ticket
[x] Assign someone to review unless the change is trivial

Do you want to have a java-knowledgeable developer look at this too? Seems pretty straight forward, but maybe I don't know what I'm looking at in some ways?

Yeah, I'll try and find a volunteer.