Add support for Copilot.Microsoft.com

Hi, this pull request should also add support for the new Bing Chat / Copilot URL, https://copilot.microsoft.com.
Uh... the README.md is much more different though as this is from my Fork of this repo so feel free to update README.

Why is support needed for https://copilot.microsoft.com/? It works perfectly for me without the add-on on Firefox.

It still has the conversation limit. Although now microsoft allows people to use copilot in other browsers, they set a limit to other non-edge browsers. On Microsoft edge, the conversation limit is 30. But on other browsers, it is 5.

From: Ian Kavanagh @.>
Sent: 17 June 2024 12:24 PM
To: IKavanagh/BingChatInFirefox @.>
Cc: EinKaffeeBitte @.>; Author @.>
Subject: Re: [IKavanagh/BingChatInFirefox] Add support for Copilot.Microsoft.com (PR #9)
Why is support needed for https://copilot.microsoft.com/? It works perfectly for me without the add-on on Firefox.
—
Reply to this email directly, view it on GitHubhttps://github.com/IKavanagh/BingChatInFirefox/pull/9#issuecomment-2173130147, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BG7AVXQR4D27Q4V2BUK4WGDZH3BPXAVCNFSM6AAAAABJNYORJWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZTGEZTAMJUG4.
You are receiving this because you authored the thread.Message ID: @.***>

Also, please remove now the sidebar as it doesn't work. It says that the site doesn't let the browser display it in an iframe.

From: Ian Kavanagh @.>
Sent: 17 June 2024 12:24 PM
To: IKavanagh/BingChatInFirefox @.>
Cc: EinKaffeeBitte @.>; Author @.>
Subject: Re: [IKavanagh/BingChatInFirefox] Add support for Copilot.Microsoft.com (PR #9)
Why is support needed for https://copilot.microsoft.com/? It works perfectly for me without the add-on on Firefox.
—
Reply to this email directly, view it on GitHubhttps://github.com/IKavanagh/BingChatInFirefox/pull/9#issuecomment-2173130147, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BG7AVXQR4D27Q4V2BUK4WGDZH3BPXAVCNFSM6AAAAABJNYORJWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZTGEZTAMJUG4.
You are receiving this because you authored the thread.Message ID: @.***>


minor bug in pytorch code?

I am using this script in pytorch available in your repository.
Currently, this line is
cam = weight_softmax[class_idx].dot(feature_conv.reshape((nc, h*w)))
But I think it should be,
cam = weight_softmax[idx].dot(feature_conv.reshape((nc, h*w)))
class_idx should be replaced by idx, becuase class_idx is a list, idx is a integer.

I agree.

Thanks. Fixed.

Thanks.


:mailbox_closed: Handle not found postcode from lookup

This handles an unknown postcode from the lookup with postcode.io
Tests in place to ensure the correct view is returned.
I've extracted the message into its' own module as it was being duplicated in a number of places.
Fixes #80

Closing as this has been superseded by #117


Minor field alignment fix

I merge it, but I'm not sure about it. I'd be not happy to find out different monitors have different alignment needs.

Hmm... Ok. I can try it out on different resolutions here. Easy fix would
be to put them in a table and align left.
On 15 Nov 2017 13:11, "nopara73"<EMAIL_ADDRESS>wrote:
I merge it, but I'm not sure about it. I'd be not happy to find out
different monitors have different alignment needs.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nopara73/HiddenWallet/pull/88#issuecomment-344587929,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYgLfaDAOtcr2PRAw0zkVrT2eSjgRWbJks5s2uMEgaJpZM4QexMN
.

It ruined it for me.

Could you send a screenshot of yours to make sure that works well?
Yes, it must be table if yours works well.

Ah, it was misaligned on mine when I got your version. I thought you had
just missed it. My"fix" obviously only works on my monitor.
We'll need a table then I guess.
On 15 Nov 2017 13:16, "nopara73"<EMAIL_ADDRESS>wrote:

It ruined it for me.
Could you send a screenshot of yours to make sure that works well?
Yes, it must be table if yours works well.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nopara73/HiddenWallet/pull/88#issuecomment-344589207,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYgLfafEnrGOidpCsHRHFmTBNpymYAYEks5s2uRIgaJpZM4QexMN
.


Set printWidth to 80 in .prettierrc.js

See my comment here: https://github.com/sharegate/craco/pull/17/files/03aa426fa63753649719ba57756042ea3394dc2b
My PR included a really long, and word-wrapping was not turned on by default in VS Code. I think it would be a good idea to set printWidth to 80-100.
However, the other option is to enable word-wrapping by default in the VS Code workspace settings. Here is a PR for that: #18
(The default IDE wrapping looks really bad, and makes it difficult to format multi-line strings properly. But that would work as well.)

Interesting, let me think about this and come back to you tonight!
Thank you


Add allow_self_issued boolean to input descriptors

We should add an allow_self_issued boolean to input descriptors so that verifiers can explicitly note that an input can be fulfilled via the user submitting their own data.

can this be closed now that #378 is merged?


Where to get the source data ?

That's a good question, actually.  We need to upload it someplace that can handle sets that big - even the reduced dataset we're working with for testing purposes is almost 3 gigs.

@facepalm Is any update for this ?


Getting mod issue

It would not allow me to download the file for the mod

it is saying it is not the right version for gorrila tag


Ensure we don't update the input value on mouseenter

Fix #74

Coverage increased (+0.02%) to 75.448% when pulling 0f281f17d6ad65c49ac34cc06a261797aaf6d610 on fix/74 into c6e78c1c672a8db25e3fe312a9c52e16de29ce86 on develop.

LGTM
That's a lot of flag passing through events/functions :D

That's a lot of flag passing through events/functions :D

Yeah, not perfect :/


chore: npx pepr dev does not wait for resources to be fully up

Description
npx pepr deploy does not wait for resources to be fully up. Added wait to cli/deploy.ts so that it will not continue until all replicas are up.
Related Issue
Fixes #419
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Other (security config, docs update, etc)

Checklist before merging

[x] Test, docs, adr added or updated as needed
[x] Contributor Guide Steps followed

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (main@137be15). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files

@@           Coverage Diff           @@
##             main     #435   +/-   ##
=======================================
  Coverage        ?   83.21%
=======================================
  Files           ?       23
  Lines           ?      727
  Branches        ?      123
=======================================
  Hits            ?      605
  Misses          ?      114
  Partials        ?        8

Flag
Coverage Δ

pull_request
83.21% <0.00%> (?)

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


2.3.1 hangs at exit

Pip package version: 2.3.1
vsCode extension version: v2.3.1
If I run the following file in vsCode:
import ocp_vscode
ocp_vscode.show()
print("Hello")

it will hang indefinitely. If you then press Ctrl-C:
Hello
^CException ignored in: <module 'threading' from '/opt/homebrew/Caskroom/micromamba/envs/cad/lib/python3.12/threading.py'>
Traceback (most recent call last):
  File "/opt/homebrew/Caskroom/micromamba/envs/cad/lib/python3.12/threading.py", line 1622, in _shutdown
    lock.acquire()

This may or may not be related but this computer did not have the Jupyter vsCode extension installed, although this was not an issue in 2.2.2

I saw this when running the unit tests, but it didn't happen when running in VSCode.
I'll try to reproduce

Found the problem.
I remove closing websockets after each call (saves up to a second).
However, is a big issue at the end. Need to see what I can do.
Uncomment the following lines and it should behave like before:
https://github.com/bernhard-42/vscode-ocp-cad-viewer/blob/b6182c681ad86166aa6f320793ff668e1fc1f2d5/ocp_vscode/comms.py#L145-L148

For reference
https://github.com/python-websockets/websockets/issues/1455

This feels like a hack but it may solve the issue: https://docs.python.org/3/library/atexit.html

I'll start with trying to fix the thread thing as mentioned above. But if that doesnt work, atexit is a good next step 😀

pip install --upgrade git+https://github.com/python-websockets/websockets.git@f0398141d2efd28f64d8e1d6d9adc179a9e5e334
This commit (from last week) fixes the issue. Need to check whether websockets 12.0 is compatible. But this would be the easiest fix
btw. atexit fails, since the program is waiting for the thread => no exit happens 😀

@MatthiasJ1 Could you maybe try the the pip installation of websockets on your machine? pip will complain tht ocp_vscode wants websockets 11, but that can be ignored.
For me it solves the problem

Installing the latest released version does not fix the issue but installing the dev version from github does fix it.

arrg, pypi doesn't allow direct links as dependencies. So, can't package this simple fix. Need to find another way

Sorry for the late reply, just confirming that it is fixed for me. Thanks for looking into this.


Use Appropriate Error Code for timeout during VM Deletion

What would you like to be added:
Currently, timeout during VM Deletion in Azure results in the generic error code Internal being returned to the MCM. See driver.DeleteVirtualMachine .  This is then set in the Machine.LastOperation.Description.
We want the error code to be changed to the appropriate code codes.DeadlineExceeded
Why is this needed:
See gardener landscape issue 125
This will allow us the opportunity to use a more suitable higher level error code in gardener ERR_INFRA_DEPENDENCIES to avoid repetitive multiple VO alerts.

/close
based on the comment on landscape issue 125


JS Memory Heap issue

Issue Description
As we've migrated terra packages to terra-framework, we've started to hit memory issues running the wdio tests.

Output from Travis CI build
><EMAIL_ADDRESS>wdio /app
> wdio node_modules/terra-dev-site/config/wdio/wdio.conf.js "--suite" "suite1"
[Terra-Toolkit:serve-static] Starting Webpack compilation
[BABEL] Note: The code generator has deoptimised the styling of "/app/dev-site-config/build/routeConfig.js" as it exceeds the max of "500KB".
<--- Last few GCs --->
[15:0x3709e80]   256747 ms: Mark-sweep 1365.9 (1536.4) -> 1365.1 (1549.4) MB, 1831.9 / 0.0 ms  allocation failure GC in old space requested
[15:0x3709e80]   258603 ms: Mark-sweep 1365.1 (1549.4) -> 1364.9 (1510.4) MB, 1856.3 / 0.0 ms  last resort GC in old space requested
[15:0x3709e80]   260446 ms: Mark-sweep 1364.9 (1510.4) -> 1364.9 (1505.9) MB, 1842.9 / 0.0 ms  last resort GC in old space requested
<--- JS stacktrace --->
==== JS stack trace =========================================
Security context: 0xc8a2bfa5ee1 <JSObject>
   2: update [crypto.js:99] [bytecode=0x34a8f5368e49 offset=30](this=0x64920ce3149 <Hash map = 0xef8ef1adcd9>,data=0x64920ce24d9 <Very long string[31365661]>,encoding=0x247723602de9 <String[6]: buffer>)
   3: arguments adaptor frame: 1->2
   5: /* anonymous */(aka /* anonymous */) [/app/node_modules/ssri/index.js:204] [bytecode=0x2e0df3d33e11 offset=32](this=0x247723602311 <undefined>,acc=0x64...
FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
1: node::Abort() [node]
2: 0x121a42c [node]
3: v8::Utils::ReportOOMFailure(char const*, bool) [node]
4: v8::internal::V8::FatalProcessOutOfMemory(char const*, bool) [node]
5: v8::internal::Factory::NewRawTwoByteString(int, v8::internal::PretenureFlag) [node]
6: v8::internal::String::SlowFlatten(v8::internal::Handle<v8::internal::ConsString>, v8::internal::PretenureFlag) [node]
7: v8::internal::String::Flatten(v8::internal::Handle<v8::internal::String>, v8::internal::PretenureFlag) [node]
8: v8::String::WriteUtf8(char*, int, int*, int) const [node]
9: node::StringBytes::Write(v8::Isolate*, char*, unsigned long, v8::Local<v8::Value>, node::encoding, int*) [node]
10: node::StringBytes::InlineDecoder::Decode(node::Environment*, v8::Local<v8::String>, v8::Local<v8::Value>, node::encoding) [node]
11: node::crypto::Hash::HashUpdate(v8::FunctionCallbackInfo<v8::Value> const&) [node]
12: v8::internal::FunctionCallbackArguments::Call(void (*)(v8::FunctionCallbackInfo<v8::Value> const&)) [node]
13: 0xb8ff3c [node]
14: v8::internal::Builtin_HandleApiCall(int, v8::internal::Object**, v8::internal::Isolate*) [node]
15: 0x37b0df38463d
Aborted (core dumped)
npm ERR! code ELIFECYCLE
npm ERR! errno 134
npm ERR<EMAIL_ADDRESS>wdio: `wdio node_modules/terra-dev-site/config/wdio/wdio.conf.js "--suite" "suite1"`
npm ERR! Exit status 134
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>wdio script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2018-12-11T22_31_33_634Z-debug.log
/home/travis/.travis/job_stages: line 523:  4792 Terminated              travis_jigger "${!}" "${timeout}" "${cmd[@]}"

The working idea is that this isn't an issue with WDIO but actually an issue happening during webpack compilition when compiling in production mode.

Moving this issue to terra-toolkit as the issue is more around our webpack config.


Bluetooth: host: RX queue improvements

Use an slist instead of a FIFO
Clean up the retrigger logic

Just a newb question: why is it better to use a slist than a fifo ?

Was thinking the same; my conclusion was that it saves memory as sys_slist_t is smaller than a struct k_fifo.

A FIFO is a kernel object with the ability to block on. An slist is a userspace pointer as the head of a linked list. Much simpler and without kernel interaction.


Implement a more proper snapshot format

The current snapshot format is only loosely established, so we add a more concrete snapshot format, which is more amenable to refactoring. This snapshot format is still temporary -- we will implement snapshots properly in a later package.
#426 is near to being ready, but it is currently still failing some tests related to snapshots. Establishing a snapshot format makes bug-hunting in the snapshot code easier.

So unless I missed anything, this is mostly just moving the interim impl of snapshots from .Internal (and .Config and .RunNumber) into .Snapshot. Which is fine.

That, and the (interim) format of a snapshot is more explicit


“No Gestures While Using Keyboard” Don’t Work

When this option is checked, it continues to work with keyboard

The feature is currently not supported on iOS 13.4 and up. Compatibility will hopefully be added in the future.

Ok ok, thanks very much. Great job!


hashsum: when the filename contains some special chars, escape them

Should fix tests/cksum/md5sum-newline.pl

Congrats! The gnu test tests/cksum/md5sum-newline is no longer failing!

Great :)


⚠ Jenkins Plugins Artifactory has degraded performance

In 24db74d, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1228 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 7c0dea1 after 16 minutes.


Push integration doesnt work, an example with new release?

Hi,
my ratchet chat-server is on and the communication with clients is fantastic. Now i am want integrate the zmq functionality. Your example with push notification, i cannot get up and running. My first problem is the integration with autobahn.js.
Do you have an example, either with a recent release of autobahn.js or an alternative to autobahn.js? I tried to integrate autobahn.ws, but that does not work either.
Thank you so much.
Marco

You need to use the older version of autobahn in order for it to work. Unfortunately you can't get that from their website anymore. There is a fork on GitHub here https://github.com/sergeyvolkov/autobahn-old


l10n - Version 4.0 - French Translations

I tried to put the error messages in French but I always get the error in English.
For example when the field is empty when it is required (FormBuilderValidators.required (context)), I have the message "This field cannot be empty".
class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      debugShowCheckedModeBanner: DotEnv().env['DEBUG'] == 'true',
      theme: CTheme().generateTheme(),
      home: LoginScreen(),
      supportedLocales: [
        Locale('fr'),
      ],
      localizationsDelegates: [
        FormBuilderLocalizations.delegate,
        GlobalMaterialLocalizations.delegate,
        GlobalWidgetsLocalizations.delegate,
      ],
    );
  }
}

