use one json for multi-language.

created i18n.json combining el and en json into language type. this will help feature add any other new language.
modified i18n.py to get language set first. fixed tools retrieve bug in the existing script.
added unit test case

centralise it for easy to maintenance is a good common practise. otherwise, when we add/modify a section or item, we have to go through many files. multi language files solution will quickly lead to unsync and errors in different language environment.

in one json, we can take the advantages of co-pilot or any AI to easy fill in other language based on initial like this.
"task_with_context": {
"el": "{task}\nΑυτό είναι το πλαίσιο με το οποίο εργάζεστε:\n{context}",
"en": "{task}\nThis is the context you're working with:\n{context}",
"cn": "{task}\n这是您正在使用的上下文：\n{context}",
"fr": "{task}\nVoici le contexte avec lequel vous travaillez:\n{context}",
"jp": "{task}\nこれがあなたが働いているコンテキストです：\n{context}",
"kr": "{task}\n이것은 당신이 일하고있는 문맥입니다 :\n{context}",
"ru": "{task}\nЭто контекст, с которым вы работаете:\n{context}",
"de": "{task}\nDies ist der Kontext, mit dem Sie arbeiten:\n{context}",
}

if your concern is loading i18n on every script will be too costly because of the file size. we can work out a singleton class to handle it at the app initial stage.

there are a few PR related on language setting. If we use one json file for multiply languages. this will be not a issue anymore. :)

@cove9988 If we combine all the languages into one file, wouldn't that potentially create really messy merge conflicts?

@cove9988 If we combine all the languages into one file, wouldn't that potentially create really messy merge conflicts?

No, there is no messy merge, it's clean cut with same I18N() interface. However, I noticed that we don't have .env (env.template) or other configure file for the app environment variables loading, such as OPENAI_API_KEY, DEFAULT_LANGUAGE etc.... So there is no entry point yet to change language. If you are happy, I can help on this matter as well. :)

I'm with @joaomdmoura, translations could get bulky fast

okay, still keep multi-language files with most people agreed.

use DEFAULT_LANGUAGE environment var to change language setting.
add test cases.

We are moving always from having translations within the project as it makes it hard to maintain, will add it as a attribute to the crew so people can use any language they wish

I'm bringing the tests in tho :D


Add support for ReadFrom.subnet

Feature Request
I want ReadFrom.Subnet feature which read from any node in the configured subnets.
Original motivation of this proposal is, I want to read from a (AWS elasticache) redis node in the same availability zone from an application.
Reading from the same availability zone reduces latency and costs as AWS charges traffics between different availability zones.
While AWS claims AZs are engineered to provide low latency network connectivity to other AZs in the same region (see link), a simple benchmark shows crossing availability zone leads bad latency:
$ redis-cli -h <same AZ node>  --latency
min: 0, max: 19, avg: 0.38 (2088 samples)
$ redis-cli -h <different AZ node>  --latency
min: 1, max: 21, avg: 1.21 (2061 samples)

Currently lettuce supports ReadFrom.NEAREST, a option of reading from any node with the lowest latency.
For my usecase it would work well if there is only one node in an availability zone.
However if there are more than two nodes in an availability zone, the option may lead unbalanced loads between the nodes.
Imagine that an extreme case - there are two read replicas in an availability zone and one lettuce application with NEAREST option deployed to the same availability zone with the replicas, the application always reads from one read replica and the other one is not used until the next topology refresh. (I did not test NEAREST actually, sorry if I am wrong)
I believe lettuce should be vendor-independent and avoid AWS specific APIs,  so I want to propose to use subnets to read from a node of the same availability zone.
I think we can use this feature for other clouds or on-premises.
As you know lettuce is well-designed and I can implement such ReadFromSubnet class by myself in my project, but I believe this feature will be in demand to other developers.
If my proposal seems reasonable I can submit a PR.
Is your feature request related to a problem? Please describe
Crossing availbility zone leads bad latency and costs.
Describe the solution you'd like
Add support ReadFromSubnet feature:
    static final class ReadFromSubnet extends ReadFrom {

        private final List<IpSubnetFilterRule> rules = new ArrayList<>();

        ReadFromSubnet(String... cidrNotations) {
            // ...
        }

        @Override
        public List<RedisNodeDescription> select(Nodes nodes) {
            ArrayList<RedisNodeDescription> result = new ArrayList<>();
            for (RedisNodeDescription node : nodes) {
                if (isInSubnetRange(node.getUri())) {
                    result.add(node);
                }
            }

            return result;
        }

        private boolean isInSubnetRange(RedisURI redisURI) {
            // return true if redisURI is in the subnets otherwise false.
        }
    }

To calculate subnet netty's IpSubnetFilterRule seems good as we do not need additional dependency.
Describe alternatives you've considered
AWS API like DescribeCacheClusters would work,
but we need an elephant awssdk dependency and this makes lettuce vendor locked-in.
Teachability, Documentation, Adoption, Migration Strategy
We need to add information to wiki page.
Example

https://github.com/lettuce-io/lettuce-core/pull/1570
I've just submitted a PR of this proposal.
If this proposal is nonsense, please close this issue and pull request.
(In Japan we have long new year holidays but you know, due to covid-19 I need to stay home and am really bored to stay home. Sorry for sending PR before your response. Happy new year and stay safe!)

https://github.com/lettuce-io/lettuce-core/pull/1570
I've just submitted a PR of this proposal.
If this proposal is nonsense, please close this issue and pull request.
(In Japan we have long new year holidays but you know, due to covid-19 I need to stay home and am really bored to stay home. Sorry for sending PR before your response. Happy new year and stay safe!)

Thanks a lot. It makes sense to use netty utilities to build a subnet-based ReadFrom rule. Not depending on AWS SDK makes sense as we don't want to include any cloud-vendor specific dependencies.

Thanks a lot. It makes sense to use netty utilities to build a subnet-based ReadFrom rule. Not depending on AWS SDK makes sense as we don't want to include any cloud-vendor specific dependencies.


Start-DbaMigration does not bring read only DB online on destination if it already exists

Before submitting a bug report:

[X] Ensure you are able to reproduce it on the latest released version (we release often)
[X] Verified this bug is not already reported in an issue
[X] Verified errors are not related to permissions
[X] Can reproduce in a clean PowerShell session (clean = powershell -NoProfile)

Steps to Reproduce
$startDbaMigrationSplat = @{
    Source = $sql2016
    Destination = $sql2017
    BackupRestore = $true
    NetworkShare = 'C:\temp\backups'
    NoSaRename = $true
    WithReplace = $true
    SetSourceReadOnly = $true
    Verbose = $true
}

Start-DbaMigration @startDbaMigrationSplat #| Select * | Out-GridView

Expected Behavior
All databases would be moved in a read/write state from the source to the destination.
Actual Behavior
If a database already exists on the destination instead of being overwritten and brought online during the backup/restore process it is just restored and left read only on the destination.
Environmental data