i follow this : https://pub.dev/packages/flutter_form_builder/versions/4.0.0-pre.5#l10n but without success...
Do you have an idea of the problem?
thank you in advance,
Cédric

Hi @cedricDevWP,
We currently don't have all translations and would actually appreciate PRs for more language translations.

Ok, I'll try to translate the sentences into French.
On the other hand I tried to follow the steps of the link which you transmitted to me, but the orders fail ...
Do you need to have special packages in addition?

I could translate the sentences into French if someone provided me with the documentation to be able to do so.
I tried the various commands from this link https://pub.dev/packages/flutter_form_builder/versions/4.0.0-pre.6 but without success.
Example :
e.g. To generate for French run: flutter pub run intl_translation:generate_from_arb --output-dir=lib/l10n --no-use-deferred-loading lib/localization/form_builder_localizations.dart lib/l10n/intl_fr.arb lib/l10n/intl_messages.arb

I found how translate, i was not on the right branch ....
https://github.com/danvick/flutter_form_builder/tree/version_4
Step for translate :
1 - In you project create file : lib/localization/form_builder_localizations.dart and paste code
2 - In your project create file : lib/l10n/intl_messages.arb and paste code
3 - In your project create file : lib/l10n/intl_xx.arb and translate code from 2
4 - Run : flutter pub run intl_translation:generate_from_arb --output-dir=lib/l10n --no-use-deferred-loading lib/localization/form_builder_localizations.dart lib/l10n/intl_xx.arb lib/l10n/intl_messages.arb
++

I close when i do a PR

pr #573

you're welcome :)

@danvick,
i have create the translation french, on the other hand it was not added like the supported languages:
File :
class _FormBuilderLocalizationsDelegate
    extends LocalizationsDelegate<FormBuilderLocalizations> {
  const _FormBuilderLocalizationsDelegate();

  @override
  bool isSupported(Locale locale) {
    return ['en', 'es'].contains(locale.languageCode);
  }

  @override
  Future<FormBuilderLocalizations> load(Locale locale) {
    return FormBuilderLocalizations.load(locale);
  }

  @override
  bool shouldReload(_FormBuilderLocalizationsDelegate old) {
    return false;
  }
}


Expose syncing with sonatype staging repo without maven central release

bintraySyncSonatypeStaging
I added it as a new task. It does the same as bintraySyncMavenCentral, except that it passes close flag set to false, so that artifacts are placed in sonatype staging repo, but not yet released.
Bintry version bump
This is required for PR to actually work, because the new version fixes bug with closing. It seems like 0.5.2 didn't get released to maven central, so this PR will have to wait until fixed.
Retries
Genesis of this change can be found in this travis build.
Log line 1546 says:
[error] java.lang.RuntimeException: failed to sync<EMAIL_ADDRESS>with maven central: {"status":"Validation Failed","messages":"[Missing Signature: '/org/quasar-analytics/quasar-common-internal_2.12/33.0.0/quasar-common-internal_2.12-33.0.0-sources.jar.asc' does not exist for 'quasar-common-internal_2.12-33.0.0-sources.jar'., Dropping existing partial staging repository.]"}

So it says, that file is missing, but it shouldn't be, because line 1249 says that it was published:
[info] 	published quasar-common-internal_2.12 to https://api.bintray.com/maven/slamdata-inc/maven-public/maven-public/org/quasar-analytics/quasar-common-internal_2.12/33.0.0/quasar-common-internal_2.12-33.0.0-sources.jar.asc

This is probably due to replicas are not yet consistent at that point of time. It can be observed that soon after publishing a version, some artifacts are visible, and some are not, or version doesn't exist at all, depending on when you refresh the page.
For this reason, to limit number of failed builds, I implemented retry mechanism. It retries according to bintraySyncMavenCentralRetries setting. If the seq is empty, it behaves just like before, and this is the default I set, to not alter behavior.
One can set it for example to:
bintraySyncMavenCentralRetries := {
  import scala.concurrent.duration._
  Seq(5.seconds, 1.minute, 5.minutes)
}

In case of exception from mavenSync task, (i.e. status other than 200 or 404) it will wait 5s ant try again, if it fails, it will retry after a minute, and finally after 5 minutes. If it still fails after 5, the exception is rethrown. If at any point it succeeds, result is returned as if nothing bad happened.
If this change is too custom or hacky, let me know, I will just remove the commit. It can be implemented in client project of this plugin in a similar way. Although I think it could be helpful if someone faces similar problem.

Is the bintraySyncSonatypeStaging same as process described in https://github.com/sbt/sbt-bintray#staging-optional ?

@2m I think not. The process that you linked stages artifacts in bintray, so they won't be available through your bintray repo, or jCenter if that is linked. At this point there is no mention of sonatype. Then it can be released to bintray with bintrayRelease. Artifacts appear in your repo and jCenter if linked. At this point we can start talking about this PR, because only once version is available in jCenter it can be synchronized with maven central through sonatype. This is the action described here: https://bintray.com/docs/api/#_sync_version_artifacts_to_maven_central Currently, the plugin always sends this request with close set to "1". This means that bintray will sync artifacts with nexus staging repository and close it causing release to maven central.
This PR, through bintraySyncSonatypeStaging task enables possibility to put your artifacts to sonatype staging repo but without closing and hence without putting them to maven central.
For a backstory, the ultimate goal is to, in a multi module/package project, on CI, run bintraySyncSonatypeStaging for all modules, and only once that succeeds, so all artifacts reach nexus and are valid, we can run bintraySyncMavenCentral  for each module to make them public in maven central. This is to avoid the situation when synchronizing of 10 packages works and for 10 packages it fails which puts incomplete product to maven central publicly.

Bumping this for attention.  (sorry!  it's blocking our release process right now) @jvican @eed3si9n?

Won't have time to have a look at this any time soon, sorry. I defer my review to @eed3si9n or @dwijnand.

This PR is now published on my personal bintray, so we'll see how well it works.  Hopefully this also alleviates the need for me to bug you guys over and over again; I know y'all are busy.  🙂
resolvers += Resolver.bintrayIvyRepo("djspiewak", "ivy")
addSbtPlugin("org.foundweekends" % "sbt-bintray" % "0.5.3+4-b634b339")

@2m This is still not verified due to some pretty crazy problem with bintray, but completely unrelated to to this. But regardless, retry mechanism seems to work and is useful.
Task to sync with staging repo without closing is possible thorugh bintray api, but not from the plugin, so I think it is a good addition as well. Even if rerunning with close trick wouldn't work it maybe can be worked around using sonatype plugin to close everything.
So, basically if you agree, could we merge and release this?


Add script execution abstraction

Background
We need a way to abstract the execution of a script so we can support Kubernetes Jobs as a script execution mechanism.
Results
This PR adds 3 new interfaces and some derived implementations
IScriptExecutor -> Represents a distinct way to execute a script
IScriptExecutorFactory -> A factory that constructs/returns an IScriptExecutor
IRunningScript -> Represents a running script
The intent is that there will be a KubernetesJobScriptExecutor and a KubernetesJobRunningScript implementations that will be used when the script is being executed via k8s jobs.
Shortcut story: [sc-61762]
How to review this PR

Quality :heavy_check_mark:

Pre-requisites

[ ] I have read How we use GitHub Issues for help deciding when and where it's appropriate to make an issue.
[ ] I have considered informing or consulting the right people, according to the ownership map.
[ ] I have considered appropriate testing for my change.

This is no longer relevant


[BUG]: update fail, levon signature

Athena OS version
Athena os
Which environment or technology are you using?

[X] Baremetal
[ ] VirtualBox Virtual Machine
[ ] VMware Virtual Machine
[ ] Hyper-V Virtual Machine
[ ] QEMU/KVM
[ ] Parallels
[ ] UTM
[ ] Container
[ ] WSL

Kernel version
6.1.70-lts
Involved software
No response
Repro steps
When updating getting error levon signature.
Latest iso just downloaded it
Fresh installation.
Even in the installation process i saw these error when installer tried to install packages
Expected behavior
To update
Actual behavior
Error
Diagnostic logs
No response

Download this file and
chmod + x strap.sh
sudo ./strap.sh
sudo pacman -Syy

I uploaded a new ISO where I removed BA. Download it and try.

Both. Use the link in the athenaos.org download link.


pkg/postgres: Fix connection leak when Listener errs out

Prevents leaking connections during reconnect phase of postgres failover.
This failure happened when reconnects happened fast enough that the pool would be fully consumed before the listener was re-established.
It always leaked connections but the effect was pronounced if multiple primary failovers happen as the connections are never reclaimed.
Closes #2439
/cc @temujin9 @titanous

Awesome find! LGTM.


Renderer Screenshot with an adjustable texture format

Hey nick, turns out I need a different format than RGBA, so I just added a parameter for changing the texture format. Worked as expected in the RenderCamera script.
Does it need support for all of the texture types? I just need it for BGRA which is the same buffer size. Maybe the others could be marked as unsupported for the time being?

This looks like an improvement to me! BGRA does kinda raise an interesting question though, what should happen if the user requests a screenshot in a format that's not valid for render textures? BGRA is not, afaik, universally supported for render target surfaces. Mostly on different platforms.

If you create a render target with an unsupported format, is this where it'd fail (or earlier)?
It turns out I don't need BGRA anymore, but would be happy to add some error handling in here for unsupported pixel formats!

I would expect this spot to be where the failure starts, that function call would have enough information to decide if it was a valid texture setup or not! But it does let execution fall through there, so it would probably halt and return a bit later when trying to create the first view item.

If the current error handling is a critical log, could we keep the same behavior here? Otherwise we'd have to change the return type of skg_tex_set_contents_arr to a bool or throw

Yeah, you're right, I think there's probably some work to be done here related to error reporting. But I don't think it's in need of immediate attention here, and can probably be a bit of a separate project later on. The logs should already highlight the core issues if they crop up!


Update mini solution of Devkit Translater

The existing library is still using AudioClass and we have provided AudioClassV2.
So we need to update this mini-solution

Please refer to PR: https://github.com/Microsoft/devkit-sdk/pull/859
we just check Chinese, do not check other languages.


Improve debug output in travis

Print body and headers if a test fails. Let's see what mailhog says.

-i show headers in output
-I fetch headers only
-s hide progress bar (and error messages unless -S)
-S show error messages.

Finally trying this out locally to try and debug the mailhog issue.
Added an option to skip cleanup and fixed an issue when BSD xargs params were used even if I had GNU xargs installed (first time I'm actually hit by a bug because of having gnu tools, it's usually the other way around with the scripts i use).

LGTM


Unable to select a different dataset after selecting one when adding a layer

STR:

Open a map using the new editor.
Click "Add" button on the Layers section.
Select one of the datasets under the "Dataset" tab.
Try to select a different dataset.

Expected result:

I can click and select datasets as many times as I want.

Actual result:

After clicking on one dataset, clicking on other datasets has no effect and I can't select a different dataset.

Here's a GIF where I click on a dataset and then click on others:

cc: @xavijam

Duplicated -> https://github.com/CartoDB/cartodb/issues/7426


Declare compatibility with PHP 8.x

Slim 4 should not be declared as compatible with PHP 8.

Coverage remained the same at 100.0% when pulling 9b325c5c37200245869cb12136811690202c59f5 on odan:patch-1 into 40c1fffd0334d2d6a418090eb2a03ca37c825a66 on slimphp:4.x.

I have just seen #3015. So this PR is not necessary.


Issue #293 sort

Going to try to get the current revisions done between tonight and tomorrow; @lil5 Though, I'd advise not bothering to do another review of it until check-box is merged, since i'll want to sync main with the sort-fork, in order to make sure everything is still functioning as it should.


Add submission count to download buton

This makes it easy to see how many submissions there are.
This won't work if there are multiple submission deadlines, but it's a start.

Given that we have a number of Blueshirt "teams" in there I'm not sure that this is actually useful?


401 on GitHub Storage login

https://github.com/Thinkmill/keystatic/assets/69188140/fd5c34d7-07c6-4934-9e61-6d8a05b24a8e

@JedWatson do you know?

Error message - redirect_uri mismatched

In the callback URLs in the app settings of the installed App that we added to our repo, you need to delete all of the localhosts, leaving only the full URL https//[your company name].com/api/keystatic/github/oauth/callback as a viable callback, that is what worked for me. Maybe it won't work on my local, but this is what I did.

I am experiencing the same error. Have you found another fix? The one described her I have tried already and is not really working.

I am experiencing the same error. Have you found another fix? The one described her I have tried already and is not really working.

Yeh, i chose to go with Keystatic cloud provider

Ps- my cms not working rn idk why

#837


Is reorder needed when all formats are "any"?

I'm not sure about one thing about the reorder operation. Suppose that I use MKL-DNN directly instead of through a higher layer framework with its own data formats and specifications. Suppose that I set all internal data formats as any so MKL-DNN can choose whatever fits well. Suppose also that I can feed the user data in MKL-DNN's preferred formats (nchw etc.). In that case, will MKL-DNN need to perform reorders for some operations? Do I still need to check for internal layers' src/dst memory format compatibility and perform reorders?

Thank you. Is there any automatic setting or flag to do that, or I should do this as in the examples, by comparing references and then issuing reorder if references do not match?

Per Intel MKL-DNN design all the reorders are explicit, so this have to be done in the same way as in the examples. You may want to build a layer on top that checks layouts and performs lazy conversions to make things easier.

Thank you.


fix: font color error

🐛 The bug

🛠 To reproduce

🌈 Expected behavior

ℹ Additional context
No response

Can you explain a bit more?

What information do you need? Nuxt or devtool version?

What do you mean? If even you don't know how to reproduce, how could we help?

We temporarily closed this due to the lack of enough information. We could not identify whether it was a bug or a userland misconfiguration with the given info.
Please provide a minimal reproduction to reopen the issue.
Thanks.

Why reproduction is required

@maicss  are you using nuxt-mongoose DB playground? it looks like that

@maicss  are you using nuxt-mongoose DB playground? it looks like that

Yes

Even when I change the theme of devtools form light to dark. Seems mongoose plugin does not change its theme with devtools. @arashsheyda

@maicss you should open an issue on nuxt-mongoose repository not here. thanks for noticing this


buildAndDeploy.yaml moved into .github/workflows folder

Current dependencies on/for this PR:

main

PR #11   👈

This comment was auto-generated by Graphite.


feat(types): Add InferQueryParams helper type

:tada: This PR is included in version 1.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


refactor update-to-tag command step (DAT-6641)

Impact

[ ] Bug fix (non-breaking change which fixes expected existing functionality)
[X] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Things to be aware of

Things to worry about

Additional Context

@StevenMassaro what is the replacement function since the update is deprecated and it is not clear/not mentioned in the Javadoc what clients can use? this is a breaking change for us as our build use of a deprecated is forbidden in our codebase..
appreciate your help.

Hello @sadanandshinde ! The replacement for update is calling the command framework directly - currently Liquibase.update command is just a wrapper for that. Here is a sample:
            CommandScope updateCommand = new CommandScope(UpdateCommandStep.COMMAND_NAME);
            updateCommand.addArgumentValue(DbUrlConnectionCommandStep.DATABASE_ARG, getDatabase());
            updateCommand.addArgumentValue(UpdateCommandStep.CHANGELOG_FILE_ARG, changeLogFile);
            updateCommand.addArgumentValue(UpdateCommandStep.CONTEXTS_ARG, contexts != null ? contexts.toString() : null);
            updateCommand.addArgumentValue(UpdateCommandStep.LABEL_FILTER_ARG, labelExpression != null ? labelExpression.getOriginalString() : null);
            updateCommand.addArgumentValue(ChangeExecListenerCommandStep.CHANGE_EXEC_LISTENER_ARG, changeExecListener);
            updateCommand.addArgumentValue(DatabaseChangelogCommandStep.CHANGELOG_PARAMETERS, changeLogParameters);
            updateCommand.execute();

The documentation is in a new documentation site that still being developed, but it may be helpful: https://contribute.liquibase.com/code/api/command-commandscope/

than you


Fix region tags for bigtable_hello_world.

I checked this against the internal tracker, and fixed the region tags.

This change is

PTAL.


birdCLEF_sort_data.py creates ? in folder names

After executing birdCLEF_sort_data.py the train and val subfolders for birds contain question marks (?). Which are not existing in the labels file.

Hello, I'm going to do this work too. If you use Windows system , yes, it cannot work.It's just about system's problem.As for me I replace d'?' to '!' and it worked

@DreamFlasher did that resolve the problem? That kind of error is new to me, but I never gave it a run on Windows.

I tried to put some info on the file into the file path: Genus, species, classid, recording quality and the source filename itself. Where exactly do the question marks occur? Maybe they are originating from some of the XML data. If that's the case, you could try to change the naming scheme. Do the question marks also occur in the filepaths.txt?

Okay the problem seems to be that the data from here (https://www.crowdai.org/clef_tasks/5/task_dataset_files?challenge_id=24) is not compatible with the 2017 data. Yes, they added more birds, which shouldn't be a problem, but the questionmarks are in the xmls, but furthermore there are no training samples for all the 1500 birds in the 2017 data set. Is there any way to obtain the 2017 dataset?

The 2017 data consists of two archives. One containing 999 species and the other one has 501 species in it. If you download both (they are both listed in the dataset section of the challenge) you should be able to re-create the dataset from 2017.
If the questionmarks occur in the xml files, you could try not to use the sub-species as part of the filename. If you use the class id only, you should be fine, the rest of the filename is just for better readability. You can change the sort data script to include only the class id in the filename.

Thank you. Yeah I figured that out today too, that there is a second part of the dataset. But it looks like the archive is missing 4 wav files. But now it looks a lot better. Also I changed the script to only use the classids. But I only get 1330 unique id's after the sorting. So something is still off.

You're right, the archive misses four files, but that shouldn't be a problem. I'm not sure why you only get 1330 species - using the class ids should result in exactly 1500 species. I tried that with the current download and it still works the same way as it did last year. Did you find a reason for this? Maybe some kind of duplicates?


Mac Homebrew formula for bx

https://github.com/fish2000/homebrew-praxa/blob/master/bx.rb
… notes and/or PRs are always welcome. Cheers!

Cool. I'll mention this in bgfx documentation here: https://bkaradzic.github.io/bgfx/build.html#alternative-build-systems
Do you have formula for bgfx?

Also nothing depends on lua -> depends_on "lua" => :build, it's all embedded in GENie.

Oh wow ok I will remove the Lua dependency – and also I’ll assuredly cook up a bgfx formula in short order, yes!


[PAY-1765] Refresh Pledge Screen UI With Payment Card From Payment Sheet

📲 What and 🤔 Why
We need to show the user the new payment option created from the Stripe Payment Sheet.
🛠 How
..coming..
👀 See
Before 🐛
..tbd..
After 🦋
..tbd..
✅ Acceptance criteria

[ ] Ensure once user has created a new payment option it appears within the pledge screen along with other cards (no support for pledging yet)

⏰ TODO

[x] Hook up signal from paymentOption to reloadPaymentMethods in PledgePaymentMethodsViewModel
[x] Expand logic for multiple cards created via the payment sheet
[ ] Handle selection of new card vs exisiting cards.
[ ] Unit/snapshot tests.

          1 Warning

:warning:
Big PR

  Generated by :no_entry_sign: Danger


Change the tutorials link

Why
Changing the link from nbviewer to IQX
Since every Qiskit user has to create an IQX account to access IBM Q Systems, I don't think there's any user who doesn't have an IQX account.
Problems that users might face
For people who don't have an IQX account, they must create an IQX account to view the tutorials. Maybe later, we can work on making notebooks viewable in public
@derivation Thanks for the idea!

I agree let’s change it.


chore: update tabler_icons_tsx version

Update tabler_icons_tsx to latest as 0.0.3.
The purpose of this PR is to unify the version of tabler_icons_tsx for those who use components.

Maybe, the test failure is the cause of GitHub Actions down.


alias/rename/document this also works with brand name Linak controllers

I recently got an iMovr Lander desk, which uses the official Linak controller that is rebranded for Idåsen desks.  I am able to successfully connect and control the desk (once I realized that I needed UUID instead of MAC ... that callout in the readme isn't visible enough).
Yes, I will probably just create myself a symlink 😉.  Mostly I just wanted to suggest mentioning that this works with other Linak controllers.

I would like to do this but as I only have an Idasen desk I can't test on other devices and I don't want to make promises I can't keep. If others wanted to commit to working on ensuring support for other devices that would be appreciated and maybe a rename would be possible then.
Maybe for now I will just add a list of devices it reportedly works with.

See https://github.com/rhyst/idasen-controller/issues/44#issuecomment-1062687180

Note added about potential compatibility with other devices in the README.


Add Media Key Support

Media keys are different from keyboard to keyboard, so we likely will need to add the ability to read in a keymap provided by the user. I suggest we use the same method used by qmk (pie in the sky version).

This was resolved by #22


Searching by Commit Hash may return wrong result

Describe the bug
Search by commit hash functionality in the Commit Search view may return wrong results.
To Reproduce
Steps to reproduce the behavior:

Clone the test repo from BitBucket.

git clone https://bitbucket.org/billsedison/gitlens-test.git

Open the test repo folder in VSCode with Open Folder
Go to the Commit Search view
In the enter commit hash field, type 28be4683 and click Search
It will return the wrong commit. Find the information for commit 28be4683 in screenshots

Expected behavior
Searching by commit hash should return the exact match of given hash
Screenshots

Additional context
Provided in the PR (#114)  description.

Good catch, thanks.


[AST] Fix the source range for InterpolatedStringLiteralExpr.

What's in this pull request?
This allows fixits to be correctly applied when wrapping such an expression.
Resolved bug number: (SR-)
rdar://26830715

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

A smoke test on macOS does the following:

Builds the compiler incrementally.
Builds the standard library only for macOS. Simulator standard libraries and
device standard libraries are not built.
lldb is not built.
The test and validation-test targets are run only for macOS. The optimized
version of these tests are not run.

A smoke test on Linux does the following:

Builds the compiler incrementally.
Builds the standard library incrementally.
lldb is built incrementally.
The swift test and validation-test targets are run. The optimized version of these
tests are not run.
lldb is tested.

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

This allows fixits to be correctly applied when wrapping such an expression.

@swift-ci smoke test OS X platform

@swift-ci smoke test OS X platform

'ClangModules/SceneKit_test.swift' failed in 'Swift Incremental RA - OS X' bot as well, merging.

@swift-ci smoke test OS X platform


DBZ-7085 Add MariaDB driver and use with MariaDB profile

https://issues.redhat.com/browse/DBZ-7085

Speaking of co-existence @jpechane, see my recent commit about "fetchSize".

Per https://jira.mariadb.org/browse/CONJ-977, the MariaDB team does not
believe that the Integer.MIN_VALUE solution for triggering streaming is
spec compliant and as of MariaDB 3.x+ drivers, they explicitly will not
allow this.
This change explicitly sets the fetchSize to 1 to mimic the same style
of behavior that the default fetchSize uses when using MySQL. If a user
provides a custom fetchSize, that will take precedences.

FYI, the link to download Mariadb connector plug-in is broken in v2.5 release page.

FYI, the link to download Mariadb connector plug-in is broken in v2.5 release page.

We'll have to adjust the logic in the UI bits to avoid adding that entry as really technically the MariaDB link should be the MySQL connector but because of how we listed the databases in the test matrix, that happened.

@jpechane so assuming that the tests pass now (I confirmed that the failures in the previous run at least passed locally now with my recent changes), this should be ready for a review.  I added a note to the strategy Jira DBZ-7083 about the protocol changes in the connection configuration class so we can find a better solution in the later refactoring we intend to do.

So it looks like MySQL's handling of TIMETSTAMP default values differs from MariaDB.  I'm looking into this, but I'm unsure whether there is a viable way to handle this uniformly across both since removing timezone=auto fixes this particular test failure but causes other tests to fail on the actual column values themselves due to timezone handling behavior.

After an in-depth discussion with some folks with MariaDB, we've concluded that both drivers are working correctly but reacting to the default value information differently due to how the two drivers manage time zones.
For MySQL, the connectionTimeZone actually tells the driver to do value manipulation at the driver level, which means that when the connector executes SHOW CREATE TABLE, the data returned, in this case, is simply a string. The time function on the server is unaware of the connectionTimeZone setting.  This is why MySQL, if a timestamp has a default value of 1970-01-01 00:00:01, gets returned unaffected.
MariaDB takes a completely different approach. In their case, their timezone=auto setting changes the physical timezone in the session at the server level, meaning that not only does the server return the values to us with using the same timezone like MySQL's connectionTimeZone when set to SERVER, but it also affects any rendering of timestamps via server-side calls, like the one that generates the output for SHOW CREATE TABLE.
So it seems logical that we need to adjust the MySqlDefaultValueIT test for MariaDB to account for this, but I'd like your thoughts @jpechane if you maybe have some idea I haven't explored.

@Naros I don't think there is other solution. Let's just modify the test to account for it.

Test failures are unrelated.

@Naros Aplied, thanks a lot


Could not load file or assembly 'Microsoft.Azure.Cosmos.Direct, Version=<IP_ADDRESS>

Describe the bug
I am using a project reference to Microsoft.Azure.Cosmos due to code changes related to my pull request. Everything works fine locally but when I publish my unit tests to the azure build pipeline i'm getting Could not load file or assembly 'Microsoft.Azure.Cosmos.Direct, Version=<IP_ADDRESS>
I noticed some funky project includes in the SDK project. I was wondering what I need to do to get the assemblies pushed out for my unit tets.
Expected behavior
No Exception
Actual behavior
Cannot load assembly
Environment summary
SDK Version: 3.0.100
Cosmos: Source
OS Version Windows 10

Do you mean the unit tests in this repo or a unit test project that you created that targets the SDK?

A unit test I created...

@fuocor since its a project reference you have to add a reference to nuget Microsoft.Azure.Cosmos.Direct.
@ausfeldt please correct me if I'm wrong.

@j82w correct

great... I had it in the wrong project


[v1.14] Investigate missed tail calls

x-ref https://github.com/cilium/cilium/issues/35594

/test-backport-1.14


error linking demo sydney 10_4_2

Error:
update.bat
I did not install development for ios
to me have to manually delete everything related to iOS on BAT as well with sydney_10_4_2.patch

zamenil v ustanovlennoy del'fi tri fayla na vashi
48 / 5000
Результаты перевода
replaced three files in the installed delphi with yours: ALFMXTypes3D.pas; FMX.Canvas.GPU.Helpers.pas; FMX.Context.GLES.pas
http://my.encore.su:8080/download/test.zip
I can't play the video in any way.

Hello, everything has been fixed in the latest committed version.


release version 0.0.3 to pypi

Hey I'd like to use version 0.0.3 and the README on PyPi seems to reflect that I can but when I do:
pip install django-elasticache==0.0.3 it fails. However, installing version 0.0.2 works, can you publish the newest one to PyPi?

Oops, sorry. uploaded.
On Mon, Mar 16, 2015 at 9:49 PM, Josh Schneier<EMAIL_ADDRESS>wrote:

Hey I'd like to use version 0.0.3 and the README on PyPi seems to reflect
that I can but when I do:
pip install django-elasticache==0.0.3 it fails. However, installing
version 0.0.2 works, can you publish the newest one to PyPi?
—
Reply to this email directly or view it on GitHub
https://github.com/gusdan/django-elasticache/issues/7.

Working! Thanks for the quick response!


update template url since it has been merged

update template url since it has been merged by Azure quick start template

@cloudmelon : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @jpe316 Follow up our email threads last time, my PR for ARM templates has been merged by Azure quick start template, so I update this doc as well. Pls have a look, thanks :)

Discussed these changes with @cloudmelon offline. Approved, #sign-off

@PeterCLu: I'm sorry - only the author of this article, @jpe316, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


OGG Standard Business Reporting LiveLab

Add here your WMS ID: 11755
General requirements

Information in the workshop is adequate and updated yes
Code is correct and working yes
Links are correct yes
Make sure you enter the help email link in your manifest.json yes
Please make sure WMS URLs are updated as needed after your PR is approved yes

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[ ] All filenames are lower case (including folders, images, files, etc.): yes
[ ] Filenames are descriptive: yes
[ ] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop) yes
[ ] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html yes
[ ] Image references in markdown contain an alternate text yes

I have resolved the conflict.

Hi @preeshuk, you have some conflicts that need to be solved


CBI Multi value dependent values not visible

The current refactoring of the cbi has broken the MultiValue dependency functionality. When the dependent value is enabled the dependents are not visible. Upon refreshing the page the dependents become visible but when cycled (disable/enable) they are no longer visible. I have found a solution but I do not know  if it has any reverse effects ? However, perhaps it will help track down the issue.
/usr/lib/lua/luci/view/cbi/mvalue.htm
<%
    local i, key
    local v = self:valuelist(section) or {}
-%>

<%+cbi/valueheader%>
<% if self.widget == "select" then %>
    <select class="cbi-input-select" multiple="multiple" data-update="click change"<%=
        attr("id", cbid) ..
        attr("name", cbid) ..
        ifattr(self.size, "size")
    %>>
        <% for i, key in pairs(self.keylist) do -%>
            <option<%=
                attr("id", cbid.."-"..key) ..
                attr("value", key) ..
                attr("data-index", i) ..
                attr("data-depends", self:deplist2json(section, self.deplist[i])) ..
                ifattr(luci.util.contains(v, key), "selected", "selected")
            %>><%=pcdata(self.vallist[i])%></option>
        <%- end %>
    </select>
<% elseif self.widget == "checkbox" then %>

        <% for i, key in pairs(self.keylist) do %>
            <label<%=
                attr("for", cbid.."-"..key) ..
                attr("data-index", i) ..
+              attr("data-depends", self:deplist2json(section, self.deplist[i]))..
+              ifattr(luci.util.contains(v, key), "checked", "checked")

            %>>
                <input class="cbi-input-checkbox" type="checkbox" data-update="click change"<%=
                    attr("name", cbid) ..
                    attr("value", key) ..
                    ifattr(luci.util.contains(v, key), "checked", "checked")
                %> />
                <%=pcdata(self.vallist[i])%>

            <% if i == self.size then write('
') end %>
        <% end %>

<% end %>
<%+cbi/valuefooter%>

this fixes the issue of the dependents not being visible but I have also noticed on the  https://<IP_ADDRESS>/cgi-bin/luci/admin/network/iface_add page the "Create a bridge over multiple interfaces" has no effect and both the Flag and MultiValues are always visible. The above does not effect nor correct this issue so it may or may not be  an unrelated issue ?

Hmm, I cannot really reproduce the problem, also mvalue.htm in current HEAD already has the data-depends line you're proposing, see https://github.com/openwrt/luci/blob/master/modules/luci-base/luasrc/view/cbi/mvalue.htm#L29
The iface_add issue has been fixed with e78759ad5aa2418e7a39e1061d430c2e5e8fe0b8 and 55597da3a8540d19b34bf25ccf08d70e759013ab .
Can you retest with a current HEAD copy of mvalue.htm ?

I reverted my changes to the mvalue.htm and retested with your latest commits e78759a and 55597da and everything works as it should.


Remove waybar overlay or make it optional

Hyprland Version
a0cf89029263decba29b0f4216b38576f4418335
Bug or Regression?
Bug
Description
nixpkgs has waybar-hyprland now, I think waybar overlay can be removed or be optional.
@fufexan
How to reproduce
waybar-hyprland
Crash reports, logs, images, videos
No response

Sure, I need the opinion of waybar users first. @NotAShelf ?

We can remove it. Though, what do you mean by "make it optional"?


Bugs in helper_files/mongo_setup.py

diff --git a/helper_files/mongo_setup.py b/helper_files/mongo_setup.py
index 90b89ad..4b56402 100644
--- a/helper_files/mongo_setup.py
+++ b/helper_files/mongo_setup.py
@@ -1,7 +1,7 @@
 from config import API_KEY, API_SECRET, POLYGON_API_KEY, RANK_POLYGON_API_KEY, MONGO_DB_USER, MONGO_DB_PASS, mongo_url
 from client_helper import strategies
 from pymongo import MongoClient
-import datetime
+from datetime import datetime
 import math

 def insert_rank_to_coefficient(i):
@@ -79,7 +79,7 @@ def initialize_time_delta():
 def initialize_market_setup():
    client = MongoClient(mongo_url)
    db = client.market_data
-   collection = client.market_status
+   collection = db.market_status
    collection.insert_one({"market_status": "closed"})
    client.close()

Closed it as it was fixed already in the latest version.


adding str where int is enforced - but i can commit.

Hello,
please take a look at https://gist.github.com/1643718
Please execute this script and watch how it adds an string to user_id where a int is needed and then successfully commit the object to sqlite3.
I use flask-sqlalchemy 0.15, flask 0.6 (debian squeezy flask-0.6-1), libsqlite3-0   3.7.3 (debian squeeze libsqlite3-0  3.7.3-1)
cheers,
robin

Not a flask-sqlalchemy issue. File this against SQLAlchemy instead.

besides if it is a issue in flask or not, here is a solution comming up from mitsuhiko:
https://gist.github.com/1676017


Allow custom method or scope for includes

I think this suggestion is distinct from #47, but it's similar and could possibly be developed at the same time, if others agree and would find this valuable. Alternatively, if there is already a solution to this that would be even better 😄
Scenario is as follows. This is for boxing/UFC fights where there are two athletes (fighters) in each bout, and many bouts in an event.
Serializers:
class FighterSerializer
  include FastJsonapi::ObjectSerializer
  attributes :first_name, :last_name
  has_many :fighter_bouts
end

class FighterBoutSerializer
  include FastJsonapi::ObjectSerializer
  attributes :date_of_event, :name_of_event, :result
end

Models:
class Fighter < ActiveRecord::Base
  has_many  :fighter_bouts

  def custom_method_with_scoping_and_eager_loading
    fighter_bouts.includes(<nested relationships>).where(<scoping of records>).order(<detailed ordering>)
  end
end

class FighterBout < ActiveRecord::Base
  belongs_to :fighter
  belongs_to :bout
end

class Bout < ActiveRecord::Base
  has_many    :fighter_bouts
  belongs_to  :event
  belongs_to  :weight_class
end

What I would like to be able to do is have an endpoint for a Fighter which is a compound document with included fighter_bouts.  Such as:
fighter = Fighter.find(1)
opts = {include: [:fighter_bouts]}
payload = FighterSerializer.new(fighter, opts).serialized_json
render json: payload

However, I want to use the custom method defined on the Fighter class, because it will eager load everything needed from a series of nested associations.  Data from these nested associations will be "flattened" and presented as attributes on the included fighter_bouts. For example, the event name will be returned as an attribute of a fighter_bout, even though in actuality this value is 2 associations away in the database.
Therefore I was thinking it would be very useful to be able to either pass an additional method name or value along with the options hash being given to the serializer.  Or, allow for a custom-named "association method" in the serializer.  Extremely roughly conceptualized as:
opts = {include: [:fighter_bouts_custom]}
payload = FighterSerializer.new(fighter, opts).serialized_json

class FighterSerializer
  include FastJsonapi::ObjectSerializer
  attributes :first_name, :last_name
  has_many :fighter_bouts

  def fighter_bouts_custom
    <reference to custom method on the Fighter class>
  end
end

Thanks for reading!

@gsaks123 we have the ability to customize the object_method_name and id_method_name for a has_many relationship. Here is the documentation https://github.com/Netflix/fast_jsonapi#customizable-options. Have you considered you these options?

@shishirmk Wow, I completely missed this. Yes, this is exactly the functionality I need.  Not only does the object_method_name scope the "relationships" payload, but it also applies (critically) to the  "included" payload. Perfect!
Thanks for your work on this gem.

Sure.  In this scenario we have an object class "Fighter", and when we call our API for a Fighter record, we want it to eager load and include a deeply nested association of other objects.  And that association is heavily scoped to only include certain results, with custom ordering.
Here is the custom method on our app/models/fighter.rb class which defines the scoping and the association. We want this logic to be applied to what is returned by our API:
  def ordered_fighter_bouts_api
    relation = fighter_bouts.includes( bout: [ {event: :promotion}, {fighter_bouts: :fighter}, :weight_class ] )
    relation = relation.where(status: Bout::STATUS_VISIBLE, sport: 'mma')
    relation.order("events.starts_at DESC", "bouts.bout_number_at_event DESC NULLS LAST", "bouts.id DESC")
  end

Then, in our serializer for Fighter records, app/serializers/v1/fighter_serializer.rb, we utilize the convenient 'object_method_name' that @shishirmk alerted me to:
class V1::FighterSerializer
  include FastJsonapi::ObjectSerializer
  has_many :fighter_bouts, object_method_name: :ordered_fighter_bouts_api
  attributes :first_name, :last_name, :middle_name, :nickname, :nationality
end

Lastly, in our controller for this API endpoint (app/controllers/api/v1/fighters_controller.rb), we include the Fighter's association using its basic/underlying class name. And it understands to use the object_method_name that was given in the serializer:
class Api::V1::FightersController < Api::BaseController
  def show
    fighter = Fighter.api_find_by_id(params[:id])
    head(:not_found) and return unless fighter
    payload = ::V1::FighterSerializer.new(fighter, {include: [:fighter_bouts]}).serialized_json
    render json: payload
  end
end

Does this help you and answer the questions you had?

Yeap that explanation makes sense. I see how it works now. Thank you for that
However, I think this doesnt help my case. What I need to do, is call object_method_name with an arg. i.e. ideally passing params to it, so I can filter against it

Actually, I figured this out. has_many supports blocks. ie.
  has_many :comments do |object, params|
    object.comments.where(status: params[:filter][:status])
  end

Hey guys, struggling with this. I feel like I'm following this advice, the object_method_name sounds like exactly what I want but I'm not getting the details I expect.
# lead_serializer.rb
class LeadSerializer
  include FastJsonapi::ObjectSerializer

  attributes :requested_amount, :loan_purpose ...

  has_many :products, object_method_name: :products_with_default
end

# lead.rb
class Lead < ApplicationRecord
...
  has_many :products, -> { unscope(:order).distinct },
           through: :matches, dependent: :nullify
...
  def products_with_default
    return products if products.present?
    Product.where("supported_loan_purposes ilike ?", "%#{loan_purpose}%")
  end
end

When I run these methods outside the serializer i.e.
$ Lead.first.products
=> []
$ Lead.first.products_with_default
=>
[
  #<Product: 0x00007fdcbe33d810> {
    id: 1234
    ...
  }
]

I get the expected outcome.
However my LeadSerializer is returning null for the products relationship
{
  :data => {
    :id            => "1234",
    :type          => :lead,
    :attributes    => {
      :requested_amount  => 10,
      :loan_purpose  => "general"
      ....
    },
    :relationships => {
      :account           => {
        :data => {
          :id   => "1234",
          :type => :account
        }
      },
      :products      => {
        :data => []
      }
    }
  }
}

Can someone point out what is going wrong with my configuration, or if there's a caveat I'm not aware of? Thank you!


dev: use with_chain_id for CfgEnv

Describe the enhancement request
Once this PR is merged in revm, add the function with_chain_id in order to set the chain id of the config.
This needs to be updated in tracing/mod.rs in the new function of the Tracer.

hey is it up for grabs? i would like to take this. : )

hey is it up for grabs? i would like to take this. : )