Name                           Value
----                           -----
PSVersion                      5.1.16299.431
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.16299.431
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>  ```

Before:

After:
[dbatools_support_pack_2018_05_29-07_19_20.zip](https://github.com/sqlcollaborative/dbatools/files/2048141/dbatools_support_pack_2018_05_29-07_19_20.zip)

thanks for the report @gbargsley. just tested this on my servers a couple different ways and it's working as expected with the destination being read-only if the source is read only, even when the destination  db existed. Please let me know if you have a different experience with the latest version.


refactor: re-arrange dashboard page js bundles

SUMMARY
I am trying to rearrange existing bundles needed by the dashboard page, make spa.js execute and route to the dashboard page quicker, so that it can start to fetch dashboard metadata earlier.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before:

After:

TESTING INSTRUCTIONS
CI and manual test
Note:
The perf improvement is hard to estimate, because I only run a few times, with same dashboard, from my own computer. I feel it might be highly impacted by my computer's performance. For example if i was in zoom meeting the page load will be slower :joy:
Since the change is relatively small, I hope could deploy to airbnb production environment and collect some real data. If thing didn't work well, i am ok to roll it back.
cc @nytai and @suddjian I saw you have a lot of work on spa and recent dashboard refactor, please take a look at this PR.
also cc @ktmud and @etr2460

@graceguo-supercat VizType filter values are now missing on the Charts page.


Code error

Check this out !

@akscoder082003 You might've have run the cells in an improper sequence, run them in proper sequence again and if you want you can replace this line of code with
ax = sns.countplot(x='level',data=df, palette = ["maroon", "darkblue"])

@IishuJainn okay i will try this out and update it.

@akscoder082003 You might've run the cells in an improper sequence, run them in proper sequence again and if you want you can replace this line of code with ax = sns.countplot(x='level',data=df, palette = ["maroon", "darkblue"])

yep this code of line solved the issue,
could i update in the main.


Multi-tenant support for GrandId?

Describe the bug
Is there support in GrandId for multi-tenant?
What area is it related to
GrandID
To Reproduce
Steps to reproduce the behavior:
Create a new .Net Core project, install dependencies, AddGrandId twice ( or more )
Expected behavior
An isolated provider per scheme/customer with different api-keys
NuGet package version
ActiveLogin.Authentication.GrandId.AspNetCore 4.0
Runtime version
.NET Core 3.1
Desktop (please complete the following information):

Windows
Chrome etc.

Additional context
Is there support for multi-tenancy when using GrandId?

Hi @Markus-Code!
Multi tenancy support is something we have implemented and used ourselves, but only for the "native" BankID-provider. You can find the documentation for that here:
https://github.com/ActiveLogin/ActiveLogin.Authentication/blob/master/docs/getting-started-bankid.md#multi-tenant-scenario
For GrandID we do not have any official support for such and have no plans to implement it. In general, we have seen a decline in interest for GrandID and will be focusing our development efforts on the native client going forward.
If you are interested in trying to solve this yourself, here are some relevant places in the code:
https://github.com/ActiveLogin/ActiveLogin.Authentication/blob/master/src/ActiveLogin.Authentication.GrandId.AspNetCore/GrandIdBuilderEnvironmentExtensions.cs#L75
https://github.com/ActiveLogin/ActiveLogin.Authentication/blob/3f2b4482c2a1b9c62d376ef4a39824d1cb3485d7/src/ActiveLogin.Authentication.GrandId.Api/GrandIdApiClient.cs#L20
From the top of my head, I think that GrandIdApiClientConfiguration (which today contains static references to the keys) would have to be replaced with callback that given the current HttpContext would resolve the keys. To maintain backwards compatibility, we could provide an overload on the constructor that would take ethe GrandIdApiClientConfiguration as an input.
Would you be up for trying to solve this and send us a PR (following our [contribution guidelines] (https://github.com/ActiveLogin/ActiveLogin.Authentication/blob/master/CONTRIBUTING.md))?
I'll close this as this answers your question, If you want to implement it, please open another issue explaining your proposed solution.


proposal: new vet check complaining about string(int) and string(rune)

Because we cannot remove string(int)/string(rune) conversions (#3939) in a backwards-compatible way, start with disallowing these conversions in vet. This paves the path for the future, backwards-incompatible change #3939.

Same as #32749?

This may be a duplicate, but not of #32749.

Sorry, I meant #32479.

Yes, indeed (for some reason I didn't find it when I was looking for it.)
Closing as duplicate of #32479.


Update quotes.json

#hacktoberfest contribution

add quote
fix a missing comma

@diwakarsingh18277 mate the pr is still in pending


stdin support

lacks extname support but I'm not sure it's necessary

Missing
if (!input && process.stdin.isTTY) {
	console.error('Input required');
	process.exit(1);
}

More info as to why this should be added

https://github.com/sindresorhus/meow/issues/20
Example: https://github.com/sindresorhus/find-versions-cli/blob/56dbd77051147819abdf845504af3b3cc1dd2935/cli.js#L30-L41

Thanks @sonnyp ! Looks good to me.

@sindresorhus r?

@sonnyp :100: :D

Thank you @sonnyp :)


backspace breaks apart multibyte characters

Windows Terminal version
1.14.2282.0
Windows build number
10.0.22000.978
Other Software
powershell core
Name             : ConsoleHost
Version          : 7.2.6
InstanceId       : 098117ae-601b-414f-851f-2c7afccd06c4
UI               : System.Management.Automation.Internal.Host.InternalHostUserInterface
CurrentCulture   : it-IT
CurrentUICulture : en-US
PrivateData      : Microsoft.PowerShell.ConsoleHost+ConsoleColorProxy
DebuggerEnabled  : True
IsRunspacePushed : False
Runspace         : System.Management.Automation.Runspaces.LocalRunspace

Steps to reproduce

copy multibyte character to clipboard, for example 🙂
paste it in the terminal
press backspace

Expected Behavior
The backspace should delete the entire multibyte character, so typing echo "🙂<backspace>" <enter> in pwsh would be the same as typing echo "" <enter> (should print an empty string).
Actual Behavior
When pressing backspace, the character is instead broken in a couple of � � characters, and the backspace deletes the second �, leaving the first one, which, if sent to pwsh, breaks everything (see here: https://github.com/PowerShell/PSReadLine/issues/2358#issuecomment-1258562344)

might be related to #12537 or #1802

Thanks for the report! Unfortunately, the handling of multi-byte sequences at a shell's "input line" is up to that shell. You may need to reopen this issue over on the PowerShell or PSReadline repository. Sorry!

FYI and future reference see here: https://github.com/PowerShell/PSReadLine/issues/2358#issuecomment-1258562344

Thank you! :smile:


[WIP] fix: refactor to proxy all Notion endpoints through lambdas

Description
This is a somewhat large refactor of the demo.

Move all notion fetches behind proxy lambdas (need for CORS, image.js already does this).
Make sure that fetching our proxy endpoints has the correct url on dev / prod (now).
Move the logic for getInitialProps out of the data folder (I'm not sure this is necessary but it was producing the "Rules of Hooks" error on client side transitions as described in the issue).

I've tagged as WIP with a few open questions:

I must be missing something about exactly why getInitialProps throws that hooks error.
Is there a cleaner way to do this? (less proxy lambdas, should we just have one express server etc)
What should live in the lambdas vs getInitialProps. Eg. we could make a lambda out of getNotionData basically...
should we be doing any of this before Notion has an official API 😬

Motivation
Mostly https://github.com/zeit/spr-landing/issues/8.
Want the ability to have multiple pages that link together. Current implementation fails on client side transitions due to Notion blocking the browser requests.
I was pretty 😱seeing this demoed in 🇯🇵
Testing
I added a super simple pages/about.js (not in PR) that links back to the original page with next/link. Works both locally and on prod.
I forked the Zeit notion page behind the demo and ensured that the SPR feature still works with the refactor (change title and first image).
https://spr-landing.wbunting.now.sh/

This is a great idea @wbunting – great thinking

wow this looks really great! 👍

Hi, we recently refactored this demo making this PR no longer valid. Thanks for taking the time to open the PR though!


Buffer underflow for repeating the same message over TCP?

I'm encountering an odd situation which does not seem to make sense to me.
One thread is a TcpListener. Another is a TcpStream which connects to that TcpListener. When I send one encoded PB to the Listener, the listener reacts as it should, and responds correctly. If I send the exact same PB again to the same listener on a new connection for a second time, it also works. Repeating this a third time however (it's a for loop with 10 iterations), I get a buffer underflow error.
Any idea how this can happen? If needed, I can write up some demo code (the code I have which currently does this is likely considered proprietary sadly).
Here's a log of this program (I can share that). The messages preceded by [DEMO] are printed by the thread which send the PB to the listener several times.
> RUST_LOG=info ./target/release/goats 1590 --test
 INFO  goats > Listening on TCP port <IP_ADDRESS>:1590
 INFO  goats > Running test #0
 INFO  goats > [DEMO] Sending 74 bytes to TCP listener
 INFO  goats > Received 74 bytes from <IP_ADDRESS>:35936
 INFO  goats > Decoding received data
 INFO  goats > [DEMO] Received 63 bytes from <IP_ADDRESS>:1590
 INFO  goats > [DEMO] Decoding response
State { datetime: 58112.04391203704, x: -2127.471563378845, y: -2737.7856037111474, z: 6880.2288479000645, x_dot: 5.207976990871051, y_dot: -4.953282710438696, z_dot: -0.36017724766516945 }
State { datetime: 58112.04391203704, x: -2127.471563378845, y: -2737.7856037111474, z: 6880.2288479000645, x_dot: 5.207976990871051, y_dot: -4.953282710438696, z_dot: -0.36017724766516945 }
 INFO  goats > Done in Duration { secs: 0, nanos: 1921582 }: State { datetime: 58112.04391203704, x: -2127.471563378845, y: -2737.7856037111474, z: 6880.2288479000645, x_dot: 5.207976990871051, y_dot: -4.953282710438696, z_dot: -0.36017724766516945 }
 INFO  goats > Running test #1
 INFO  goats > [DEMO] Sending 74 bytes to TCP listener
 INFO  goats > Received 74 bytes from <IP_ADDRESS>:35938
 INFO  goats > Decoding received data
 INFO  goats > [DEMO] Received 126 bytes from <IP_ADDRESS>:1590
 INFO  goats > [DEMO] Decoding response
State { datetime: 58112.04391203704, x: -2127.471563378845, y: -2737.7856037111474, z: 6880.2288479000645, x_dot: 5.207976990871051, y_dot: -4.953282710438696, z_dot: -0.36017724766516945 }
State { datetime: 58112.04391203704, x: -2127.471563378845, y: -2737.7856037111474, z: 6880.2288479000645, x_dot: 5.207976990871051, y_dot: -4.953282710438696, z_dot: -0.36017724766516945 }
 INFO  goats > Done in Duration { secs: 0, nanos: 3665367 }: State { datetime: 58112.04391203704, x: -2127.471563378845, y: -2737.7856037111474, z: 6880.2288479000645, x_dot: 5.207976990871051, y_dot: -4.953282710438696, z_dot: -0.36017724766516945 }
 INFO  goats > Running test #2
 INFO  goats > [DEMO] Sending 74 bytes to TCP listener
 INFO  goats > Received 74 bytes from <IP_ADDRESS>:35940
 INFO  goats > Decoding received data
 INFO  goats > [DEMO] Received 128 bytes from <IP_ADDRESS>:1590
 INFO  goats > [DEMO] Decoding response
thread 'main' panicked at 'could not decode response DecodeError { description: "buffer underflow", stack: [("State", "datetime")] }', src/main.rs:170:27
note: Run with `RUST_BACKTRACE=1` for a backtrace.

Also, here are the PB definitions I'm using:
syntax = "proto3";
package sttr.comms;

message State {
  double datetime = 1;
  double x = 2;
  double y = 3;
  double z = 4;
  double x_dot = 5;
  double y_dot = 6;
  double z_dot = 7;
}

message PropInfo {
  State init = 1;
  double GM = 2;
  double prop_time = 3;
}

Looking at the logs, it seems that the amount of bytes received keeps on increasing. Yet, here's the relevant receiving code, which exists within a for loop:
            // Read the response.
            let mut buf = [0; 128];
            let amt = socket.read(&mut buf).expect("ugh");
            info!(
                "[DEMO] Received {} bytes from {}",
                amt,
                socket.peer_addr().expect("how?")
            );
            let mut dec_buffer = Bytes::with_capacity(amt);
            dec_buffer.extend_from_slice(&buf);
            dec_buffer.truncate(amt);

Are you clearing the write buffer in between each call to encode?

No, I'm not. But it's declared within the for loop, so I was expecting that it would be clearer at each iteration automatically as a new allocation.

Could you provide the code that encodes the message and writes it to the socket?
Also, it doesn't appear that you are using any kind of length delimiting, which is typically necessary when sending multiple messages across a stream.

See these links for info about length delimiting messages in streams:
https://developers.google.com/protocol-buffers/docs/techniques#streaming
https://docs.rs/prost/0.3.2/prost/trait.Message.html#method.encode_length_delimited
https://docs.rs/prost/0.3.2/prost/trait.Message.html#method.decode_length_delimited
I'm not sure what you mean about truncating.

Your insight was on the nose! First, I was not using length_delimited as I should have, and secondly, my response buffer was initialized before the loop which handled the incoming connections. Thanks for your help, the Rust backend works now (onto the C sender now).


Verousca: UX/UI Designs, 2 Weeks

my fork of module repository
Learning Objectives
Week 1

[ ] I have pushed my progress to my fork of exercises repo

I Need Help With:
Nothing at the moment
What went well?
Communication on what to do and what not to do
What went less well?
Clearly understanding the task due to online participation.
Lessons Learned
Some aspects of UX/UI Designs
Sunday Prep Work
Studying Figma

hey @Verousca , I see your week2 label but the comments for it.

Hi Tugba, I left it that way because I have the same comments. However, I will rewrite it next time. Thanks for the feedback


problems in chrome

the example doesn't seem to work in latest webVR chrome Version 56.0.2902.0 (64-bit)
i briefly see dynamic bodies before sim starts then they just disappear and all of the positions and rotations end up as NaN
things work as expected in firefox nightly and it works in release chrome without VR Version 55.0.2883.87 m (64-bit)
perhaps related to bad time values in this release of webVR chrome https://github.com/aframevr/aframe/issues/2207

per discussion on aframevr/aframe#2217 it seems that a workaround for chromium would need to be done outside aframe core

The previous build of WebVR Chromium (Sep 23) will work while we wait for a fixed build, and you can check for the offending browser with:
window.hasNativeWebVRImplementation && navigator.userAgent.includes('Chrome/56')

if you want to notify users.

thx - however Sep23 build does not include Oculus Touch support, and a few other things.  Core team has clearly stated they will not accept systemic workarounds for one-off browser flaws, so until new Chromium builds fix it, components will have to fix themselves (as was done for controllers as part of #2192)

Latest version of Chromium is back to normal. 🎊


add artifacthub-resource

Signed-off-by: Patrick Geschinski<EMAIL_ADDRESS>

@PG2000 Thanks!

@PG2000 Thanks!

@muntac done.

@muntac done.

@PG2000 this URL https://github.com/PG2000/artifacthub-resource seems broken as the repo is gone. Could we remove this resource type or you have some other plan? Thx.

@xtremerui Hey, i moved the repo to another org and wait for approval until i can make it public again.
So i think it will be available again by the end of the week.
If you like i can open a new pr then.
What do you think?

@PG2000 good to know. We will remove it now to fix our CI and add it back later with updated url in your new PR.


Add a menu item for reloading a game from the JSON cycles

We should have an option to load a game from an existing set of cycles, i.e. the ones users can download as a JSON file. This is important for recovery cases where the user hits an error that can't be fixed, or if they want to restore the game to make corrections.
This should belong in the New Game menu category. We can make New Game a split menu, with the default button used for bringing up the New Game dailog.

Playing around with this, I found it would be better if this was just another pivot in the New Game Dialog.
We should also look at making the export cycles menu item open a dialog, which lets you choose if you want to export just the cycles or the whole game (packet, players, cycles).


106 Cohort taking very long time

It is at 40% for a whole night.
Restarted Postgres and the package, running for two hours now.
Any reason for this? should i wait longer?

Hi Peter - For my own reference, the SQL for cohort 106 is here.
Looking at the corresponding JSON for this cohort, there is an inclusion rule called "has 0 condition at increased risk of COVID-19 in prior 1 year" which checks a number of domain tables and in some cases values. For example, ensuring that there are 0 BMI measurements between 30 and 60 in the year prior to index. Some of the tables & fields that you might want to check for an index: measurement.value_as_number, observation.value_as_number, observation.value_as_concept.


add mysql delete undo log builder

What this PR does:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report

Merging #296 (f0e7146) into master (aa569c9) will increase coverage by 3.87%.
The diff coverage is 42.80%.

@@            Coverage Diff             @@
##           master     #296      +/-   ##
==========================================
+ Coverage   50.11%   53.99%   +3.87%
==========================================
  Files          66       73       +7
  Lines        3376     3845     +469
==========================================
+ Hits         1692     2076     +384
- Misses       1594     1642      +48
- Partials       90      127      +37

Impacted Files
Coverage Δ

pkg/datasource/sql/hook/basic_undo_builder.go
34.17% <ø> (ø)

pkg/datasource/sql/hook/logger_hook.go
8.33% <0.00%> (ø)

pkg/datasource/sql/hook/update_undo_hook.go
0.00% <ø> (ø)

pkg/datasource/sql/stmt.go
0.00% <0.00%> (ø)

pkg/tm/init.go
0.00% <0.00%> (ø)

pkg/datasource/sql/hook/undo_log_hook.go
6.66% <6.66%> (ø)

pkg/tm/transaction_executor.go
52.17% <18.18%> (-17.83%)
:arrow_down:

...asource/sql/undo/builder/basic_undo_log_builder.go
30.30% <30.30%> (ø)

pkg/datasource/sql/conn.go
24.20% <33.33%> (+24.20%)
:arrow_up:

.../sql/undo/builder/mysql_delete_undo_log_builder.go
41.81% <41.81%> (ø)

... and 13 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Investigate working checkout for publish from admin button

We saw Oakland publish Springfield, and traced the cause back to a “nobody” directory.

I think this will get solved through the refactor/rewrite that has been talked about. Closing.


Range of integer values

Right now, you can specify params either as discrete values or continuous values (range). It'd be nice to be able to specify that a param is within a range but only takes integer values.
For example, if I specify:
"--nhid": {
"type": "integer_range", "min": 100, "max": 400
},
It'll only search for integer values between 100 and 400, inclusive.
Thanks!

I think you should be able to easily do that with the "values" type. Since the configuration file is just a Python file, you can always write something like this:
"--nhid": {
    "type": "values", "values": list(range(100, 401)),
}

You can even specify non-consecutive ranges with similar tricks if need to. There might be some overhead from constructing this list explicitly, but only when you need to search really large spaces (like integers from 1 to 10^7), which does not seem to be the usual case in practice.

Oooh that's smart. Thanks a lot!


Incorrect use of az ad sp show .. --query password

Note that az ad sp show --id $SP_ID --query password --output tsv is not possible anymore.
Instead, the user needs to safely keep the given password that was the result of the previous step and use that in this CLI command.
Please let this document (as noted also in https://learn.microsoft.com/en-us/cli/azure/ad/sp?view=azure-cli-latest#az-ad-sp-create-for-rbac) show this as well.

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

ID: c06ec64b-fd1c-892d-88cf-e1c8fa66473a
Version Independent ID: 9a57446a-8ded-4ccd-11d1-e574617e6741
Content: Deploy container image from Azure Container Registry using a service principal - Azure Container Instances
Content Source: articles/container-instances/container-instances-using-azure-container-registry.md
Service: container-instances
GitHub Login: @tomvcassidy
Microsoft Alias: tomcassidy

@renald
Thanks for your feedback! We will investigate and update as appropriate.

@renald
According to this doc: Microsoft Graph migration | Microsoft Docs
"Microsoft Graph related commands like az ad and az role will fail in Azure Stack environments which don't have Microsoft Graph support. Please use Azure CLI 2.36.0 or earlier versions for Azure Stack environments."
Using a version 2.36.0 seems to be a workaround for this issue currently. This seems to be an Azure CLI issue, so the Azure CLI team would have more information on the correct command for Azure CLI 2.37.0. As of now the work around would be to use the Azure CLI 2.36.0.

@renald
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation


Keywords in facet option are not complete

The facet option returned with a empty search (without specify any kind of filter) are not complete.
We found this bug analyzing "Keywords" as an example.
If the user are searching for Document without filter we can see that some results have keyword that are not in the available option in the filter

If the user refines the search clicking on some filters, SOLR returns results and the available option for the facets. Now the user can see the keyword there weren't present before

The same behavior is present in the current production environment

To verify: Ensure that not only text literals are Retrieved, but also keywords in the DefinedTerm stanzas
@fils and @jmckenna to verify graph and SOLR congruence on our side

An example call made to Solr for the Documents Topic:
params={q=:&facet.field=type&facet.field=txt_keywords&facet.field=txt_contributor&facet.field=txt_provider&start=0&q.op=AND&sort=indexed_ts+desc&facet.mincount=1&fq=%2Btype:(CreativeWork)&rows=10&facet=true}

This doesn't seem to be resolved.

If the user refines the search clicking on some filters, SOLR returns results and the available option for the facets. Now the user can see the keyword there weren't present before

Note that this is not the Front-end behaviour anymore, all keywords are retained when a Provider or other facet is used to subset results.


ttl/ops

Move some basic operators from nn/ops defined in https://github.com/lgarithm/stdnn-ops to ttl/ops

#30


Huge Performance issues

I want to render some ratings in a FlatList. However, when I navigate to the screen where the list is supposed to be rendered, it takes a considerable amount of time to open. But actually is good Package. Can u check that.

Hello, thanks for writing the issue! It's not enough information to figure out what the problem could be. Is it happens in iOS or Android? Simulator or real device? Expo or bare? Any errors in the console? What navigation package used? How many items in the FlatList? What else are you rendering in the FlatList besides the rating?

I have created a development build using Expo and tested it on an actual Android device. For navigation purposes, I have implemented React Navigation. The screen features a list of 8 items, each representing a different doctor. Each item in the list includes the rendering of an image, name, doctor's type, numerical rating, and price.
Interestingly, I observed that navigating to the screen without the rating component results in a speed improvement of approximately 2 to 3 seconds compared to when the rating component is included. If necessary, I can provide a video demonstrating the performance difference.

Yes please. Also, try to set disabled=true for the rating component. Will that help or stays the same?

Its stays the same

please tell me when u got something

Sure, will let you know. Btw, could you provide the video demonstration you've mentioned above?

Uploading screen-20240116-161032~3.mp4…

https://github.com/kolking/react-native-rating/assets/137432044/624870ed-eaf8-4dc1-ba8d-3c324ec7ef88

screen-20240116-161032.3.mp4

In the upper once, I am using your package and in the bottom once my own.

Hmm... To be honest, I don't see a big difference between top / bottom tiles in the video. I looks like there's approximately the same delay after pressing the tile to display the list. At least, I definitely don't see the 2-3 seconds delay that you've mentioned before.

Maybe its not 3 second but still very noticable. Especially for the user.

I've just published v1.2.0, could you try and check is it better or not? During testing please make sure you set disabled=true for the Rating component and please use "release" (not "debug") build.

To ensure that the rating component does not encounter the performance issues mentioned earlier, I measured the rendering time using the react-native-performance package. I created two screens for comparison; both screens contain a FlatList with 500 items. The "reference screen" renders a static image in place of the rating, while the "measure screen" renders the rating component in every FlatList item. After testing, the average time to render was 24 milliseconds for the "reference screen" and 34 milliseconds for the "measure screen." I believe the 10-millisecond difference is not a significant issue. The code I used for this testing can be found in the test-performance branch.
https://github.com/kolking/react-native-rating/assets/4656448/d7d067e1-d1aa-42f1-b894-6f6bc1f9d797


Message graphs incomplete

The graph for a model only shows the conditions of the links between fields. This gives only a partial picture, as the constraints defined by the field types are missing.

When adding that, we should also think about an alternative format. The (Condition, Length, Start) notation is not all too intuitive. Maybe we should prefix them (e.g. Condition => or even when ...\nif ...) and leave out defaults.


VehicleX Datasets

The 2020  AI CITY CHALLENGE is over, how to apply for the datset? Can you share? I am a master in Huazhong University of Science and Techology, thanks a lot!

I guess the synthetic vehicle dataset (VehicleX) for 2021 AI City Challenge is the same as that for 2020 Challenge. If so, you can require the dataset directly from the challenge committee. Please double-check if the dataset this year is the same as that used for last year.

meet the same problem, the link  https://www.aicitychallenge.org/2020-track2-download/ shows "need password to view". How should I do next step?

@.***


New owner of the Java implementation

Its here now: github.com/igr/jrsmq
Please update the docs :)

Done.


Include license file in MANIFEST.in

This PR makes it so that the license file is included in the source distribution.

Coverage remained the same at 98.44% when pulling a35e7a8921f53a1d5993338b933f9d3effaa2c08 on synapticarbors:patch-1 into 0c7a39b640ad04165eb1e7b4af0f29df1f508183 on timothyb0912:master.

Thanks for adding this!


refactor return the error status instead of static_cast

Proposed changes
static_cast<void> has ignored the wrong status, some of them should make the query finished with error status, so replace static_cast<void>  with RETURN_IF_ERROR.
Remaining Works
The following three scenarios need to be handled separately and cannot be simply replaced:

The outer function returns void;
Call status function inner constructors or destructors;
Call status function with best effort, and should ignore the wrong status.

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall


How to tell difference between 2 actions that result in same mutation

Hello,
I am building an application that is using UITableView for displaying list of some items. There is also some other screen where you can pick a filter for those items. The problem I encountered with state is that I have an array of those items in state and I am running diff on that array whenever state changes so I can remove/insert rows from/to table view with animation. However, when I change the filter, I want to actually do this without the animation. The issue is that I do not, know how to tell when it is just items that changed for whatever reason and when the items changed because of the new filter was applied. Even if I have filter in my state struct.
Maybe my whole approach is wrong.
I have action .subscribeToListChanges that generates a mutation whenever something in Realm changes (it is basically never completing Observable). Then I have action .setFilter that sets filter. That changes filter which triggers also new mutation from .subscribeToListChanges Observable.

I think you should add another property to the state:
struct State {
  var isFilterSetRecently: Bool // set to true on `.setFilter` and false on other actions
  var items: [Item]
}

yes, I suppose that is the way. However I also think, it would be nice to have the actual mutation that was happening available so you can maybe determine you want to do some sort of animation in that case, etc


Follow Dracula specs

I changed the colors so it follow the specification (source) and fixed green colors being swapped with cyan ones (which causing weird colors on my tmux/git).
I should also update the picture on the README, but I am not sure on how to do that.
Before: https://imgur.com/PrVJS1W.png
After: https://imgur.com/6sTNtEj.png

@Drowze
Hi, thank you for feed back. I'll check the Dracula specs.


[Bug] dead link in language/README.md

🐛 Bug
Reporting a dead link in the README at https://github.com/diem/diem/tree/main/language
The dead link is named 'tests' under the 'Exploring the Move Language'
This link gives a 404
System information
Brave version 1.30.87 | MacOS Big Sur 11.4

@bob-wilson and @MaxMohammadi: I've gone ahead and fixed the broken link. I figured it'd be a good patch for now even if the team ends up shortening the README later.


test: add tests for getUserDetails

Hi, before I review, could you follow the pattern karthik had in his PR (#48 ), it's pretty nice, so keeping it unifrom will help when we refactor things into the setup part.

Closing because stale, and too many changes to backend code since this PR


Updating Azure-CNI to v1.0.11 for Windows

This is a cherry-pick of #3660 to update Azure-CNI to v1.11.0 for Windows only. Linux is not changed.

/lgtm


Fix 2D/3D view artifacts on view's border when using fractional zoom

Related

Fixes https://github.com/rerun-io/rerun/issues/8330

What
Wasn't able to repro the issue in the first place though - tried fractional zoom and a contrast heavy background to no avail

[ ] @emilk please confirm this fixes it

@abey79 did you have repro of the issue earlier? I failed to reproing it so I'd like to confirm whether this fixes it


Add | SMKN 1 Jenangan to the list

Adding SMKN 1 Jenangan to Jetbrains existing school list.

@setiawanjoko First stage of verification passed, official domain is smkn1jenpo.sch.id as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

Why is it closed and not merged? Do I need to do something else?


CSL-1800 Move not csl-specific utility modules into a separate package

Now we have cardano-sl-util package, which contains various utility functions which are not cardano-sl-specific. There are two reasons why it has been done:

It allows us to move Pos.Crypto into a separate package (because Pos.Crypto depends on some utility modules).
It makes cardano-sl-core smaller, this package has more than a hundred modules, so we want to move something from it.

Previously we had Pos.Util.Util module which was very big and contained groups of utilities. I put some big groups into separate modules (only those which logically formed a separate group and looked good to be separated). For instance, orphan instances for 3rdparty types/classes are now in Pos.Util.Orphans.

It's ready for review.


Fix autofunction Reference

requests.codes is a class (LookupDict), not a function.

:sparkles: :cake: :sparkles:


Add to djangopackages.org

hi, thx for your nice and simple tree :)
i've added your package to the https://djangopackages.org/grids/g/trees-and-graphs/ comparison grid.
could you take a look if i entered the information right?
thx

Wow, thanks! I think this is correct. The Drop In thing isn't totally fair because you could use the queryset without adding the TreeNode abstract model to the code (I think) but this may be awkward. So it's probably fine and the additional checkbox really isn't worth it :)


isPristine feature is not present in dist package

Hello,
I just started using your package, very good one, thanks. But I found out that isPristine feature is not present in the code donwloaded from NPM
https://registry.npmjs.org/react-use-form-state/-/react-use-form-state-0.12.1.tgz
Is it right or am I missing something here?
Thank you very much

Hi @mpfullstack – isPristine has not been published yet.
The documentation for 0.12.1 is available here:
https://github.com/wsmd/react-use-form-state/blob/0.12.1/README.md
isPristine should be coming soon with 0.13.0 (it's an overdue update tbh, but I have been busy lately! 😅  stay tuned!)

Oh I forgot to mention that isPristine is available in the beta release! react-use-form-state@beta. Feel free to use it in the meantime!

Oh ok, I see.
Thank you very much for the info.

Any update on this? Just ran into the same thing too since it's in the README 😅.
If you need help implementing this I'd love to help out if needed.

0.13.0 is now available on npm! 😄


Add a "shared cache" subdirectory of .nox and add API for accessing it

@theacodes And I have discussed this before but I keep forgetting to file an issue.
A few applications:

Python extensions with long compile times can cache object files and other built resources there (can share object files across different Python versions)
Store hashes of non-Python source files, so that way the .so/.pyd modules in an editable install can be updated when the source changes

I'm all for this idea, but we'd need someone to sketch out what the API should look like.

Would {envdir}/.shared be an appropriate name? Does there exist a check against the virtualenv_dirname for "reserved" names?

That makes sense to me.

There's nothing blocking this issue, and it's a great candidate for a contribution. We'd just need to sketch out the API together.

Noting here that pip's use case would benefit from this, since we maintain a common wheels "cache" for our test suite (downloaded setuptools and wheel .whl files, which can be installed during the tests in our various environments).
For now, I'm just dumping them in a local directory that's git-ignored.

We have some of this in #320, this just makes it higher-level by having a directory that is shared across sessions and persists between Nox runs.

@theacodes @dhermes How is this issue going? If there's something else to do, I can do it. Just tell me.

So, can I work on this issue?

Go for it!
On Thu, Sep 9, 2021 at 5:40 PM Diego Ramirez @.***>
wrote:

So, can I work on this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/theacodes/nox/issues/72#issuecomment-916456709, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB5I4YXRDKW2TYA7KLFTU3UBESTFANCNFSM4EAFMCSQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Ok. I'll open a PR the next week (I don't feel so good this week :disappointed:).

Absolutely no pressure, and I hope you feel better soon.
On Thu, Sep 9, 2021 at 5:50 PM Diego Ramirez @.***>
wrote:

Ok. I'll open a PR the next week (I don't feel so good this week 😞).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/theacodes/nox/issues/72#issuecomment-916462289, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB5I434UXOPO6XRRKSIXHLUBET35ANCNFSM4EAFMCSQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Add TantaresFairingExtension

Closes #2773

copy-pasting is happiness, copy-pasting is life.

Woops, we should filter Source Code.txt.


TensorFlowEstimator can not be found

when i am running sequence predicting program in windows10 ,it stucked in an error :AttributeError: module 'tensorflow.contrib.learn' has no attribute 'TensorFlowEstimator'. the tensorflow edition is 0.12rc  and python3.5 .how can i fix this problem?

We removed the TensorFlowEstimator class in favor of Estimator. The interface is somewhat different, but it should be an easy conversion.


Let's change the db names to proper reference and also check the possibility of db name collision.

https://github.com/ledokku/ledokku/blob/eb8eaf7317370e80dc391babfdb43048ab9f0656/ledokku-bootstrap.sh#L228-L233
Issue Showcase
In the above code, we can see, that the databases created are all named as ledokku, which itself is not an error prone variable assignment => but we can be more explicit in our naming conventions.
My Suggestions
Let's make our database name more explicit and add some logic to check to verify the duplication of databases. It would be something like this:

Changing Redis db name to: ledokku-redis-db
Changing Postgress db name to: ledokku-pg-db

and finally, during the installation process, let's add some logic to check whether there is any name collision => If there is a name collision we will prompt the user that we are about to delete the old databases and install new ones.

Reason => In case of installation failure(in the middle), not everyone might be familiar with dokk-cli to troubleshoot things.

@pradel @Akirtovskis What do you say? Shall I start working on it by creating a template PR?

@akhil-naidu Thanks for the idea, looks good to me! 🚀


Fix of core handling - can't use loop variable inside goroutines

Check failed due to DPDK site access, not because of commit.


Provenancing of xarray output files

Files produced by an xarray save via method to_netcdf should retain information related to how they are generated.  This includes at a minimum the machine, script and arguments called, user, and date generated, although there are probably other important criteria too (git hash).  This issue should be used to discuss relevant criteria that are useful prior to an implementation of a wrapped to_netcdf function.
See also https://github.com/pydata/xarray/issues/826
cc @toddringler, @milenaveneziani

@milenaveneziani, can you please take a quick look at this -- it is probably something we should very briefly talk about in today's meeting.

Will be addressed in https://github.com/MPAS-Dev/MPAS-Analysis/issues/35, not here.


Fix minor typographical errors in CONTRIBUTING.md

Thank you for your contribution to the KodaDot NFT gallery.
👇  _ Do a quick check before the merge.
PR type

[ ] Bugfix
[ ] Feature
[ ] Refactoring

Before submitting Pull Request, please make sure:

[ ] My contribution builds clean without any errors or warnings
[ ] I've merged recent default branch -- main and I've no conflicts
[x] I've tried respect high code quality standards
[x] I've didn't break any original functionality
[ ] I've posted screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[ ] I've tested PR on mobile and everything works
[ ] I found edge cases

What's new?

[ ] PR closes #<issue_number>
[ ] <brief_description_of_what_I've_added>

Had issue bounty label ?

[ ] Fill up your KSM address: Payout

Community participation

[ ] Are you at KodaDot Discord?

Screenshot

[ ] My fix has changed something on UI, a screenshot for others, is best to understand changes.

Hello here, hope you are good. I will love to contribute more but I am
having challenges in setting up my environment variables. Is there an
alternative to retrieving these variables, maybe from an open encrypted
server or on discord or something? I will greatly appreciate that a lot of
I can have that.
Thank you and best regards
On Mon, Dec 27, 2021, 14:31 Matej yangwao @.***> wrote:

Good stuff! Let's make try to pick another great issue there to receive a
payout!
—
Reply to this email directly, view it on GitHub
https://github.com/kodadot/nft-gallery/pull/1609#issuecomment-1001563429,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHODQETRZ6DLJJE7QXMGUO3UTBQ4NANCNFSM5KZLIXSQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hello here, hope you are good. I will love to contribute more but I am having challenges in setting up my environment variables. Is there an alternative to retrieving these variables, maybe from an open encrypted server or on discord or something? I will greatly appreciate that a lot of I can have that. Thank you and best regards

Hey!
I don't think you need any environment variable to run Kodadot app.
Feel free to come on discord anytime if you need help to setup the project.


Faulty name for “and with dot”

While going through the glyph names of a font, I came across a strange suggested unicode value for a glyph called “dot”.
Turns out n2u was suggesting the unicode for “and with dot”.
To replicate:
from glyphNameFormatter.reader import n2u

name = 'dot'
v = n2u(name)

print(name, hex(v), chr(v), u2r(v))

Output:
dot 0x27d1 ⟑ Miscellaneous Mathematical Symbols-A

Fixed:
>>> print(u2n(0x27d1))
andwithdot
>>> print(n2u("dot"))
None

Thank you @arialcrime!


Crash in Scenario editor when previewing compute with Non-failure scenario

When attempting to preview compute in the Scenario editor when using a non-fail scenario, an exception is thrown when attempting to draw Y axis labels. This appears to be caused by the fact that in the GraphicalUncertainPairedData object on line 96, the generated pairedData object returned by the SamplePairedData method is not passed the metadata from the GraphicalUncertainPairedData, only y and x values. When this returned paired data is used to label the y axis (on line 23 of the FrequencyRelationshipControl) it throws an exception as pd.Metadata is null. It appears the fix is to simply pass the curvemetadata to the returned paireddata object in the GraphicalUncertainPairedData:
PairedData expandedPairedData = new PairedData(ExceedanceToNonExceedance(CombinedExceedanceProbabilities), expandedStageOrLogFlowValues, CurveMetaData);

I am unable to replicate this issue.


Design VA Profile and MPI integration

As a VANotify developer
I want access to technical documentation on VA Profile and MPI integration
So the team is aligned on the approach
And I have context to confidently work on implementation
As VANotify consumer

I want access to OpenApi documentation
So that I can easily integrate with VANotify
And consume its services
In scope / Checklist:

[x] sequence diagram for sending notification that requires looking up contact information from VA Profile and translating identifier using MPI - https://github.com/department-of-veterans-affairs/notification-api/blob/master/documents/contact-info-lookup-sequence-diagram.puml

[x] Open API documentation of the new API that utilizes VA identifiers (happy path and error responses)
[x] list of changes to data model to support VA Profile and MPI integration:

Change the to column in the notifications table to be nullable
Create a new identifiers table (name TBC), with the following columns:

id (UUID, not nullable)
notification_id (UUID, foreign key constraint to notifications.id, not nullable)
identifier_type (String, enum of participant_id, vet360_id, not nullable)
identifier_value (String, not nullable)

[x] list of additional notification states / statuses if applicable -> See ADR. No new statuses.
[x] documentation of error paths - https://github.com/department-of-veterans-affairs/notification-api/blob/master/documents/contact-info-lookup-error-cases.md
[x] ADR entry for integration design - https://github.com/department-of-veterans-affairs/vanotify-team/blob/master/documents/adr/0004-integration-with-va-profile-and-master-person-index.md

Reviewed ADR with devs in discussion today. Moving to Review, but we may be able to just close it?


dqn.cc build error

dqn.cc build error
Hello, I encountered an error during the recent compilation process.
error output
 error: return-statement with no value, in function returning 'open_spiel::Action' {aka 'long int'} [-fpermissive]

Compile parameters

OPEN_SPIEL_BUILD_WITH_PYTHON=OFF
OPEN_SPIEL_BUILD_WITH_LIBNOP=ON
OPEN_SPIEL_BUILD_WITH_LIBTORCH=ON
BUILD_SHARED_LIB=ON
gcc 9.4.0

My Solution
I can compile by simply returning 0 but I don't know if it is correct.
https://github.com/google-deepmind/open_spiel/blob/16d21cfb71630b6d8294b519899bdd081904cb8e/open_spiel/algorithms/dqn_torch/dqn.cc#L40

Ah yes, good catch. Compilers are too leanient these days... :)
That should be return kInvalidAction;
That's easy enough for me to fix on my end, but I like to encourage people to submit a PR fix to get the public credit. Do you have a preference?

Fixed by https://github.com/google-deepmind/open_spiel/commit/b85fd2420941d547349c2af6e7e13b904bbf2207
Thanks!


SerialBT issue

Hi!
I'm working on esp32 and I need SerialBT lib AND button2.. button2 work great if I comment out SerialBT command in sketch.
As soon as I uncomment SerialBT command, button2 refuse to work. SerialBT is working.
There's no delay is code.

If u working with esp32 use RTOS tasks for non blocking actions!

I'll take some time for me to reproduce this. I'll have to dig up an ESP32 and give it a try.
I usually use ESP8266s so I don't have one at hand.

Sorry, I don't  really understand your request.
In the constructor you can define the button mode  (INPUT or INPUT_PULLUP).
Does this not solve your problem?
If not, could you please elaborate a bit? thx.

Buttons2 and serialBT does not work together..

Buttons2 and serialBT does not work together..

On esp32 all works....

It might have been, that the loop got stuck because of no delays(). Added one to the Button2 loop.


Question: Having subpath/Virtual Directory in IIS hosted development setup

I am using Microsoft.DotNet.Web.Spa.ProjectTemplates::2.0.0-rc1-final which internally uses Create-react-app and AspnetCore.ReactDevelopmentServer
My Dev. Environment is using IIS for hosting multiple api's say api1 as http://localhost/api1 and api2 as http://localhost/api2
Problem is if I host my Asp.Net core React app from Microsoft.DotNet.Web.Spa.ProjectTemplates::2.0.0-rc1-final template to http://localhost/myApp - it expects the files are always served with PublicPath "/" and hence does not work.
This feature is due for release in react-scripts 2.0 which should include https://github.com/facebook/create-react-app/pull/1887. This will allow subpaths to be included in url from where the files are being served in create-react-app.
Just wanted to check if there is any work around for it at this time?

Just wanted to check if there is any work around for it at this time?

We're limited to what create-react-app itself offers, so please see create-react-app docs for any techniques to control what URLs it looks for resources at. Hope that's OK!

For people coming to this thread - create-react-app has a proxy api requests feature - https://github.com/facebook/create-react-app/blob/master/packages/react-scripts/template/README.md#proxying-api-requests-in-development which helped in my case


[Question] How to use Email channel?

Currently i am using slack channel with bot builder. I want to use email channel, is there any sample code for this. What is effective way to send email from bot to avoid spam I am using node js

@aakashkag You can find out more about how to configure the Email channel here:
https://docs.microsoft.com/en-us/bot-framework/channel-connect-email
Connect a bot to Office 365 email

Bots can communicate with users via Office 365 email in addition to other channels. When a bot is configured to access an email account, it receives a message when a new email arrives. The bot can then respond as indicated by its business logic. For example, the bot could send an email reply acknowledging an email was received with the message, "Hi! Thanks for your order! We will begin processing it immediately."

Step 1: Create new Microsoft email account for your bot.
Step 2: Configure Email channel in Bot Portal at https://dev.botframework.com

Closing as duplicate of https://stackoverflow.com/questions/45694405/how-to-use-email-channel-in-microsoft-bot-builder


If loaded after zone.js, oidc-client kills change detection in angular 2 rc5

This plunker shows a simple example of an Angular 2 app that changes in response to an observable.
This one only differs in that the script tag for oidc-client is moved after the script tag for zone.js, but angular no longer updates the display. The underlying data is still changing, and if you cause the change detection to fire through any other means then it'll show the correct state at that point in time.
For what it's worth, this issue first appeared when we upgraded angular to rc5.

And can you determine what the cause is?

I haven't spent any time digging into oidc-client code; it took me the
better part of two days to trace the problem to this library, and I wanted
to get an issue logged with a minimum reproduceable case before I left
work. Does oidc-client mess with any html5 shims? Maybe defining the
Promise type or something like that?
By the way, this was causing issues in previous versions of Angular 2 but I
had just assumed it was a shortcoming with their change detection strategy
and I came up with a workaround. RC5 just made it so my workaround stopped
working.
On Tue, Aug 23, 2016, 9:16 PM Brock Allen<EMAIL_ADDRESS>wrote:

And can you determine what the cause is?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/IdentityModel/oidc-client-js/issues/87#issuecomment-241946401,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAbrdzWpaW8qL4ZjjGQQXz41pKLkaM7fks5qi7eSgaJpZM4JraXh
.

Yes, the ~/dist/oirc-client.js includes babel's polyfill: https://github.com/IdentityModel/oidc-client-js/blob/master/README.md

Are you using a module loader? I guess the assumption is that with something more modern like Angular 2 you would be...

Yeah. I'm still feeling like a beginner with TypeScript and ES6 module loading, so it took me quite a while to figure out the right way to include oidc-client into my project. I was expecting it to be more along the lines of Angular, where the import statement clues the TypeScript compiler into the fact that this library is being used, and causes it to be included.  I was expecting to need something like this:
import { UserManager } from "oidc-client";

Now I think I understand that even though I'm using a module loader I'm actually expected to bring oidc-client in as an ambient dependency. Then, I need to use a /// <reference> to the .d.ts file, to help TypeScript to know that things like UserManager exist. I don't need an import statement like the one above at all. Am I missing something?

To be honest, I wish I could say.... I also don't use TS or Angular -- I had to have others who know how module loaders work to help me organize this project. Keep at it, and let me know what you find.

I was able to get things to work using the strategy I mentioned above, but it feels a little wrong to me. If I change oidc-client.d.ts to look like this:
export class AccessTokenEvents {

    load(container);

    unload();

    addAccessTokenExpiring(callback:(ev:null) => void);
    removeAccessTokenExpiring(callback:(ev:null) => void);

    addAccessTokenExpired(callback:(ev:null) => void);
    removeAccessTokenExpired(callback:(ev:null) => void);
}
export class InMemoryWebStorage {
    getItem(key: string);

    setItem(key: string, value: any);

    removeItem(key: string);

    key(index: number);

    length?: number;
}
export class Log {
    NONE();
    ERROR();
    WARN();
    INFO();

    reset();

    level(value);

    logger(value);

    info(...args);
    warn(...args);
    error(...args);
}

export class MetadataService {
    constructor (settings: any);
    getMetadata();

    getIssuer();

    getAuthorizationEndpoint();

    getUserInfoEndpoint();

    getCheckSessionIframe();

    getEndSessionEndpoint();

    getSigningKeys();
}
export class OidcClient {
	constructor (settings: OidcClientCtor);

    createSigninRequest(args?: any): Promise<any>;
    processSigninResponse(): Promise<any>;

    createSignoutRequest(args?: any): Promise<any>;
    processSignoutResponse(): Promise<any>;

    clearStaleState(stateStore: any): Promise<any>;
}

export interface OidcClientCtor {
    authority?: string;
    metadataUrl?: string;
    metadata?: any;
    signingKeys?: string;
    client_id?: string;
    response_type?: string;
    scope?: string;
    redirect_uri?: string;
    post_logout_redirect_uri?: string;
    prompt?: string;
    display?: string;
    max_age?: number;
    ui_locales?: string;
    acr_values?: string;
    filterProtocolClaims?: boolean;
    loadUserInfo?: boolean;
    staleStateAge?: number;
    clockSkew?: number;
    stateStore?: any;
    ResponseValidatorCtor?: any;
    MetadataServiceCtor?: any;
}

export class UserManager extends OidcClient {
    constructor (settings: UserManagerCtor);

    clearStaleState(): Promise<void>;

    getUser(): Promise<User>;
    removeUser(): Promise<void>;

    signinPopup(args?: any): Promise<User>;
    signinPopupCallback(url?: string): Promise<any>;

    signinSilent(args?: any): Promise<User>;
    signinSilentCallback(url?: string): Promise<any>;

    signinRedirect(args?: any): Promise<any>;
    signinRedirectCallback(url?: string): Promise<User>;

    signoutRedirect(args?: any): Promise<any>;
    signoutRedirectCallback(url?: string): Promise<any>;

    events: UserManagerEvents;
}
export class UserManagerEvents extends AccessTokenEvents {
    load(user: User);
    unload();

    addUserLoaded(callback:(ev:null) => void);
    removeUserLoaded(callback:(ev:null) => void);

    addUserUnloaded(callback:(ev:null) => void);
    removeUserUnloaded(callback:(ev:null) => void);

    addSilentRenewError(callback:(ev:null) => void);
    removeSilentRenewError(callback:(ev:null) => void);
}
export interface UserManagerCtor extends OidcClientCtor {
    popup_redirect_uri?: string;
    popupWindowFeatures?: string;
    popupWindowTarget?: any;
    silent_redirect_uri?: any;
    automaticSilentRenew?: any;
    accessTokenExpiringNotificationTime?: string;
    redirectNavigator?: any;
    popupNavigator?: any;
    iframeNavigator?: any;
    userStore?: any;
}
export class WebStorageStateStore {
    set(key: string, value: any);

    get(key: string);

    remove(key: string);

    getAllKeys();
}
export class User {
    id_token: string;
    session_state: any;
    access_token: string;
    token_type: string;
    scope: string;
    profile: any;
    expires_at: any;
    state: any;
    toStorageString(storageString?: any);
}

Then it allows me to treat it like an ES6 module in my TypeScript file, like this:
import { UserManager } from "oidc-client/oidc-client";

let manager = new UserManager({...});

I just need to set up my module loader to know where to find it. So in my case, I add some lines to the properties in system.config.js:
	var map = {
		'app': "client/dist/app",
		'@angular': "node_modules/@angular",
		'angular2-in-memory-web-api': "node_modules/angular2-in-memory-web-api",
		'rxjs': "node_modules/rxjs",
		"oidc-client": "node_modules/oidc-client/lib"
	};
	var packages = {
		'app': { main: "main.js", defaultExtension: "js" },
		'rxjs': { defaultExtension: "js" },
		'angular2-in-memory-web-api': { main: "index.js", defaultExtension: "js" },
		"oidc-client": {main: "oidc-client.js", defaultExtension: "js"}
	};

I don't need to have anything in my references.d.ts file, and theoretically if I had a part of my app that didn't need oidc-client, it wouldn't need to be included in the source for that page. This feels better to me, but I'd like to get some input from an expert. Do you know of anyone we could run this by?

Sorry -- no idea. I'll close for now since you seem to have a workaround. Reopen if you need.

For future reference, the changes made in v1.2 now make it possible to do what I wanted.

Ah, good to hear. Thanks.


implemented SameSite checks for cookies

Stage 8 will now check for the 'SameSite' cookie and verify which value it's assigned (if any).

Looks great, thanks! I left one small nit.


Add basic security to API

The API should be protected by credentials that will be sent on requests, to authenticate.

The implementation can follow this official documentation: https://docs.spring.io/spring-security/reference/servlet/authentication/passwords/basic.html


Can't manage to get bignumber.js to behave, unknown repair?

Dear Michael,
I am encountering problems trying to get you Javacript 'bignumber.js'
library to work in a web page loaded by 64 bit Firefox 66.02.
I have used the file 'bignumber.js' entirely on its own, away
from any of the download files.  I then have that in the same
directory as my .html file, Rotate.html.
The code involved is right at the end of my Rotate.html file.
I am getting an error that I do not understand.  How
can I have your library work seamlessly?
These should be renamed Rotate.html and bignumber.js:
Rotate.txt
bignumber.txt
By the way, I have heard that your library
"only works at the block level".  Is this true,
and does it impact my problem?  Are there
restrictions about where your BigNumber
objects can and cannot be invocated?

How can I have your library work seamlessly?

A good start would be to use the script element correctly: it should not have code inside its tags if there is a src attribute specified. Try:
<script src='bignumber.js'></script>
<script>
let x = new BigNumber(144).toString();
document.write('<a>' + x + '</a>');
</script>

By the way, I have heard that your library "only works at the block level". Is this true, and does it impact my problem? Are there restrictions about where your BigNumber objects can and cannot be invocated?

No, no, and no.

You've been absolutely right.  I now have things running just as desired.
-Is there a functions or trigonometry library I can use directly on your
BigNumber type in this library?
var pi = radian_arc_cosine(-1);
...

Not that I am aware of, but see decimal.js.


Fix selftest getting stuck on Windows

rez selftest got stuck on test_rex_code. Fixes #481
@instinct-vfx @bpabel
What do you think about this one?

Ok, the problem at hand is fixed as soon as I apply the same changes from PR #488. Therefor I am closing this PR for now.

Ah, my bad, I mixed two branches by mistake. This fix is still relevant.
Pinging the other active Windows devs on here again: @instinct-vfx @bpabel
What do you guys think?
In order to confirm my suspicion and isolate this issue from the other Windows related problems I removed all tests from rez/tests and only left test_shells.py.

Just a note, apologies I've been quiet of late, will be back onto sorting
these PRs out soon.
On Wed, Feb 14, 2018 at 10:36 AM, Sebastian Kral<EMAIL_ADDRESS>wrote:

rez selftest got stuck on test_rex_code. Fixes #481
https://github.com/nerdvegas/rez/issues/481
@instinct-vfx https://github.com/instinct-vfx @bpabel
https://github.com/bpabel
What do you think about this one?
You can view, comment on, or merge this pull request online at:
https://github.com/nerdvegas/rez/pull/482
Commit Summary

Fix selftest getting stuck on Windows

File Changes

M src/rezplugins/shell/cmd.py
https://github.com/nerdvegas/rez/pull/482/files#diff-0 (14)

Patch Links:

https://github.com/nerdvegas/rez/pull/482.patch
https://github.com/nerdvegas/rez/pull/482.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nerdvegas/rez/pull/482, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABjqSvCVq4YJ3nWQTAcd9xc6qqdnCnoEks5tUhyLgaJpZM4SEkuK
.


Task: update copyright year at the beginning of 2024

TRG 7.00 requires that the copyright header be updated at the beginning of each new year.

Only touched files get updated.


Question: two-column format?

Starting off w/ Slate for the first time, I'm wondering: is there simple/std way to disable 3-column format and go to 2-column w/ code snippets inline? Reason: I'm used to have the browser on half a screen, i.e. 960px wide, which makes things a bit crowded with the default format. The snippets won't be that large for the docs I'm writing.
Apologies if this is a FAQ, but a first search in the wiki and issues didn't lead to any hints.
-jcw

No worries! Does changing this variable to be a larger value fix your problem?

If you look at my fork, there is a variable that you can set in the YAML directives at the top that pullls in a different CSS to provide a two column format with code snippets in-line.
It has a lot of other changes that you're probably not interested in but if you look at the layout template, you'll see how the CSS is used to get two columns.
Paul
https://github.com/pnerger/dpSlate

@lord - Ah, very nice, thx: it snaps into two columns when set to 958 (MacOS, Safari, using Moom).
@pnerger - That was indeed exactly the other part of my question - thanks for the link.
You guys rock.

Whoops - closed this issue just a tad too quickly, I'm afraid...
When trying out dpSlate, I get errors which seem to prevent middleman from launching:
Expected string default value for '--instrument'; got false (boolean)
Expected string default value for '--reload-paths'; got false (boolean)
Expected string default value for '--instrument'; got false (boolean)

This was after the bundle install. I did find the rightPanel setting in source/_defaults.yml.

(I'm going to close this since we don't support other forks, but feel free to continue discussion here if you'd like.)

Understand - I didn't find another way to report this to @pnerger but it's not an issue for this repo.


FEATURE: Add caption

I add the ability to add a caption to the table. Also is the data and caption set via property and not via @context

Thanks @jonnitto Your PR is merged and released. I did some small refactorings.


Clean up @ni/jasmine-paramaterized package metadata and examples

Pull Request
🤨 Rationale
npmjs allows searching for packages by keywords but this package didn't specify any so its SEO performance was sub-optimal 😔.
https://www.npmjs.com/search?q=keywords%3Ajasmine
There was also a typo in the examples.
👩💻 Implementation
Add some keywords borrowed from jasmine core packages and the unscoped jasmine-paramaterized package
Fix typos in README and code comment.
🧪 Testing
N/A
✅ Checklist

[ ] I have updated the project documentation to reflect my changes or determined no changes are needed.

For some reason the UI Tests status check is stuck in Pending despite the actual build having passed. Going to try some operations to poke it.

@jattasNI happened in this PR yesterday too, not sure why: https://github.com/ni/nimble/pull/1837#issuecomment-1953213289


Help with easylist.txt

Any help on getting the filter rules from easylist.txt, please?
Also, wouldn't it be faster to use a dictionary/set instead of a list?

Hey @JoaoLages,
To get rules from easylist.txt file you should read it and pass its lines to AdblockRules, something like
with open('easylist.txt', 'rb') as f:
    raw_rules = f.read().decode('utf8').splitlines()

rules = AdblockRules(raw_rules)

For rules matching re2 regex engine is used, it is faster than checking all rules one-by-one. There may be ways to make it faster though; suggestions are welcome.


fix: never read reserved txs from the mempool

closes #1266
see details there.

CI is failing here

yeah thanks, I'm waiting on this comment as I'm not sure if this is correct property or not https://github.com/celestiaorg/celestia-core/pull/1267#discussion_r1523638682

while separate from this change, I'm fine with not merging until we resolve https://github.com/celestiaorg/celestia-core/pull/1267#discussion_r1526736715

I don't understand this change enough to approve.

I don't understand this change enough to approve.

@rootulp Does this sentence in the original issue help, or perhpaps we can sync on it quickly to get to the bottom it?

CAT will return txs that are reserved but not yet stored. This can result in weird behavior, such as passing the application a zero length transaction during PrepareProposal.

The fundemental problem is just that we can't pass len 0 txs to the application, as it will panic. Since the mempool processes things async, atm its possible to reap a reserved tx (a tx that is in the process of being added to the mempool, but only has a hash)

Consider adding a separate map for reserved transactions

This seems plausible as well. We essentially just want to check this map when a new transaction comes in, in case we are already processing it. It could just be a map of the tx hashes

closing in favor of #1272


Unable to upload workbench.aspx to an old created developer Site Collection

Hello
I have an old Developer Tenant. About 2 years old. I was trying to use the originally provisioned Developer Site Collection, but when I was trying to upload the workbench.aspx file, I got many issues. I got a lot of access denied errors. Even on SP Designer
What I did to make it work was, spin up a new fresh developer site collection. On the new one, I was able to upload the workbench.aspx file successfully
I was wondering, if may help put something on the documentation suggesting create a new developer site col.

thx Raul,
added note on this to the guidance. Issue what you see is pretty strange, since that would imply that you can't upload any files to that site collection. workbench.aspx is just normal file to upload. Have though added a short note on the wiki at https://github.com/SharePoint/sp-dev-docs/wiki/Setup-SharePoint-Tenant.

Yes, you need to create a developer site collection per the instructions - if you use the default root site collection you get access denied uploading the workbench.aspx. You can upload other file types - I think aspx is blocked but I didn't really dig any deeper.

@SPDoctor you can't upload aspx-pages to the root-site-col because of the deactivated scripting (see https://support.office.com/en-us/article/Turn-scripting-capabilities-on-or-off-1f2c515f-5d7e-448a-9fd7-835da935584f)
I activated scripting in the root site using powershell like this:
Connect-SPOService -Url https://[tenant]-admin.sharepoint.com -Credential<EMAIL_ADDRESS>Set-SPOsite https://[tenant].sharepoint.com -DenyAddAndCustomizePages 0

Thanks @nyn3x for pinpointing the reason.


Cannot set property 'singleArc' of undefined

Hi there,
I did have the issue of #17 but adding the # to the selector fixed that.
I now have the following issue, which related to d3 but cannot see anything similar in their issues...
TypeError: Cannot set property 'singleArc' of undefined
    at setupD3 (modules.js?hash=b610b31…:18632)
    at Object.setupAndRender (modules.js?hash=b610b31…:18636)
    at PersonalitySunburstChart.show (modules.js?hash=b610b31…:18016)
    at Blaze.View.<anonymous> (app.js?hash=d456a3f…:6892)
    at blaze.js?hash=983d07a…:1875
    at Function.Template._withTemplateInstanceFunc (blaze.js?hash=983d07a…:3687)
    at blaze.js?hash=983d07a…:1873
    at Object.Blaze._withCurrentView (blaze.js?hash=983d07a…:2214)
    at viewAutorun (blaze.js?hash=983d07a…:1872)
    at Tracker.Computation._compute (tracker.js?hash=9f8a0ce…:339)

Looks like a couple of spots... this is how I am calling it...
var chart = new PersonalitySunburstChart({
            'selector': '#sunburstChart', 'version': 'v3'
        });
        chart.show(currentField.value.personalityProfile);

Looks like https://github.com/personality-insights/sunburst-chart/blob/6ebc228b612ba3c2e4fdb1acc61f2c8092f4d5a9/lib/personality-chart-renderer.js#L592
I have the following script file in my header    <script src="https://d3js.org/d3.v4.min.js"></script>
Any ideas pls?
thanks

I got it to work, I changed the d3 library to the one in the example, not the one recommended on the d3 web site.
That said my visualization in grayscale, not colour... is there some param for colour?

This should be fixed with the latest update.

This is related to the version of d3 you are using. This library doesn't work with D3 v4.
If you are using v4 then d3.svg is undefined so d3.svg. singleArc will throw TypeError: Cannot set property 'singleArc' of undefined
See https://keithpblog.org/post/upgrading-d3-from-v3-to-v4/


⚠ Bofrost.fr has degraded performance

In 34ea394, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1901 ms

Resolved: Bofrost.fr performance has improved in 2b088f8 after 13 minutes.


Allow returning data when onConflict is set to ignore

Currently, when the onConflict action is set to ignore and returnSelect is not false, BigAl will throw Unknown error getting created rows back from the database when there is a conflict creating a record.
Best solution would be to find a way to return data in that case. Secondarily, preventing BigAl from allowing ignore with return data would be less desirable.
See: https://stackoverflow.com/a/42217872/3085

Would it be the worst to, under the hood, set onConflict to merge with a noop update, in order to get the data sent back?


Remove Richard's serial numbers from the metadata form and file name if he wants to only use the spreadsheet as input

We actually want to keep Richard's serial number in the metadata form and output.


Carbonite.Transaction meta data persists across subsequent transactions when using fixtures

Description
I am experiencing an issue where a Carbonite.Transaction created in a fixture setup persists into subsequent tests. As a result, the update operation is being associated with the same note_created transaction instead of a new note_updated transaction.
  test "update_note/2 with valid data updates the note" do
    note = note_fixture()
    update_attrs = %{title: "some updated title", body: "some updated body"}
    assert {:ok, %Note{} = note} = Notes.update_note(note, update_attrs)
    assert note.title == "some updated title"
    assert note.body == "some updated body"

    meta =
      Carbonite.Query.current_transaction()
      |> CarboniteIssueDemo.Repo.one!()
      |> Map.fetch!(:meta)

    # This assertion fails because the meta type is "note_created"
    assert ^meta = %{"type" => "note_updated"}
  end

Environment
❯ elixir --version
Erlang/OTP 27 [erts-15.1.2] [source] [64-bit] [smp:10:10] [ds:10:10:10] [async-threads:1] [jit] [dtrace]

Elixir 1.17.3 (compiled with Erlang/OTP 27)

❯ psql --version
psql (PostgreSQL) 15.8 (Homebrew)

Steps to Reproduce

Clone the repository: carbonite_issue_demo
Run the following test file: notes_test.exs

Test result
❯ mix test test/carbonite_issue_demo/notes_test.exs
Running ExUnit with seed: 66933, max_cases: 20

....[test/carbonite_issue_demo/notes_test.exs:51: CarboniteIssueDemo.NotesTest."test notes update_note/2 with valid data updates the note"/1]
Carbonite.Query.transactions(preload: true) #=> #Ecto.Query<from t0 in Carbonite.Transaction, prefix: "carbonite_default",
 preload: [changes: #Ecto.Query<from c0 in Carbonite.Change, prefix: "carbonite_default">]>
|> CarboniteIssueDemo.Repo.all() #=> [
  %Carbonite.Transaction{
    __meta__: #Ecto.Schema.Metadata<:loaded, "carbonite_default", "transactions">,
    id: 64894,
    xact_id: 60662,
    meta: %{"type" => "note_created"},
    inserted_at: ~U[2024-10-26 05:43:35.206181Z],
    changes: [
      %Carbonite.Change{
        __meta__: #Ecto.Schema.Metadata<:loaded, "carbonite_default", "changes">,
        id: 328,
        op: :insert,
        table_prefix: "public",
        table_name: "notes",
        table_pk: ["243"],
        data: %{
          "body" => "some body",
          "id" => 243,
          "inserted_at" => "2024-10-26T05:43:35",
          "title" => "some title",
          "updated_at" => "2024-10-26T05:43:35"
        },
        changed: [],
        changed_from: nil,
        transaction_id: 64894,
        transaction: #Ecto.Association.NotLoaded<association :transaction is not loaded>,
        transaction_xact_id: 60662
      },
      %Carbonite.Change{
        __meta__: #Ecto.Schema.Metadata<:loaded, "carbonite_default", "changes">,
        id: 329,
        op: :update,
        table_prefix: "public",
        table_name: "notes",
        table_pk: ["243"],
        data: %{
          "body" => "some updated body",
          "id" => 243,
          "inserted_at" => "2024-10-26T05:43:35",
          "title" => "some updated title",
          "updated_at" => "2024-10-26T05:43:35"
        },
        changed: ["body", "title"],
        changed_from: nil,
        transaction_id: 64894,
        transaction: #Ecto.Association.NotLoaded<association :transaction is not loaded>,
        transaction_xact_id: 60662
      }
    ]
  }
]

  1) test notes update_note/2 with valid data updates the note (CarboniteIssueDemo.NotesTest)
     test/carbonite_issue_demo/notes_test.exs:42
     match (=) failed
     The following variables were pinned:
       meta = %{"type" => "note_created"}
     code:  assert ^meta = %{"type" => "note_updated"}
     left:  ^meta
     right: %{"type" => "note_updated"}
     stacktrace:
       test/carbonite_issue_demo/notes_test.exs:59: (test)

...
Finished in 0.1 seconds (0.00s async, 0.1s sync)
8 tests, 1 failure

Additional Information
I tried resolving this issue by using override_mode, as I thought it might help. However, it didn’t work as expected. Even when following Option 1 from the documentation, the test still fails as shown in the attached result. While the changes record for the note creation is indeed skipped, the meta[:type] is still "note_created". You can reproduce this issue on the following branch: here.

Option 1 test result
❯ mix test test/carbonite_issue_demo/notes_test.exs
Running ExUnit with seed: 491934, max_cases: 20

[test/carbonite_issue_demo/notes_test.exs:51: CarboniteIssueDemo.NotesTest."test notes update_note/2 with valid data updates the note"/1]
Carbonite.Query.transactions(preload: true) #=> #Ecto.Query<from t0 in Carbonite.Transaction, prefix: "carbonite_default",
 preload: [changes: #Ecto.Query<from c0 in Carbonite.Change, prefix: "carbonite_default">]>
|> CarboniteIssueDemo.Repo.all() #=> [
  %Carbonite.Transaction{
    __meta__: #Ecto.Schema.Metadata<:loaded, "carbonite_default", "transactions">,
    id: 64898,
    xact_id: 60675,
    meta: %{"type" => "note_created"},
    inserted_at: ~U[2024-10-26 05:47:51.584674Z],
    changes: [
      %Carbonite.Change{
        __meta__: #Ecto.Schema.Metadata<:loaded, "carbonite_default", "changes">,
        id: 332,
        op: :update,
        table_prefix: "public",
        table_name: "notes",
        table_pk: ["246"],
        data: %{
          "body" => "some updated body",
          "id" => 246,
          "inserted_at" => "2024-10-26T05:47:51",
          "title" => "some updated title",
          "updated_at" => "2024-10-26T05:47:51"
        },
        changed: ["body", "title"],
        changed_from: nil,
        transaction_id: 64898,
        transaction: #Ecto.Association.NotLoaded<association :transaction is not loaded>,
        transaction_xact_id: 60675
      }
    ]
  }
]

  1) test notes update_note/2 with valid data updates the note (CarboniteIssueDemo.NotesTest)
     test/carbonite_issue_demo/notes_test.exs:42
     match (=) failed
     The following variables were pinned:
       meta = %{"type" => "note_created"}
     code:  assert ^meta = %{"type" => "note_updated"}
     left:  ^meta
     right: %{"type" => "note_updated"}
     stacktrace:
       test/carbonite_issue_demo/notes_test.exs:59: (test)

.......
Finished in 0.1 seconds (0.00s async, 0.1s sync)
8 tests, 1 failure

Hey @yasutakekazu
you're not fully understanding override_mode: This only disables the trigger itself so that statements on your notes table won't raise exceptions if you don't insert a Carbonite.Transaction before. But in your option1 branch, even with the override_mode in place, you do insert a Carbonite.Transaction in your Notes.create_note/1 call in the test fixture. You need to change your fixture code to use override_mode and not insert a transaction (e.g. by making that optional somehow in your domain logic, or by creating the note differently, e.g. with custom factory code or ex_machina or so). Finally, why the note_created transaction wins over the note_updated one is documented here.

I appreciate the detailed guidance and link to the documentation. Thank you for the clarification!


Add character: Agent Prodder

Hero Description
A secret agent with a very convenient briefcase.
Concept Art
Inspiration
Insipred by MIB agents

Moves
Idle

Hurt

[STICK] Walk

[LEFT] Memory Flash
Prodder uses a device to erase people's memory in front of him. It causes a small amount of damage and silence for X seconds.

[MIDDLE] Portal Beacon
Prodder sets a beacon on the floor, activating a portal. (He'll be able to move sooner than the gif shows).

Whenever the player clicks middle again, Prodder will teleport back to the beacon place and it will stop working. After that, the player must set up another beacon.

(He'll be able to move as soon as he leaves the portal).

[Q] The Briefcase
The briefcase transforms into a powerful automatic rifle for a limited amount of time. He may walk freely while holding the rifle.

 The briefcase will turn back to normal once the time runs out.

OPTIONAL
[E] TAUNT

[SPACE]/Shift
Prodder brings his briefcase in front of him and it protects him from most incoming attacks and projectiles

Tasks

[x] Idle
[x] Walk
[x] [LEFT] Memory Flash
[x] [MIDDLE] Portal Beacon
[x] [RIGHT] Mechanical Arm
[x] [Q] The Briefcase

Approved for work!

Daily 01/05
Took me 3h
Idle + Left

Daily 02/05
Took me 5h30
Walk + Middle

Daily 03/05
Took me 5h
Middle + Right

Daily 05/05
Took me 3h45
Right + Ultimate

Daily 06/05
Took me 2h30
Ultimate + Defense + Hurt + Taunt


Added probe for statefulset health

Relates to https://github.com/form3tech/k6-tests/issues/425

Codecov Report

Merging #4 into master will increase coverage by 0.43%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
+ Coverage   86.45%   86.89%   +0.43%
==========================================
  Files          10       11       +1
  Lines         635      656      +21
==========================================
+ Hits          549      570      +21
  Misses         86       86

Impacted Files
Coverage Δ

chaosk8s/statefulset/probes.py
100.00% <100.00%> (ø)

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 93a472f...7d20f52. Read the comment docs.


Fix completion with enter key

The plugin is working great when the completion is done with the mouse (i.e. clicking on a suggestion with the mouse), but is buggy when a suggestion is chosen with the enter key.

Hi @tleruitte, thanks for the patch. It looks good to me. I'll bump the version :+1:


Dockerfile: create symlink of Dockerfile.builder to envoy/

This will allow to use docker hub to build the this docker image without requiring a developer to build it and upload it manually.
Signed-off-by: André Martins<EMAIL_ADDRESS>

test-me-please

Would appreciate context; can you update the description of the PR to explain why this is needed?

Would appreciate context; can you update the description of the PR to explain why this is needed?

@ianvernon  done

test-me-please

The title indicates a symlink, but I see the file was moved instead.

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please


Set the NOMINMAX Macro in the Build

Windows.h, for a number of probably good reasons at the time, #defines min and max. These of course conflict with std::min and std::max methods.
Windows also offers NOMINMAX, which avoids those defines.

Is there a reason why you prefer
<AdditionalOptions>/DNOMINMAX</AdditionalOptions>
instead of
<PreprocessorDefinitions>...;DNOMINMAX</PreprocessorDefinitions>
?
I think it would be more consistent to see all build options under the PreprocessorDefinitions, in one central spot. Because it is easy to not notice the extra option if it is only explicitly added to additional options. It also forces you to go into a separate UI screen in Visual Studio to see it, i.e. additional opportunity for a failure.
If we can avoid doing that #define NOMINMAX, completely.. I feel we'd have less unpredictable interactions with other code that may be building the SDK from source for Windows OS, in non-standard build environments where they may also be needing the original definitions of these macros. We could have wrapped all usages of std::min and std::max like this:
(std::min)(a, b); // added parentheses around function name
(std::max)(a, b);

This is search-and-replace in 11 files. But should be rather simple change. We'd still have to keep an eye on any other new usage of std::min and std::max. But it would not require defining a NOMINMAX. Rather hard to tell what exact spot in Windows still require their original macro definition.

It's about the same scope: 10 project files with define versus (less than) 11 code file targeted fixes to use (std::min) and (std::max) - that'd fix it in less invasive manner, with about the same amount of total lines changed.

There's a <PreprocessorDefinitions> blob? Heck yes I'll use that!

There's a  blob? Heck yes I'll use that!

Yeah, that'd be much cleaner. I mean - I am OK if you do that. I would have probably preferred if we did not need the macro definition at all, but having it in PreprocessorDefinitions should be fine too.


Update Login.vue, input type switch

Into Login form, password input: onclick on svg icon --> toggle text/password input type.

Neat idea @alessandrofuda, but this is more than I want in this demo project, which is designed to primarily illustrate how Inertia.js works. I want to keep this source code as simple as possible. 👍
Thanks either way for your interest in this project!


[api-major] Remove some deprecated constants

All of the these constants have been deprecated for a while, and with the upcoming major version this seems like a good time to remove them. For the string-constants we can simply remove them, but the number-constants are left commented out since we don't want to re-number the list to prevent third-party breakage.

/botio test


fix: bump codeaction version requirement

Fixes issue where it throws error on version 10.0 as described in this issue: https://github.com/nvim-telescope/telescope-ui-select.nvim/issues/29

@Conni2461 Could you take a look at this when you find the time?

I am looking into it right now. but tbh i dont get the problem. if you have an outdated neovim nightly build, then telescope doesnt support that. if you decide to run neovim nightly we require an up to date build.

Neovim (v0.9.0) or the latest neovim nightly commit is required for telescope.nvim to work.

from the current readme. i merged the original branch with this version check because it worked with the latest nightly build at that time.

i just build current master resulting in this version
NVIM v0.10.0-dev-1641+g53cc835b5
Build type: RelWithDebInfo
LuaJIT 2.1.1700008891
Run "nvim -V1 -v" for more info

running lua print(vim.version.cmp(vim.version(), vim.version.parse "0.10.1-dev")) results in -1 running lua print(vim.version.cmp(vim.version(), vim.version.parse "0.10")) as well lua print(vim.version.cmp(vim.version(), vim.version.parse "0.10.0")) as well.
only lua print(vim.version.cmp(vim.version(), vim.version.parse "0.10.0-dev")) results in 0
running codeactions on this version works fine. It also works perfectly fine on nvim-0.9.4 (which is my daily driver version). Now checking out this branch breaks neovim nightly version i just compiled, 0.9.4 still works

So i've have 0 idea which neovim version you guys are running, and i dont see how this fixes anything. i need more information whats actually wrong here.

no idea what version 10.0 means, ngl

this is the last nightly build which still shows a -dev version: https://github.com/neovim/neovim/releases/tag/nightly

Turns out it was an outdated build and I was unaware. Closing this, thanks @Conni2461.


Missed php module ext-xmlwriter

Hi,
I tried to use phpunit/phpunit:5.7.5
During running command composer install happened following error:
Loading composer repositories with package information
Installing dependencies (including require-dev) from lock file
Your requirements could not be resolved to an installable set of packages.
Problem 1
- Installation request for phpunit/php-code-coverage 4.0.8 -> satisfiable by phpunit/php-code-coverage[4.0.8].
- phpunit/php-code-coverage 4.0.8 requires ext-xmlwriter * -> the requested PHP extension xmlwriter is missing from your system.
Problem 2
- phpunit/php-code-coverage 4.0.8 requires ext-xmlwriter * -> the requested PHP extension xmlwriter is missing from your system.
- phpunit/phpunit 5.5.7 requires phpunit/php-code-coverage ^4.0.1 -> satisfiable by phpunit/php-code-coverage[4.0.8].
- Installation request for phpunit/phpunit 5.5.7 -> satisfiable by phpunit/phpunit[5.5.7].
To enable extensions, verify that they are enabled in your .ini files:
- /etc/php7/php.ini
- /etc/php7/conf.d/00_bcmath.ini
- /etc/php7/conf.d/00_ctype.ini
- /etc/php7/conf.d/00_curl.ini
- /etc/php7/conf.d/00_dom.ini
- /etc/php7/conf.d/00_json.ini
- /etc/php7/conf.d/00_mbstring.ini
- /etc/php7/conf.d/00_mcrypt.ini
- /etc/php7/conf.d/00_opcache.ini
- /etc/php7/conf.d/00_openssl.ini
- /etc/php7/conf.d/00_pcntl.ini
- /etc/php7/conf.d/00_pdo.ini
- /etc/php7/conf.d/00_session.ini
- /etc/php7/conf.d/00_soap.ini
- /etc/php7/conf.d/00_tokenizer.ini
- /etc/php7/conf.d/00_xml.ini
- /etc/php7/conf.d/00_zip.ini
- /etc/php7/conf.d/00_zlib.ini
- /etc/php7/conf.d/01_exif.ini
- /etc/php7/conf.d/01_mysqlnd.ini
- /etc/php7/conf.d/01_pdo_pgsql.ini
- /etc/php7/conf.d/01_pdo_sqlite.ini
- /etc/php7/conf.d/01_phar.ini
- /etc/php7/conf.d/01_xmlreader.ini
- /etc/php7/conf.d/02_pdo_mysql.ini
- /etc/php7/conf.d/xdebug.ini
You can also run php --ini inside terminal to see which files are used by PHP in CLI mode.
I think this error was happened by missed php extension ext-xmlwriter which requires in the phpunit/php-code-coverage package for version 4.0.4 and higher.

having the same error.  no update on this?

Update your alpine:latest image by running docker pull alpine:latest

It doesn't work:
Igors-Mac-mini-2:fb-comments-hider igortregub$ docker pull alpine:latest
latest: Pulling from library/alpine
Digest: sha256:1072e499f3f655a032e88542330cf75b02e7bdf673278f701d7ba61629ee3ebe
Status: Image is up to date for alpine:latest
Igors-Mac-mini-2:fb-comments-hider igortregub$ docker run -v $(pwd):/app -it --entrypoint /bin/bash phpunit/phpunit:5.7.5
bash-4.3# cd www/ && composer install
Loading composer repositories with package information
Installing dependencies (including require-dev) from lock file
Your requirements could not be resolved to an installable set of packages.
Problem 1
- phpunit/php-code-coverage 4.0.8 requires ext-xmlwriter * -> the requested PHP extension xmlwriter is missing from your system.
- phpunit/php-code-coverage 4.0.8 requires ext-xmlwriter * -> the requested PHP extension xmlwriter is missing from your system.
- Installation request for phpunit/php-code-coverage 4.0.8 -> satisfiable by phpunit/php-code-coverage[4.0.8].
To enable extensions, verify that they are enabled in your .ini files:
- /etc/php7/php.ini
- /etc/php7/conf.d/00_bcmath.ini
- /etc/php7/conf.d/00_ctype.ini
- /etc/php7/conf.d/00_curl.ini
- /etc/php7/conf.d/00_dom.ini
- /etc/php7/conf.d/00_json.ini
- /etc/php7/conf.d/00_mbstring.ini
- /etc/php7/conf.d/00_mcrypt.ini
- /etc/php7/conf.d/00_opcache.ini
- /etc/php7/conf.d/00_openssl.ini
- /etc/php7/conf.d/00_pcntl.ini
- /etc/php7/conf.d/00_pdo.ini
- /etc/php7/conf.d/00_session.ini
- /etc/php7/conf.d/00_soap.ini
- /etc/php7/conf.d/00_tokenizer.ini
- /etc/php7/conf.d/00_xml.ini
- /etc/php7/conf.d/00_zip.ini
- /etc/php7/conf.d/00_zlib.ini
- /etc/php7/conf.d/01_exif.ini
- /etc/php7/conf.d/01_mysqlnd.ini
- /etc/php7/conf.d/01_pdo_pgsql.ini
- /etc/php7/conf.d/01_pdo_sqlite.ini
- /etc/php7/conf.d/01_phar.ini
- /etc/php7/conf.d/01_xmlreader.ini
- /etc/php7/conf.d/02_pdo_mysql.ini
- /etc/php7/conf.d/xdebug.ini
You can also run php --ini inside terminal to see which files are used by PHP in CLI mode.
bash-4.3#