Assigned

If this is still available, can i take it @tcoratger ?

If this is still available, can i take it @tcoratger ?

Hey @Pycomet sorry but this issue is blocked until next release of reth when https://github.com/bluealloy/revm/pull/1327 is included. And this issue is already assigned but feel free to pick another one and if you don't find one, stay tuned for the next one we release, happy to help :)

please assign


photo_of_my_messy_setup.png evtl. shows minus Cable on wrong port

Hi Martin,
i discoverd your code  and i am very happy to give it a try. Thank you for your work and for making it public. I already ordererd all the necessary hardware and am waiting for it to arrive. In the meantime i try to find out what cables need to go where based on the infos you provided. I am a bit confused, as you wrote Segway battery J4 should be connected to Arduino GND. But i am almost shure that on the photo its connected to the 5V connector of the Arduino. Can you please clearify. Thanks.

It's been years since I last touched this code or looked at the diagrams.  No matter what, you should be VERY CAREFUL about your setup.  The live POS(+) terminal has 74V and will happily fry anything you connect to it as solidly as any welder can -- while at the same time popping a fuse inside the battery that simply cannot be reached without completely disassembling the battery -- consider yourself warned
I think the photo I took was using one of my dead Segway batteries.  It's entirely possible I had it plugged into the wrong pin on that arduino ( which shouldn't do much to an arduino -- 5V arduino to ground on the battery will do very little -- not a lot of current to be had there )
But you're going to want to make sure it's connected to a GND pin.  You're going to need a common ground to make sure your i2c signals come in clear and strongly with a common ground for the floor of the signal.

Thanky you very much. You are absolutely right by saying "you should be VERY CAREFUL about your setup", thats exactly why i asked. So i am going with ground. I am very exited with your code., it will probably help me a lot. Thank you !

His code works great ..  it has been 4 yrs since I used it ..  Sent from my iPhoneOn Jun 24, 2024, at 6:53 PM, secman248 @.***> wrote:
Thanky you very much. You are absolutely right by saying "you should be VERY CAREFUL about your setup", thats exactly why i asked. So i am going with ground. I am very exited with your code., it will probably help me a lot. Thank you !
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thanks Nazar...
I wish I had a reason to continue developing it.  What I put together
though is at least a good base for other people who want to explore the
battery BMS and what the communication protocol is.
There are some parts I never finished decoding -- charge/discharge rates,
serial number, production date.. that kind of thing.  It's in there, and
another person ( who made a closed source solution ) named Felix managed to
figure it out.  It's just a question of some patient analysis, time, and
hacking.
On Mon, Jun 24, 2024 at 6:06 PM nazar404 @.***> wrote:

His code works great ..  it has been 4 yrs since I used it ..  Sent from
my iPhoneOn Jun 24, 2024, at 6:53 PM, secman248 @.***> wrote:
Thanky you very much. You are absolutely right by saying "you should be
VERY CAREFUL about your setup", thats exactly why i asked. So i am going
with ground. I am very exited with your code., it will probably help me a
lot. Thank you !
—Reply to this email directly, view it on GitHub, or unsubscribe.You are
receiving this because you are subscribed to this thread.Message ID:
@.***>
—
Reply to this email directly, view it on GitHub
https://github.com/martinbogo/pt-battery-diagnostics/issues/15#issuecomment-2187577189,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB5KKJIUD7ZX2436EEKK3TZJCRAXAVCNFSM6AAAAABJ2STGAKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBXGU3TOMJYHE
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Side note!  I don't think I implemented this fully:
register 29, contains soc means "state of charge"
register 212 contains amp means charge/discharge current
soc=(int)((k_avg_soc*raw_avg_soc[2])/254);
if(summ_volt<76.5 && raw_avg_soc[2]==254) soc=0; //Case of Under Level SOC
amp=(256*(raw_avg_amp[1])+raw_avg_amp[2])/k_cell_amp;
float k_avg_soc=100;
float k_cell_amp=128;

cool. i try my best to understand and implement this stuff.

I have Felix product as well ,  it worked well as well !!  But I no time to update Segway batteries anymore ,  as I have around 40 plus core to update with new cells ..  maybe one day Sent from my iPhoneOn Jun 24, 2024, at 7:20 PM, secman248 @.***> wrote:
cool. i try my best to understand and implement this stuff.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>


Marvins fly when grappled

When you grapple marvins they forget what gravity is and start floating, they also have text over them that I don't remember existing in the vanilla version
https://user-images.githubusercontent.com/16687318/148127432-da9b701e-cba0-4796-a506-deca4d5d84d9.mp4

I don't think this is a bug, it's a goddamn feature

In the vanilla game mrvns just die if you grapple them iirc, shouldn't be hard to implement if we want to
tbh I kinda like it though

while this is great it should probably be fixed, some people in the discord figured out this is likely an issue with ai having their team set to 0, the vanilla game likely uses custom spawning logic to set the teams and stuff up for marvins here so it'd probably be easiest for us to fix this through replicating that

i've tested this, it seems marvins still have this issue regardless of their team

Vanilla Marvins don't get affected by the grapple so it is likely an issue of Marvins being pulled by the grapple when they aren't supposed to.
https://user-images.githubusercontent.com/97146561/150320861-2d9e62a7-c917-4980-92cb-4639e9f2d40a.mp4

entity.kv.contents = (int(entity.kv.contents) | CONTENTS_NOGRAPPLE)
i wonder if this would fix it

^ yup that did fixed it will open a pr


Setup lifecycle for push builds in build.lbry.io/app

Every push build is kept in http://build.lbry.io/app/. Old builds aren't being used anymore so no need to keep them. The exception is http://build.lbry.io/app/release/. This folder should not be deleted. It's being used for app updates.

done!


Add Brand name to ProductDetail in SimplCommerce.Module.Catalog module

Add and mange BrandDetail (Name and Slug) to ProductDetail in SimplCommerce.Module.Catalog module

Name and Slug

I don't get your point. Currently you can select brand for products

Allow to show brand name in product detail view


feat: print last output when doWaitFor times out

Fixes #116

not sure why the test is failing

fluke probably, will investigate, merged for now.
thanks!


Update scalatags to 0.11.0

Updates com.lihaoyi:scalatags from 0.9.4 to 0.11.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

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

labels: library-update, early-semver-major, semver-spec-minor

Superseded by #907.


32-bit linux binaries where

?

There are no official 32-bit unix packages.

Why no?

@SteveL-MSFT Could you please answer?

@ek4 we haven't had any requests for x86 built binaries until now.  Unless there is a number of customers asking for 32-bit support, it's a burden to regularly build and test.  For now, we could update our build script to output 32-bit binaries if the packaging/validation can be owned by the community.

@rkeithhill excellent point!  Looks like https://github.com/dotnet/coreclr/issues/9265 tracks .NET Core support for x86 Linux.  Until then, nothing we can do.

@SteveL-MSFT There is actually one prior request here, plus a SE question, so there's at least some interest.


Add colormaps for the different channels

Inspect nd2 metadata to see whether we can introspect colormaps - potentially look for nm emission/nm wavelength

Closed by #9


Export constants on sync fs API

Description of changes

Also exporting constants on the sync fs API

Checklist

[x] Created unit tests in tests/unit and/or in Rust for my feature if needed
[x] Ran make fix to format JS and apply Clippy auto fixes
[x] Made sure my code didn't add any additional warnings: make check
[x] Updated documentation if needed (API.md/README.md/Other)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks 👍


Consider other Formats in com_google_fonts_check_STAT_gf_axisregistry_names

Description
My fonts have opsz AxisValue Format 2, so this line https://github.com/googlefonts/fontbakery/blob/master/Lib/fontbakery/profiles/googlefonts.py#L4843 results in a ERROR because Format 2 doesn't have the .Value attribute, instead it has .NominalValue.
The check currently assumes Format 1 or 3.
To Do

[x] update CHANGELOG.md (new check is unpublished, so not changing things here)
[ ] wait for checks to pass (except github/pages, which is stuck)
[ ] request a review

thanks!


Filezilla on Openshift

I'm trying to run jlesage/filezilla image on Openshift, but I'm getting this message:
s6-mkdir: warning: unable to mkdir /var/run/s6: Permission denied
Image is built from dockerhub image, without particular customization.
Do you how to solve this issue?
Thank's

Closing this issue.  Please re-open if needed.


Modified rule to perform impact analysis on zUnit Tests

According to the documentation the impact resolver's initial query returns resources (i.e. test configuration files) that potentially are impacted by changes to the program being test (i.e. the test configuration file). By applying the resolution rules it can be determined that a test is impacted by the specific changed program.
Therefore the logical dependency of the zUnit test of type ZUNITINC and library SYSPROG should be resolved to the cobol source and not to the configuration file.

I am closing the PR, we might need to reopen if necessary.


Correct and streamline documentation

Correct and streamline readme. In particular, new docs on WIDE_INTEGER_NAMESPACE are slightly incorrect regarding the default state of the preprocessor option which is now not defined.

List of things to improve/correct:

Docs on WIDE_INTEGER_NAMESPACE (see above).
Section on Conversion rules

Not sure if this is the right place but in terms of API design, I'd highly recommend doing as the ints do.

I try. It's a fine place for a small version of this discussion.
This is really a tough one for me, cause sometimes I struggle with the syntax and enable_if-ing this-or-that and it gets super confusing.

If I made a wide integer out of two shorts, I would expect it to behave exactly like an int. Make sense?

Anyway...,
Yes. Wide integer is close, but not exactly there. ultimately a UDT can, in my opinion, never behave truly like a built-in. If I go for some implicit conversions with int, float, of short, long long, I might end up getting some warnings. Fine. I get warned.
If, however, i'm writing a C++ UDT, I seem to get two (maybe three) choices, either I can silently allow implicit conversion, or I can demand explicit conversion, or i can delete functions and get an error. What I can't get is a warning that sails through the compiler.
The long form of this discussioon belongs in the SG for numerics (is that SG6?) and beyond. Things like wide-integer can (or could) be drafts or even get close to reference implementatins. But ultimately we need specification(s) --- needed for big fixed, float, int, resulting in a whole bunch of missing specs.

Yes I'd stop short of implementing all promotions. E g. a two-char wide
integer should not promote to int width.
On Thu 20 Jan 2022, 17:27 Christopher Kormanyos, @.***>
wrote:

Not sure if this is the right place but in terms of API design, I'd highly
recommend doing as the ints do.
I try. It's a fine place for a small version of this discussion.
This is really a tough one for me, cause sometimes I struggle with the
syntax and enable_if-ing this-or-that and it gets super confusing.
If I made a wide integer out of two shorts, I would expect it to behave
exactly like an int. Make sense?
Anyway...,
Yes. Wide integer is close, but not exactly there. ultimately a UDT can,
in my opinion, never behave truly like a built-in. If I go for some
implicit conversions with int, float, of short, long long, I might end up
getting some warnings. Fine. I get warned.
If, however, i'm writing a C++ UDT, I seem to get two (maybe three)
choices, either I can silently allow implicit conversion, or I can demand
explicit conversion, or i can delete functions and get an error. What I
can't get is a warning that sails through the compiler.
The long form of this discussioon belongs in the SG for numerics (is that
SG6?) and beyond. Things like wide-integer can (or could) be drafts or even
get close to reference implementatins. But ultimately we need
specification(s) --- needed for big fixed, float, int, resulting in a whole
bunch of missing specs.
—
Reply to this email directly, view it on GitHub
https://github.com/ckormanyos/wide-integer/issues/169#issuecomment-1017744290,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFTGN57ECCOUPKLXRADKOLUXBAWJANCNFSM5LTCUKDA
.
You are receiving this because you commented.Message ID:
@.***>

Yes I'd stop short of implementing all promotions. E g. a two-char wide integer should not promote to int width.

Oh geez. I guess. Well I think it's a good starting point. With successive rounds we can probably get around to all the necessary syntax and enable_if-ing.
Anyway, I'm pretty happy with the state, being used off the rack in cnl and many other projects. So it's a good start...
Thanks as always John for sage advice!


pull request for e2e test at 01/05/2017 04:45:08

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Text_input double-click selection issue

When I double or triple click on text in text_input it first selects a whole word (or whole line for triple click) then reselects from the beginning up to the current cursor position. I can see the correct selection quickly flash for a frame or so.
The bug is present in all input boxes I've tried, like examples/todos.
Tested with both mouse and trackpad.
Platform: macos / wgpu
Branch: master
Video of the issue:
TextInput Bug.mov.zip

Thanks for the report!
I have just opened #620, which should fix the issue.


[Issue 3275] [fix] Extension - Fix bug connecting to uquid dapp with Wallet Connect

🚀 Deployed on https://pr-3349--sw-web-runner.netlify.app


Sitecore 9.2 assembly version list don't work with excludes : an extra header was introduced

with Sitecore 9.2 (and maybe 9.1, didn't check), the lists couldn't be used anymore. With 4 fields of information (Filename,Version,FileVersion,InfoVersion) instead of 3 (Filename,Version,FileVersion,InfoVersion) the list couldn't be parsed anymore. This happened due to an explicit check on the amount of fields.

Thanks for the heads up. I'll try to fix it in the next few days but feel free to PR if you need it in a hurry - just add a specific unit test for it in the tasks project.

Ah PR is being prepared. My previous one contained 100+ Commits for some reason

PR #71 solves this issue. Succesfully tested

Released as 1.5.6

Looks to be an issue in 10.1 - manually changing the header to this fixes it:
Filename,FileVersion,Version,InfoVersion


React-hot-loader compitability

Here stated that it is possible to use react-hot-loader with this project.
However, I see that react-hot-loader changed major version. Is it still compatible with systemjs-hot-reloader? I'd like to test it out, but because of complete lack of documentation how to add it, I can't figure it out (yeap, I'm too dump for those things. Yet).
Can someone elaborate on this? Thanks!

Looks like v3 is finally out. Haven't had time to write docs yet. But basically, its just a babel transform.
Follow the steps here: https://github.com/gaearon/react-hot-loader/tree/master/docs
In summary, its something like this:

Install react-hot-loader@next
Add react-hot-loader/babel to your babel plugins
import react-hot-loader/patch in your app BEFORE your app code runs. You could also import it before you import your app if you want to.
wrap your top level component in AppComponent imported via import { AppContainer } from 'react-hot-loader'.

Last I checked that worked. I did run into an issue however where it only worked if your top level component used JSX. You can use normal react functions in the rest of your app. But otherwise this should give you persistent react state :)
Let me know how it works for you

Thanks for the instructions!
Btw, import { AppContainer } from 'react-hot-loader' no longer works in SystemJS 0.20.0.
So we need to do this instead:
import rhl from 'react-hot-loader'

const { AppContainer } = rhl

However, while it doesn't throw any errors, I tried to make counter ticking on index page, and it gets resetted after hot reloading. So, it doesn't work.
Instructions and official example shows, that we also need to add something like
if (module.hot) module.hot.accept('./App', () => render(App));

I believe module is hydrate somewhere from webpack, thus it doesn't work in JSPM.
Those changes were introduced in v3, so I think it will no longer work that easily for JSPM.

Hello,
I have this all working with JSPM.
What they are doing is just ensuring that the HMR only works under development scenarios.  There is only filtering logic there, use x for dev and y for prod.  We can easily use JSPM to map our requirements.  You just have to add this to your jspm.config.js - since it will work well for both dev/prod.
,
```
map: {
"react-hot-loader-patch": {
"~production"<EMAIL_ADDRESS>"production"<EMAIL_ADDRESS>},
"react-hot-loader-container": {
"~production"<EMAIL_ADDRESS>"production"<EMAIL_ADDRESS>}
}

Then you just import the two like so..

import 'react-hot-loader-patch'
import AppContainer from 'react-hot-loader-container'

The only problem here is the fact we have to use the `@next` qualifier here - so once they are general release this will have to change.  If I missed something so that we don't have to use the qualifier, then please let us know.

Apologies that I haven't been present. Have major deadlines for Friday. Will be pushing forwards next week.
@ArmorDarks you don't need that. The systemjs mechanics are such that this is baked in. Everything should still work ATM. But you may need to use JSX at the top level (due to a bug in react-hot-loader).


Webui backend pulls from our GCS bucket of processed data, instead of a local folder

As an APISnoop developer wanting a persistent source for our data,
In order to avoid manually logging into our server to retrieve data,
and to avoid the time required to manually process this retrieved data,
I want a --consume script that pulls processed data, according to our sources.yml file, into our backend's data folder.
Given a sources.yml
And a GCS prefix that holds data processed by our apisnoop utility
When I run a --consume script
Then it downloads the process data into our backend's local data folder

@zachmandeville I'm feeling we have this with ./apisnoop.sh --download-apiusage
If you agree, go ahead and close this one.

For now let's not have the backend application pull from GCS, we'll just run '--download-apiusage'

We have our intended results now with the apisnoop cli.  The backend downloads from a GCS bucket, to a local folder, then updates its api using these new files.


testnet: merge dev for relaunch version

Add precompiles and da signers module
Remove DAS
Rename denom
Fix tests

This change is

client/docs/ibc-go-swagger.yml line 132 at r1 (raw file):

                    base_denom:
                      type: string
                      description: base denomination of the relayed fungible token.

why change this?

client/docs/ibc-go-swagger.yml line 132 at r1 (raw file):
Previously, zeroxsatoshi (0xsatoshi) wrote…
why change this?

I will recover these in another PR.

migrate/utils/periodic_vesting_reset_test.go line 44 at r1 (raw file):
Previously, zeroxsatoshi (0xsatoshi) wrote…
use chaincfg.MakeCoinForGasDenom?

I wiil do this in another PR and merge to testnet/v0.1.x


smtp may be too pedantic (for localserve)

Background: I'm trying to use mox localserve for testing Odoo's processing of both in- and outgoing mails - maybe not completely the intended use-case, but thanks to the new webmail feature something conveniently possible. Mox functions as both a replacement and extension of mailpit in this case.
Error description:
I use mox localserve, with pedantic deactivated. Ports/connections itself work just fine, but I got the error message: 5.5.2 bad syntax: need at least one char for subdomain.
Looking at the log, I see:
trace: "LS: 220 localhost ESMTP mox v0.0.6\r\n" (pkg: smtpserver; cid: 18a2c656776; delta: "174.221µs")
trace: "RC: ehlo MYHOSTNAME.\r\n" (pkg: smtpserver; cid: 18a2c656776; delta: "360.184µs")
debug: "smtp command result": "bad syntax: need at least one char for subdomain (remaining \"\")" (pkg: smtpserver; kind: submission; cmd: ehlo; code: 501; ecode: 5.5.2; duration: "33.644µs"; cid: 18a2c656776; delta: "128.314µs")

I guessed that the trailing dot of the hostname (as sent by Odoo and outside my practical sphere of influence) might be the issue.
Locally patching https://github.com/mjl-/mox/blob/61a5eb61a446581f59bc338aa23b6074cd068695/smtpserver/parse.go#L245C1-L249:
func (p *parser) xdomain() dns.Domain {
        s := p.xsubdomain()
        for p.take(".") {
                if !p.empty() { // <----- HAACK
                        s += "." + p.xsubdomain()
                }
        }

worked. AFAIU the referenced RFC, domain names with trailing dot are not explicitely forbidden, but I'm not sure, and I haven't dug out all the referenced RFCs and sections.
Is this a bug (and could be fixed generally), or if not, is it possible to allow it in non-pedantic mode for localserve?

Thanks for reporting. I think we should probably be more lenient (perhaps even completely ignore) the argument to ehlo in the case of submission. Unlike with regular SMTP, the value isn't used at all with submission. Plenty of mail clients just write something there.
Will try to make a fix later today. It will probably not be in xdomain(), but in the helo/ehlo handling in smtpserver/server.go.

I hope the commit solves this: For submission without pedantic, we just ignore the parameters since we don't use them.
What odoo is sending seems invalid to me. The SMTP RFC 5321 has the following syntax:
rfc/5321:1829:
   ehlo           = "EHLO" SP ( Domain / address-literal ) CRLF

rfc/5321:2291:

   Domain         = sub-domain *("." sub-domain)

rfc/5321:2303:

   sub-domain     = Let-dig [Ldh-str]

   Let-dig        = ALPHA / DIGIT

   Ldh-str        = *( ALPHA / DIGIT / "-" ) Let-dig

So a dot at the end is not allowed. But it'll be ignored now (:.

Thanks for the very quick response, this solves my issue, and I think in a most appropriate way.
I agree, the RFC seems very clear about this. Color me surprised, though: A complete/absolute domain name is designated with a dot at the end (as per the antique RFC 1035), and I thought that it is universally valid. Seems Odoo, or some Python library, was written with an analoguous, erroneous assumption.


Sample code from README never ends?

This code (from README, using non-promises version) hangs after displaying The solution is:  2
import {createConnection, QueryError, RowDataPacket} from 'mysql2';

const connection = createConnection(process.env['DB']);

connection.query('SELECT 1 + 1 AS solution', (err: QueryError, rows: RowDataPacket[]) => {
    console.log('The solution is: ', rows[0]['solution']);
});

I had to add a
connection.end();

to the end of file to fix this. Then the program finishes normally.
Is this the expected behavior?
Using mysql2 version 1.2.0. Node.js version v6.10.3.

The example in the readme is copied directly from mysql2 and only adapted to use TypeScript, but I can provide an explanation nonetheless:
NodeJS only exits the process if there are no more events in the event loop and there are no more open streams, unless you unref them. Typically Node is used for web applications that are supposed to run until terminated. So yes, this is expected.


For 1180, add filters for minimum number of pages and minimum read percent

For "ROW or PAGE compression opportunities may exist":

It does not seem worth the time to analyze small indexes, or to compress them once analyzed.
It seems like compressing indexes with a high ratio of writes to reads is likely to cause more CPU overhead for the regular workload than it would be worth.

Adding parameters (@CompressionMinimumPageThreshold and  CompressionMinimumReadPercentage for those numbers and filtering on them seem like a good addition that would not significantly slow down the query, as the data is available in sys.partitions and sys.dm_db_index_usage_stats.
Kalen Delaney has such code (although not parameterized) in her stored procedure usp_estimate_data_compression_savings. Her procedure is much more comprehensive and takes much longer to run, but it seems easy enough to add another CTE and a WHERE clause to yours.
In one of my databases, her procedure produces 58 rows of compression recommendations (after 15 minutes of execution time), while yours produces 478 rows of XML output.

I had to take a closer look to refresh my memory. As it happens, we already consider the read and write (or scan and update to be precise) ratios when deciding whether to recommend compression and what compression type to recommend. This is parameterized:
-- 1180: The maximum ratio of updates to all operations to define "infrequent updates"
@CompressionPartitionUpdateRatioThreshold1 decimal(3,2) = 0.2,

-- 1180: The maximum ratio of updates to all operations to define "more frequent but not frequent enough updates"
@CompressionPartitionUpdateRatioThreshold2 decimal(3,2) = 0.5,

-- 1180: The minimum ratio of scans to all operations to define "frequent enough scans"
@CompressionPartitionScanRatioThreshold1 decimal(3,2) = 0.5,

Now, the default ratios are somewhat of a best guess, so open to adjusting that. If the results you are getting with default ratio are questionable, it would be interesting to see which adjustments make them more reasonable in your case.
We are using sys.dm_db_index_operational_stats() rather than sys.dm_db_index_usage_stats, in essence following the approach in Data Compression: Strategy, Capacity Planning and Best Practices.
Regarding your first suggestion, I updated the wiki for 1080 to clarify decision points around what to compress when.

@dimitri-furman You are, of course, correct. In fact, you consider not just seeks but "user_seeks + user_scans + user_lookups".
Kalen uses at least 500 pages and 75% reads as a filter when creating the set of indexes to analyze, but does not consider cases where an index is "more frequently updated but also more frequently scanned". (The logic in that CASE statement in your code could be explained a bit in the Wiki, but is not, although the code is well commented.)
I see your point about analyzing the small tables, although one can also make the case that if it is a small table that will be growing significantly, it is hard to guess at what the read/write ratio will look like later. In the early stages, I would presume that such a table would mostly be written to and so it would get skipped anyway. For tables that start small and stay small, such as a lookup table of a few dozen rows, those will likely stay as almost entirely read-only, making compression a tiny win. In the end, I agree with you that filtering on size is not particularly useful.
You are effectively using the equivalent of filtering on 80% reads (@CompressionPartitionUpdateRatioThreshold1 decimal(3,2) = 0.2) and that is a small difference. Hopefully admins will run this script periodically, as databases change over time and the recommendations may as well.
In the Wiki, you state: "For most workloads, ROW compression should be enabled by default for new tables and indexes." It is probably the correct that Microsoft hasn't made that the default because it cannot be predicted in advance whether any given index or table in a general purpose database will end up having a workload skewed more toward reads or writes.
I was recently reading https://eitanblumin.com/2022/01/17/detect-low-compression-rates-in-all-databases/ and I think you are already handling those cases. Is that true?
Adding the command needed to implement the suggestion to the Details output would be a nice enhancement. If you really wanted to go crazy, that Details output could be written as full XML so it would be easy to pop into Excel. Both of these suggestions apply to 1530 (Disabled or not trusted constraints found) and probably other sections as well.
Thanks for publishing (and maintaining) this script. This kind of analysis that creates actionable information is very helpful.

@dimitri-furman It looks good to me. A user could always do a quick find/replace on the output to change the options (ONLINE, etc.) if desired. I am not a good test case for this, as my only partitioned indexes are very write-heavy. (We use partitioning as a way to reduce blocking on the very frequent INSERTs.) But the output looks good on the databases I've run it on. I just never see anything other than "partition range: 1-1" and your code is smart enough not put a PARTITION option into the commands.

Thanks @m60freeman. I'll close this now, please open new issues if you have more feedback on the script.


[DNM] Test of sanitycheck filtering later

This one is dropping tests:
with this PR:
sanitycheck -T tests/kernel/fatal/
Cleaning output directory /home/anashif/zephyrproject/zephyr/sanity-out
JOBS: 112
Selecting default platforms per test case
Building testcase defconfigs...
Filtering test cases...
54 tests selected, 916 tests discarded due to filters
total complete:   16/  54  29%  failed:    0
54 of 54 tests passed with 0 warnings in 10 seconds
In total 5 test cases were executed on 16 out of total 195 platforms (8.21%)

with sanitycheck in master:
sanitycheck -T tests/kernel/fatal/
JOBS: 112
Cleaning output directory /home/anashif/zephyrproject/zephyr/sanity-out
Selecting default platforms per test case
Building testcase defconfigs...
Filtering test cases...
24 tests selected, 961 tests discarded due to filters
total complete:   24/  24  100%  failed:    0
24 of 24 tests passed with 0 warnings in 14 seconds

this one is already integrated in sanitycheck