Unable to import 'postprocessing' in ESM-based Node.js

I'm using Ava and Node.js to run unit tests, and postprocessing is in the dependencies of the tested code. When running the tests, Node.js hits this error:
SyntaxError: The requested module 'postprocessing' does not provide an export named 'Effect'

Of course postprocessing does export Effect, in fact. I think this comes down to https://github.com/nodejs/node/issues/32137. Node.js expects that ES Modules will either (a) use "type": "module" or (b) use .mjs, if I understand this correctly. Because postprocessing currently does neither, it cannot resolve the import.
This isn't urgent for me, but I just wanted to flag the issue. I see Remove CJS bundles, change type to module on the roadmap; that would certainly solve the issue as far as I'm concerned. I believe it would be a non-breaking change to add "type": "module" now, as long as you keep the main entry and change its extension to .cjs. Example:
https://github.com/donmccurdy/KTX-Parse/blob/d7e7fefbef44642d5dbb9444a64bc055d4483ce8/package.json#L5-L14

EDIT: I'm testing with Node v18.5.0

Sorry, I don't think I follow the connection with those threads — the example config from ktx-parse is what I'm proposing, and hoping might not be a breaking change. I'm not proposing that postprocessing use .mjs files. I don't mind waiting for v7 either, though. Thanks!

Those threads were problems that occured after I tried changing the package type to module or the extension of the bundle to .mjs. I don't remember the exact configs I tried, but it all turned out to be a breaking change in some way or another. Maybe the configuration you propose is actually non-breaking, but I'd rather make the change in v7.

Makes sense, thanks! The whole cjs / mjs situation is really something. I'm glad to see more packages moving to ESM-first. Let me know if I can help here, or feel free to close the issue if this is already covered by the roadmap.


add AMM task

The job is to add loadgen/task-amm.js to define the setup and cycle of a task that exercises the AMM contract.
The setup phase should

install and instantiate a contract which implements a pair of Issuers and an AMM
mint some tokens of each type, in a pair of holding purses
record the objects in the "scratch" table for later use

The cycle function should:

fetch the objects from the scratch table (which does not require a chain operation)
decide whether we should sell A for B, or sell B for A

overall we should alternate between these two operations
the state which helps us remember this should be somewhat durable

we can do a one-time prepare function which queries the balances of both purses, and chooses an initial direction which moves us towards parity
then for as long as the loadgen program runs, it can just alternate
if the loadgen program is killed, the next time we should move towards parity instead of away from it
as long as we have enough tokens to handle a couple of swings, we should never run out

create an offer to sell one for the other, deliver it, fetch the two resulting balances and display them

This task won't use the wallet at all, so it may induce somewhat less load than a real user operation. A later task will be to perform a more complete simulation of the wallet approval process.

I'm pretty sure this is expected for the staking dynamics phase.

I just landed this, thanks @katelynsills for the help! We decided to make the AMM task perform both trades in each cycle, rather than alternating.


fix(clerk-js): Preview known SAML errors during Sign in/up

Type of change

[x] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[x] @clerk/clerk-js
[ ] @clerk/clerk-react
[ ] @clerk/nextjs
[ ] @clerk/remix
[ ] @clerk/types
[ ] @clerk/themes
[ ] @clerk/localizations
[ ] @clerk/clerk-expo
[ ] @clerk/backend
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/fastify
[ ] @clerk/chrome-extension
[ ] gatsby-plugin-clerk
[ ] build/tooling/chore

Description

[x] npm test runs as expected.
[x] npm run build runs as expected.

We add to the list of known sign in/up errors, two SAML related error codes in order to preview the error message to the end user and gain feedback on what failed
Before

After

This PR has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.


about PARAM-SNREF

e.g
PARAM xsi:type="VALUE" SEMANTIC="DATA">
SHORT-NAME>Param_DTCValue
LONG-NAME>DTC Value
BYTE-POSITION>2
DOP-REF DOCREF="ES_DataLibra" DOCTYPE="LAYER" ID-REF="DOP_UINTHEX3Bytes"/>
/PARAM>
ENV-DATA-DESC ID="ENVDESC_DTCExtenDataRecor">
SHORT-NAME>ENVDESC_DTCExtenDataRecor
LONG-NAME>DTC Extended Data Record
PARAM-SNREF SHORT-NAME="Param_DTCValue"/>
ENV-DATAS>
environmentdatadescription.py
line 133
if not isinstance(prev_dop, DtcDop):
DOP_UINTHEX3Bytes is touble_code.the dop is not DtcDop.
Is it correct to change it to this?
    for prev_param, prev_param_value in reversed(encode_state.journal):
        if prev_param.short_name == self.param_snref:
            dtc_param_value = prev_param_value
        dtc_param = prev_param
        prev_dop = getattr(prev_param, "dop", None)
        if not isinstance(prev_dop, DtcDop):
            continue
        dtc_dop = prev_dop

other question.
minmaxlengthtype.py
line 114
odxassert(value_len % len(termination_sequence) == 0)
if if self.termination == "END-OF-PDU",termination_sequence is None. An exception is thrown

I read up the spec, and it indeed allows any parameter that uses a DOP with a A_UINT32 base data type while odxtools currently only allows DTC-DOPs. I'll only be able to fix this in January, though...

if if self.termination == "END-OF-PDU",termination_sequence is None. An exception is thrown

Hm, I cannot see how MinMaxLenthType.__termination_sequence() can ever return None...

def __termination_sequence(self) -> bytes:
    """Returns the termination byte sequence if it isn't defined."""
    # The termination sequence is actually not specified by ASAM
    # for A_BYTEFIELD but I assume it is only one byte.
    termination_sequence = b''
    if self.termination == "ZERO":
        if self.base_data_type not in [DataType.A_UNICODE2STRING]:
            termination_sequence = bytes([0x0])
        else:
            termination_sequence = bytes([0x0, 0x0])
    elif self.termination == "HEX-FF":
        if self.base_data_type not in [DataType.A_UNICODE2STRING]:
            termination_sequence = bytes([0xFF])
        else:
            termination_sequence = bytes([0xFF, 0xFF])
    return termination_sequence

if self.termination is "END-OF-PDU",__termination_sequence return None
e.g
DATA-OBJECT-PROP ID="DOP_BYTEFIELD0ToEOPBytes">
SHORT-NAME>DOP_BYTEFIELD0ToEOPBytes
LONG-NAME>BYTE FIELD 0 to EOP Bytes
DESC>
p>Bytefield beliebiger Länge (incl. 0 Bytes)
/DESC>
COMPU-METHOD>
CATEGORY>IDENTICAL
/COMPU-METHOD>
DIAG-CODED-TYPE xsi:type="MIN-MAX-LENGTH-TYPE" BASE-DATA-TYPE="A_BYTEFIELD" TERMINATION="END-OF-PDU">
MIN-LENGTH>0
/DIAG-CODED-TYPE>
PHYSICAL-TYPE BASE-DATA-TYPE="A_BYTEFIELD"/>
/DATA-OBJECT-PROP>

the is another issue
diaglayer.py
line 408
if not gnr_found:
raise e
if i have 3 service,One of them meets the conditions. It shouldn't be throwing an exception.
           if not gnr_found:
               continue

Whether it's correct or not

if i have 3 service,One of them meets the conditions. It shouldn't be throwing an exception. [...] Whether it's correct or not

in this case, the telegram which you are trying to decode could not be interpreted by odxtools (neither as a request, nor as a response or as a global negative response). In my opinion, raising a DecodeError is well justified in this case. Most likely your ODX dataset is incomplete (i.e., does not describe all data which the ECU can send) or you are stumbling over some issues encountered during the decoding procedure and you are not using the non-strict mode.

(as a work-around you can catch the exception in the code that calls .decode...)

thanks, I understand


after making entire text as bold, enter key is not working

Reproduce step

Implement editor
Select entire text and make it bold/italic
Place cursor at the end of the text and press enter
Cursor is being moved to the beginning of the text

Expected behavior

Just go to next line

More information which could be helpful

Even the cursor is not working properly, a new line will be added. I can go to new line with arrow keys/mouse.

fixed in v0.1.6

all works, thanks


ScalaJS Implementation for our slf4j-api wrapper.

This would mean backporting some of the internals of slf4j to scalajs so this works as well. Some previous art exists in other scala logging libraries.

Couldn't this be done with just log4s?

https://github.com/ChristopherDavenport/log4cats-log4s - Log4s already has a scalajs backend, but the intermediate layer is unnecessary for this abstraction.


wrong key resolving for trunk table hasOneThrough/hasManyThrough

Laravel Version: 5.7
Database Driver & Version: mysql

Description:
wrong key resolving for trunk table
Steps To Reproduce:
class SomeItem {

    public function trunk()
    {
        return $this->belongsTo(Trunk::class, 'trunk_id', 'id');
    }

    public function category()
    {
        return $this->hasOneThrough(Category::class, Trunk::class, 'id', 'id' , 'id', 'product_category_id')
            ->select('category.id as category_id');
    }
}

So, if we run
SomeItem::find(13)->category
 App\Models\Category {#3519
     category_id: 5,
     id: 13, <------ MUST BE '5', '13' is ID of Trunk record
   }

Schema
some_item:

id = 13
trunk_id = 31 <----- issue with this column, instead of searching on '31' it will search on '13'

tunk:

id  = 31   <---------- issue with this column, never searches on '31', instead searches on '13' and overwrites ID '5' from category row to '13'
product_category_id = 5

category:

id = 5
name = 'some name'

Your relationship is incorrect, you aren't specifying the trunk_id column anywhere:
public function category()
{
    return $this->hasOneThrough(Category::class, Trunk::class, 'id', 'id', 'trunk_id', 'product_category_id')
        ->select('category.id as category_id');
}

@staudenmeir
my bad, you are right about trunk_id, but it doesn't fix the issue of ID replacement.
with proper trunk_id  it properly searches on trunk now, but it still replaces ID of category with proper trunk ID now
=> App\Models\Category {#3519
     category_id: 5, <-- correct category ID
     id: 31, <--- correct trunk ID, but must be '5'
   }

The column replacement only happens in Laravel 5.7, right?

yes
i have tested 5.7 only with hasManyThrough and  backported hasOneThrough from 5.8

The column replacement issue was fixed in Laravel 5.8 (#25812), but the fix can't be backported to 5.7 (breaking change).
You can use this package of mine for Laravel < 5.8: https://github.com/staudenmeir/belongs-to-through

i see. thanks for your help!


[FR] Show client ip address in log

This should appear with the --log-ip option, is that working?

solved, got the new option after upgrade


Improve API response to sendTransaction calls

Description
The API currently doesn't provide very meaningful responses when sendTransaction() is called. This PR improves the responses it send back. Note that lots of the functionality provided in this PR isn't currently used, but might be helpful in the future. For example, whether a transaction is successfully sent, or already sealed, or repaid, the API still just responds with the hash in all cases. However, this PR lets the API distinguish between these cases, so in the future it would be easy to let the user know what happened.
Type of change
Insert x into the following checkboxes to confirm (eg. [x]):

[ ] Bug fix.
[ ] New feature.
[x] Enhancement.
[ ] Unit test.
[ ] Breaking change (a fix or feature that causes existing functionality to not work as expected).
[ ] Requires documentation update.

Testing
Modified the existing modApiServer sendTransaction test. Testing of the PendingState changes is desirable, but hard to implement, so it's not included in this PR.
Verification
Insert x into the following checkboxes to confirm (eg. [x]):

[x] I have self-reviewed my own code and conformed to the style guidelines of this project.
[x] New and existing tests pass locally with my changes.
[x] I have added tests for my fix or feature.
[ ] I have made appropriate changes to the corresponding documentation.
[x] My code generates no new warnings.
[x] Any dependent changes have been made.

This is ready for a second review...I've implemented all suggested changes, except the request to add more tests, which is unfortunately pretty hard to do (because PendingState is difficult to test, even with a StandaloneBlockchain).


Flutter tries to first run the first activity instead of looking for Main?

I was trying to install https://github.com/hnvn/flutter_image_cropper which involved adding an activity to the android manifest.  However, it broke when the new activity was placed above the original activity so we assumed that flutter tries to run the new activity since it was placed higher than Main Acitivity.
At least based on this Log:
String: Exit code 1 from: C:\Users\YourFavouriteOreo\AppData\Local\Android\sdk\platform-tools\adb -s LGH86090aac5bf shell am start -a android.intent.action.RUN -f 0x20000000 --ez enable-background-compilation true --ez enable-dart-profiling true --ez enable-checked-mode true com.example.cookbook/com.yalantis.ucrop.UCropActivity:

Starting: Intent { act=android.intent.action.RUN flg=0x20000000 cmp=com.example.cookbook/com.yalantis.ucrop.UCropActivity (has extras) }

It may be a better option in the long term to find
 <intent-filter>
      <action android:name="android.intent.action.MAIN"/>
      <category android:name="android.intent.category.LAUNCHER"/>
  </intent-filter>

Rest of the Error Log:
## command

flutter run

## exception

String: Exit code 1 from: C:\Users\YourFavouriteOreo\AppData\Local\Android\sdk\platform-tools\adb -s LGH86090aac5bf shell am start -a android.intent.action.RUN -f 0x20000000 --ez enable-background-compilation true --ez enable-dart-profiling true --ez enable-checked-mode true com.example.cookbook/com.yalantis.ucrop.UCropActivity:
Starting: Intent { act=android.intent.action.RUN flg=0x20000000 cmp=com.example.cookbook/com.yalantis.ucrop.UCropActivity (has extras) }

java.lang.SecurityException: Permission Denial: starting Intent { act=android.intent.action.RUN flg=0x30000000 cmp=com.example.cookbook/com.yalantis.ucrop.UCropActivity (has extras) } from null (pid=25887, uid=2000) not exported from uid 10337
	at android.os.Parcel.readException(Parcel.java:1683)
	at android.os.Parcel.readException(Parcel.java:1636)
	at android.app.ActivityManagerProxy.startActivityAsUser(ActivityManagerNative.java:3177)
	at com.android.commands.am.Am.runStart(Am.java:640)
	at com.android.commands.am.Am.onRun(Am.java:388)
	at com.android.internal.os.BaseCommand.run(BaseCommand.java:51)
	at com.android.commands.am.Am.main(Am.java:121)
	at com.android.internal.os.RuntimeInit.nativeFinishInit(Native Method)
	at com.android.internal.os.RuntimeInit.main(RuntimeInit.java:262)

#0      runCheckedAsync (package:flutter_tools/src/base/process.dart:251:5)
<asynchronous suspension>
#1      AndroidDevice.startApp (package:flutter_tools/src/android/android_device.dart:440:34)
<asynchronous suspension>
#2      FlutterDevice.runHot (package:flutter_tools/src/resident_runner.dart:287:54)
<asynchronous suspension>
#3      HotRunner.run (package:flutter_tools/src/run_hot.dart:262:39)
<asynchronous suspension>
#4      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:391:37)
<asynchronous suspension>
#5      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:372:18)
#6      _asyncThenWrapperHelper.<anonymous closure> (dart:async/runtime/libasync_patch.dart:77:64)
#7      _rootRunUnary (dart:async/zone.dart:1132:38)
#8      _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#9      _FutureListener.handleValue (dart:async/future_impl.dart:129:18)
#10     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:642:45)
#11     Future._propagateToListeners (dart:async/future_impl.dart:671:32)
#12     Future._complete (dart:async/future_impl.dart:476:7)
#13     _SyncCompleter.complete (dart:async/future_impl.dart:51:12)
#14     _AsyncAwaitCompleter.complete (dart:async/runtime/libasync_patch.dart:28:18)
#15     _completeOnAsyncReturn (dart:async/runtime/libasync_patch.dart:295:13)
#16     RunCommand.usageValues (package:flutter_tools/src/commands/run.dart)
#17     _asyncThenWrapperHelper.<anonymous closure> (dart:async/runtime/libasync_patch.dart:77:64)
#18     _rootRunUnary (dart:async/zone.dart:1132:38)
#19     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#20     _FutureListener.handleValue (dart:async/future_impl.dart:129:18)
#21     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:642:45)
#22     Future._propagateToListeners (dart:async/future_impl.dart:671:32)
#23     Future._complete (dart:async/future_impl.dart:476:7)
#24     _SyncCompleter.complete (dart:async/future_impl.dart:51:12)
#25     _AsyncAwaitCompleter.complete.<anonymous closure> (dart:async/runtime/libasync_patch.dart:33:20)
#26     _rootRun (dart:async/zone.dart:1124:13)
#27     _CustomZone.run (dart:async/zone.dart:1021:19)
#28     _CustomZone.bindCallback.<anonymous closure> (dart:async/zone.dart:947:23)
#29     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#30     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#31     _runPendingImmediateCallback (dart:isolate/runtime/libisolate_patch.dart:115:13)
#32     _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:172:5)

flutter doctor
[✓] Flutter (Channel beta, v0.9.4, on Microsoft Windows [Version 10.0.17134.345], locale en-US)
    • Flutter version 0.9.4 at C:\Users\YourFavouriteOreo\Documents\flutter
    • Framework revision f37c235c32 (5 weeks ago), 2018-09-25 17:45:40 -0400
    • Engine revision 74625aed32
    • Dart version 2.1.0-dev.5.0.flutter-a2eb050044

[✓] Android toolchain - develop for Android devices (Android SDK 28.0.0)
    • Android SDK at C:\Users\YourFavouriteOreo\AppData\Local\Android\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b02)
    • All Android licenses accepted.

[✓] Android Studio (version 3.1)
    • Android Studio at C:\Program Files\Android\Android Studio
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b02)

[!] IntelliJ IDEA Ultimate Edition (version 2018.2)
    • IntelliJ at E:\Program Files\JetBrains\IntelliJ IDEA 2018.2.5
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.io/intellij-setup/#installing-the-plugins

[✓] VS Code, 64-bit edition (version 1.26.1)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 2.19.0

[✓] Connected devices (1 available)
    • LG H860 • LGH86090aac5bf • android-arm64 • Android 7.0 (API 24)

! Doctor found issues in 1 category.

#21765 might be related

Hi @YourFavouriteOreo
Given the mentioned issues are closed
If you're still experiencing this issue
can you please provide your flutter doctor -v and flutter run --verbose?
I will reopen it
Thank you


Implement get_config() getter

Coverage increased (+0.1%) to 96.121% when pulling 7b3eed3ea7692c8dd14bb3526f3b6e402d87f2b1 on get-config into 02b125d9e0ee77b464dc3dc920bf8186cbc9ed6b on develop.


[FEATURE REQUEST] Read checkout to be able to read data from staging area

Is your feature request related to a problem? Please describe.
Currently, read checkout won't be able to read data from the staging area. I was wondering what would be the technical difficulties for reading data from staging area
Describe the solution you'd like
repo.checkout(stage=True) or something similar could give access to the staging area. I think the major bottleneck from implementing this is the fact that the staging area could change and might even make the data unavailable. Maybe we could have a flag changes when the data changes and on read from a stage checkout, it could let the reading process know that the data changed (not the specific information but just that the data changed) and it can invalidate the checkout?

This isn't the craziest thing to do from a technical standpoint, but I'm wondering what the motivation for allowing this is?


environment: line 10: postgres_exporter: command not found

What did you do?
I used this image for monitoring
What did you expect to see?
Working exporter
What did you see instead? Under which circumstances?
environment: line 10: postgres_exporter: command not found
Environment

System information:
insert output of uname -srm here

postgres_exporter version:
insert output of postgres_exporter --version here

postgres_exporter flags:

insert list of flags used here

PostgreSQL version:
insert PostgreSQL version here

Logs:

kubectl logs -n ergo ergo-postgres-instance1-d4mv-0 -c exporter
Starting postgres_exporter with the following flags...
--extend.query-path=/tmp/queries.yml --web.listen-address=:9187
environment: line 10: postgres_exporter: command not found

Any luck? I have the same problem.


🐞 Source editor disappears when deleting all text

Description
Deleting all text from the source editor results in the source editor being replaced by a QuickLook view.
To Reproduce

Open a project
Open a file in the project
Delete all of the text in the file
Notice the source editor is replaced by a big file icon (QuickLook view)

Expected Behavior
The source editor should remain in place.
Screenshots
https://github.com/CodeEditApp/CodeEdit/assets/806104/2d0bb1b3-d69a-43f8-866c-111bf3136b5a
Related issues

#1698
#1707

@plbstl I believe this was introduced in your recent PR #1713. Would you be able to take a look at this one?

I see. I'm taking a look at it now

Oh okay. I'll test it out also

I found some issues with #1723.
Doesn't work for:

files without file extensions
files with only file extensions

Also, it opens some binaries as text files.
https://github.com/CodeEditApp/CodeEdit/assets/49006567/8dc8eb89-6b0a-41af-81ce-870d96af3996

This is what #1724 looks like:
https://github.com/CodeEditApp/CodeEdit/assets/49006567/5e9a5b42-8f6c-4d56-b6b9-67917bfcba77

Good catch, does this PR replace the other one entirely?

Yes it does. They are both minor changes.


Doesn't work with aliases properly

When WebPack has aliases propery:
module.exports = {
...
resolve: {
    extensions: ['.js'],
    alias: {
      default: path.resolve(__dirname, 'default'),
      './default': path.resolve(__dirname, 'default'),
    }
  }
}

and extract-loader is being used with an html-loader
module.exports = {
...
module: {
    rules: [
      {
        test: /\.html$/,
        use: [
          {
            loader: 'extract-loader',
          },
          {
            loader: 'html-loader',
          }
        ]
      }
    ]
}
...
}

end temple that uses aliases default:

...

extract-loader - gives me an error:
Error: Cannot find module './default/course-default.jpg' from '/webpack-demo/src'

But with the same configuration and old version "extract-loader": "^2.0.0" - everything is working just fine!
Is there something wrong happening with the extract-loader?

@wizardion I ran into the same issue, caused by extract-loader attempting to resolve requires without considering aliases.
I forked the repo and fixed this issue - you can always try and giving it a spin to see if it works for you as well.

@vseventer I resolved my problem by getting rid of this horrible extract-loader in my main HTML file template. The extract-loader also brokes my AngularJS templates in my project.
I used it only for ejs variables in my main HTML template, within such configuration:
module: {
    rules: [
        {
            test: /\.html$/,
            use: [
                "ejs-loader",
                "extract-loader",
                "html-loader"
            ]
        }
    ]
}

So I changed the main template of my application to *.ejs file extension to allow my HtmlWebpackPlugin to serve the the ejs variables and got rid of extra loaders. The "html-loader" is still in use for all other HTML files in my project.
But the main issue with this loader is still present, and I don't know why it's not going to be fixed for so long time... 'cause this issue exists from v3.0.0. It's frustrating by spending so much time figuring out for just a simple problem.

Can we get this PR merged in and released? I'm having the same alias issue.


Update README.md

Description
enter here if you have any details
Program Name -
Issue No.- #

Type issue number after hash(#)

Checklist(self check)

[ ] Yes, this task assigned to me.
[ ] My code follows the style guidelines(Clean Code) of this project
[ ] I have performed a self-review of my own code
[ ] My Filename is in camelCase format
[x] I have added Problem statement, Problem link in file.

It will be good If you solve a problem of improve something good in documentation.
Hacktober is for support open source and encourage programmer to contribute to that. Please choose a better task for HacktoberFest.


Switch buildBazelPackage fetchConfigured on by default

Motivation for this change
fetchConfigured (i.e. using build --nobuild) tends to produce better result then using fetch, but it does produce different hashes.
Flip the default, and for all non-marked-as-broken packages in nixpkgs, regenerate the hashes.
Fixes #123601
Fixes #116439
Fixes #87922
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

[x] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 126034 at 233b6859 run on x86_64-linux 1

30 packages skipped due to time constraints:

envoy
gpt2tc
pomerium
python38Packages.arviz
python38Packages.baselines
python38Packages.dm-sonnet
python38Packages.edward
python38Packages.graph_nets
python38Packages.mask-rcnn
python38Packages.optuna
...

3 packages built successfully:

bazel-watcher
gvisor
nixos-install-tools

11 suggestions:

warning: unused-argument
Unused argument: protobuf.
Near pkgs/applications/virtualization/gvisor/default.nix:14:3:
   |
14 | , protobuf
   |   ^

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/development/go-modules/generic/default.nix:223:5:
    |
223 |     installPhase = args.installPhase or ''
    |     ^

warning: missing-patch-comment
Consider adding a comment explaining the purpose of this patch on the line preceeding.
Near pkgs/development/tools/bazel-watcher/default.nix:11:5:
   |
11 |     ./use-go-in-path.patch
   |     ^

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/virtualization/gvisor/default.nix:108:5:
    |
108 |     installPhase = ''
    |     ^

warning: name-and-version
Did you mean to pass pname instead of name to mkDerivation?
Near pkgs/build-support/build-bazel-package/default.nix:55:10:
   |
55 |   inherit name bazelFlags bazelBuildFlags bazelFetchFlags bazelTarget;
   |          ^

Near pkgs/development/tools/bazel-watcher/default.nix:16:3:
   |
16 |   version = "0.14.0";
   |   ^

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/development/tools/bazel-watcher/default.nix:71:5:
   |
71 |     installPhase = ''
   |     ^

warning: unused-argument
Unused argument: future.
Near pkgs/development/python-modules/tensorflow/default.nix:7:3:
  |
7 | , future, setuptools, wheel, keras-preprocessing, google-pasta
  |   ^

warning: name-and-version
Did you mean to pass pname instead of name to mkDerivation?
Near pkgs/build-support/build-bazel-package/default.nix:55:10:
   |
55 |   inherit name bazelFlags bazelBuildFlags bazelFetchFlags bazelTarget;
   |          ^

Near pkgs/applications/virtualization/gvisor/default.nix:56:3:
   |
56 |   version = "20210518.0";
   |   ^

warning: unused-argument
Unused argument: pythonAtLeast.
Near pkgs/development/python-modules/tensorflow/default.nix:4:36:
  |
4 | , buildPythonPackage, pythonOlder, pythonAtLeast, python
  |                                    ^

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/servers/http/envoy/default.nix:99:5:
   |
99 |     installPhase = ''
   |     ^

warning: unused-argument
Unused argument: callPackage.
Near pkgs/applications/virtualization/gvisor/default.nix:4:3:
  |
4 | , callPackage
  |   ^


Cannot remove extra quads in the cylindrical part of CapsuleMesh and CylinderMesh

Godot 3.0.4
There are extra Quads in the cylindrical section of CapsuleMesh and CylinderMesh.
If you remove rings in CapsuleMesh, it also reduces them on the spherical sections, which is unwanted.
You cannot remove all rings from CylinderMesh, there is always at least one in the middle.

You cannot remove all rings from CylinderMesh, there is always at least one in the middle.

This was fixed for CylinderMesh by https://github.com/godotengine/godot/pull/49731.
As for CapsuleMesh, the comments on https://github.com/godotengine/godot/pull/49730 make me think that allowing this change (and adding another property for it) would be undesired. It's not always faster to render less triangles, if having long thin triangles is the only alternative.


redundant info provided for chat slash commands

start typing a slash command
see or hear the slash command's description
accept it via tab
see or hear the slash command's description (ghost text) 🐛

I think it would be better if the one we show as ghost text provides an example of what one might want to type
https://github.com/microsoft/vscode/assets/29464607/d3e0b0d2-3424-4e43-a0f2-58b96bc040ae

I don't think that seeing the same description in the placeholder is an issue- there's many reasons I may not have seen or memorized the detail from the suggest widget and would find it useful to look at while I think about what to type. But showing an example might be nice too. Reminds me of the glob examples in the search view

Yeah and tbh I struggle to sometimes know what to type when demo-ing so I imagine beginners are really puzzled without an example


Support GHC JS backend

Should we remove callbacks? That module has been upstreamed into base.

I had a quick read already, but will do a proper review at the end of today


General Availability

[Enter feedback here]
When will this be GA? What are the known issues, if any?

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

ID: 25ccb59e-98fc-0382-030a-c3805494afd2
Version Independent ID: ed04168c-41ba-e6c6-fe83-f6e5c760ba5a
Content: Create a profile container with Azure Files and Azure Active Directory (preview)
Content Source: articles/virtual-desktop/create-profile-container-azure-ad.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

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

@MarkRagnar
Can you please elaborate on your ask to help me understand better? Thanks

GitHub issues in this repo aren't for product roadmap queries. A better place is the Azure Virtual Desktop Feedback page: https://techcommunity.microsoft.com/t5/azure-virtual-desktop-feedback/idb-p/AzureVirtualDesktop
#please-close


More configuration options for missed rehearsals

How many missed rehearsals are allowed is currently hard-coded; this should be configurable for choir-admins. Additionally, missed rehearsals for which the user has not excused themselves, should be weighted differently. This second weight should be configurable by choir-admins per rehearsal.

Is anybody already working on this? If not, I'm going to start working on this soon :)


Bluetooth: Mesh: Pass correct pointer to publish_sent

This commit fixes a bug where incorrect pointer passed to publish_sent
in access.c caused bus fault.
Signed-off-by: Pavel Vasilyev<EMAIL_ADDRESS>

@jhedberg not so long, was introduced here in #34310.


Added controls to character to move between targeted characters

fixes #170

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

14 of 28 (50.0%)  changed or added relevant lines in 5 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.5%) to 95.591%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Battle.Logic/Characters/Character.cs
5
19
26.32%

Files with Coverage Reduction
New Missed Lines
%

src/Battle.Logic/SaveGames/GameSerialization.cs
3
52.63%

Totals

Change from base Build<PHONE_NUMBER>:
0.5%

Covered Lines:
2524

Relevant Lines:
2648

💛  - Coveralls


Updated import statement

Closes #
📝 Description

Add a brief description

⛳ Current behavior (updates)

Please describe the current behavior that you are modifying

🚀 New behavior

Please describe the behavior or changes this PR adds

💣 Is this a breaking change (Yes/No):

📝 Additional Information

Importint from @chakra-ui/react ist the recommended way.


Adding Negation Evaluators

PR to add some negated evaluators.

[ ] Classes for negated evaluators
[ ] Tests for newly added evaluators
[ ] Code is commented and well formatted

(Related to https://github.com/StackGuardian/tirith/issues/101)


Grid API can't reache API server any more

Steps :

Create vm1 with (js 8.2.0 - g8core - start AYS - clone ays template branch 1.1.0-alpha)
Create vm2 with grid api version 1.1.0-alpha then
run (/opt/code/bin/api --bind :8080 --ays-url http://<IP_ADDRESS>:5000 --ays-repo grid)

Actual result:
Ays

Grid API

Hint
seems that the sanic team did an update yesterday
https://github.com/channelcat/sanic

wrong repo please move

supported version of sanic is 3.0.0, make sur that the version you are running

I didn't choose the version .. it is automatically installed in the requirements I think
moved to
https://github.com/g8os/grid/issues/88


stores don't work

Getting this error: ConstructorError: could not determine a constructor for the tag 'tag:yaml.org,2002:python/tuple'

Fixed.


how to understand the code

Hi, priseborough! I am really grateful that you released the code about inertial navigation. But I can't understand the details. Could you please add some doc about the theory! Any help would be really appreciated!

The most recent code are avaliable in the ECL library: https://github.com/PX4/ecl


GetResourceDownload : A parameter cannot be found that matches parameter name 'ThrowOnError'

This occured when running Start-LabConfiguration against the built-in NanoExample:
VERBOSE: [08:24:22] Resource 'E:\Lability\ISOs\2016_x64_EN_Eval.iso' does not exist.
GetResourceDownload : A parameter cannot be found that matches parameter name 'ThrowOnError'.
At line:114 char:41
$resource = GetResourceDownload @PSBoundParameters;
 CategoryInfo          : InvalidArgument: (:) [GetResourceDownload], ParameterBindingException
 FullyQualifiedErrorId : NamedParameterNotFound,GetResourceDownload

VERBOSE: [08:29:03] Resource 'E:\Lability\ISOs\2016_x64_EN_Eval.iso' checksum does not match '18A4F00A675B0338F3C7C93C4F131BEB'.
Resource download 'E:\Lability\ISOs\2016_x64_EN_Eval.iso' checksum does not match '18A4F00A675B0338F3C7C93C4F131BEB'.
At line:135 char:13
throw ($localized.ResourceChecksumMismatchError -f $Desti ...
 CategoryInfo          : OperationStopped: (Resource downlo...7C93C4F131BEB'.:String) [], RuntimeException
 FullyQualifiedErrorId : Resource download 'E:\Lability\ISOs\2016_x64_EN_Eval.iso' checksum does not match '18A4F00A675B0338F3C7C93C4F131BEB'.

When running the command a second time, without any modifications in the mean time, it ran without issues. I understand that this might be difficult to re-produce...maybe a timing issue?

@janegilring Is this with the published 0.11.0 version on the Gallery? If so it has been fixed in the dev branch. This will be published to gallery soon.

Correct. That`s great :)


Add brauhelfer.app v<IP_ADDRESS>

I've checked all of the requirements mentioned below, the PR complies with them. Hope you accept it.
Thanks & regards
Alex
Adding a new cask

[x] Checked there aren’t open pull requests for the same cask.
[x] Checked there aren’t closed issues where that cask was already refused.
[x] When naming the cask, followed the token reference.
[x] Commit message includes cask’s name.
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.

Thank you.


some methods are missed

Hi I have compiled openssl 1.1.0d using your script for ios armv7 when I use it in my static lib it is complaining (undeclared identifiers) about few methods like:
OpenSSL_add_all_algorithms();
SSL_load_error_strings();
method = (SSL_METHOD*)TLSv1_2_client_method()
SSL_library_init()
Do you have a clue about this?
I am using the same code and same headers for android or windows and linux without problems
Thanks

Do you have setup your project correct, included the headers and the library files?

Im going to close this issue - feel free to reopen if you still need help.


Add commit hash to static path

charles@charles-ThinkPad-P52 ~/cra_frontend_prototype (change-style-filename) $ yarn build
yarn run v1.19.1
$ react-scripts build
Creating an optimized production build...

...

File sizes after gzip:

  152.58 KB  build/staticcf2536b0930692e9ee29f2f0fb094c78dec21c75/js/2.a8abe973.chunk.js
  23.71 KB   build/staticcf2536b0930692e9ee29f2f0fb094c78dec21c75/css/2.32642bf8.chunk.css
  3.25 KB    build/staticcf2536b0930692e9ee29f2f0fb094c78dec21c75/js/main.3c1a6756.chunk.js
  778 B      build/staticcf2536b0930692e9ee29f2f0fb094c78dec21c75/js/runtime-main.e910cc3e.js

charles@charles-ThinkPad-P52 ~/cra_frontend_prototype (change-style-filename) $ BUILD_ID=foobar yarn build
yarn run v1.19.1
$ react-scripts build
Creating an optimized production build...
Compiled successfully.

File sizes after gzip:

  152.58 KB  build/staticfoobar/js/2.a8abe973.chunk.js
  23.67 KB   build/staticfoobar/css/2.8fe61e7a.chunk.css
  3.25 KB    build/staticfoobar/js/main.3c1a6756.chunk.js
  778 B      build/staticfoobar/js/runtime-main.e910cc3e.js

also, an example without a BUILD_ID => it just serves from static/ in this case
charles@charles-ThinkPad-P52 ~/cra_frontend_prototype (change-style-filename) $ yarn build
yarn run v1.19.1
$ react-scripts build
Creating an optimized production build...
Compiled successfully.

File sizes after gzip:

  152.58 KB  build/static-/js/2.a8abe973.chunk.js
  23.67 KB   build/static-/css/2.2b94736a.chunk.css
  3.25 KB    build/static-/js/main.3c1a6756.chunk.js
  778 B      build/static-/js/runtime-main.e910cc3e.js


Test docker-compose deployment

After #42 is done, we should perform E2E tests using the new setup + deployment workflow.
Steps to be taken:

Do a fresh download/clone of the repository
Follow README instructions
Execute pipeline lightweight demo using both Gaia node and Ollama
Provide feedback or describe to issues found

Waiting for feedback from @wtfsayo


Identify libraries that we need to review in order to increase our confidence in ZK Apps working properly in Mina.

The next big task that lies ahead in this project is increasing our confidence in how ZK Apps are handled in Mina. To do this, we need to identify libraries and modules responsible for this and then review their tests to make sure they're sufficient. And are more tests where needed. The resuilt of solving this issue will be creating more issues describing in detail the scope of investigation for each particular component related to ZK Apps.

This is a duplicate of #12442, so closing.


enable "mosh"

surely this should go in some kind of system-wide configuration but I will try
to wedge it into the dot files to start.
https://nixos.wiki/wiki/Mosh

error: The optionprograms.mosh' does not exist.`


Added Mobility Calculator

Fixes Issue🛠
Closes #1204
Description👨💻

The Mobility Calculator estimates the movement efficiency of electric charges in a material, based on factors like electric field strength and charge carrier properties, i.e., drift velocity.
It inputs drift velocity and electric field from the user and returns mobility in S.I. unit.
Type of Change📄

[x] New feature (non-breaking change which adds functionality)

Checklist✅

[x] I am an Open Source contributor
[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project
[x] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

Please add label to it.

Pls add your calculator part in the main index.html file to display it on the website by making the new commit in this same PR @vaishali-gupta32

Please review it.


Unable to download spreadsheet from ingest UI using GEO accession

Hi there,
I tried to download a spreadsheet for GSE176509, project UUID 5ae10405-795e-42e5-85de-3c3154f9c526 and got the following error:

Any ideas what to do?

Hi @brittneydwick! thanks for bringing this up
This is a very weird error indeed. I will bring it to the attention of our devs to see how we can resolve this


fix(deploy): Improve Sign up by email flow during a deploy

Today, when signing up for cloud with email, the deployment flow is interrupted. We land user on an empty cloud instance (without any projects association) that points to Docs page. Not helpful and users end up in a broken state with no clear next steps.
Solution TBD

This already works. The original issue was that redirects was broken.


[BUG] Heavy load on the PC.

Describe the bug
Hello everyone, I noticed only today that Hydra puts a lot of strain on the PC! More specifically, RAM (16GB) and disk! (1TB) And the main thing is that it just works, that is, I don't even do anything in it, and it doesn't download anything, what should I do? By the way, for some reason, when the browser is running, the load disappears...
Steps to Reproduce
?
Expected behavior
No response
Additional information and data
Processor - 50% (AMD Ryzen 5 5500)
RAM - 99% (16GB)
Disk - 100% (1TB)
Operating System
Windows 11
Hydra Version
3.1.2
Before opening this Issue

[ ] I have searched the issues of this repository and believe that this is not a duplicate.
[ ] I am aware that Hydra team does not offer any support or help regarding the downloaded games.
[ ] I have read the Frequently Asked Questions (FAQ).

Точно такая же проблема, приходится полностью закрывать лаунчер чтоб нормально играть. Эта проблема походу и появилась с версией 3.1.2


warning: deleting object of polymorphic class

I'm trying to use this lib with a photon but I get this error.

lib/SparkJson/src/./DynamicJsonBuffer.h: In destructor 'ArduinoJson::DynamicJsonBuffer::~DynamicJsonBuffer()':
lib/SparkJson/src/./DynamicJsonBuffer.h:20:33: warning: deleting object of polymorphic class type 'ArduinoJson::DynamicJsonBuffer' which has non-virtual destructor might cause undefined behaviour [-Wdelete-non-virtual-dtor]
~DynamicJsonBuffer() { delete _next; }
^

I'm also seeing this issue, the code will not compile. I've imported the lib using Build IDE.
In file included from lib/SparkJson/src/./ArduinoJson.h:7:0,                  from lib/SparkJson/src/SparkJson.h:13,                  from lib/SparkJson/src/SparkJson.cpp:1: lib/SparkJson/src/././DynamicJsonBuffer.h: In destructor 'ArduinoJson::DynamicJsonBuffer::~DynamicJsonBuffer()': lib/SparkJson/src/././DynamicJsonBuffer.h:20:33: warning: deleting object of polymorphic class type 'ArduinoJson::DynamicJsonBuffer' which has non-virtual destructor might cause undefined behaviour [-Wdelete-non-virtual-dtor]    ~DynamicJsonBuffer() { delete _next; }


Index out of bounds issue inside proxy protocol parser

https://github.com/envoyproxy/envoy/blob/6f910198c37a25c1932e88102915e2e2b1d81ea6/source/common/filter/listener/proxy_protocol.cc#L122-L135
It's possible to cause index out of bounds error when the search_index_ is 0 and the condition buf_[search_index_] == '\n' is true.
In this case the second condition check buf_[search_index_ - 1] == '\r' will cause an error because search_index_ - 1 will be equal to -1 and will refer to unexisted index.
Some more  details about this type of bug: https://cwe.mitre.org/data/definitions/127.html

Yep. This is good

OK thanks for confirming. Closing.


Instagram requires access token for oEmbed now

Description
Starting Oct 24, 2020, Facebook/Instagram requires an access token (combo of app id and client token) to use their oEmbed product, which is what this plugin uses. See their notice: https://developers.facebook.com/docs/instagram/oembed/

I noticed this because a client site is getting all HTTP 400 errors
The dependency package embed/embed is getting updated to use the new API. The new code is sitting the v3.x branch right now.
Looks like this will pair with some new settings options: https://github.com/oscarotero/Embed#settings
More than anything I wanted to put this on your radar with the hopes you can plan to address it sooner than later once the changes in embed/embed are released in a new version.
Edited:
More PR work submitted for the embed/embed package: https://github.com/oscarotero/Embed/pull/399

Looking at the merged pull request for Embed 3, it's just using the Facebook API key as the access token, which Embedded Assets already has a setting for.  If that's it and there are no extra settings we need to pass, then we just need to enforce an Embed update.
I've just released Embedded Assets 2.4.3 which updates the Embed minimum requirement; does that resolve the issue?

No luck yet, we can't add any instagram embeds anymore at this moment.

Edit: tried to update to the latest embed/embed library (https://github.com/oscarotero/Embed/releases/tag/v4.3.0) , but that seems impossible for now;

@skoften is that with using the Facebook API key in the Embedded Assets settings?
Regarding the Embed update: Embedded Assets supports Embed 3, and won't support Embed 4 until the release of Craft 4; the reason for this being Embed 4's requirement of PHP 7.4 or later, which looks at this stage like it will also be Craft 4's minimum PHP requirement.  However, Embed 3.4.9 should have fixed Instagram embedding (using the Facebook API key as the access token).

@ttempleton without a key at the moment. Can you tell me which key we need exactly? Little bit confused here. Do I need to fill in an app secret or an access token?

Same for us
We've tried adding a Facebook Token using both these methods:

Access Token: App ID | App Secret (https://developers.facebook.com/docs/facebook-login/access-tokens#apptokens)
Client Access Token: App ID | Client token (https://developers.facebook.com/docs/instagram/oembed/#client-access-tokens)

The Client Access Token seems to work for now.
But we are still getting the information from the login screen (see screenshot)

Glad to hear the client access token is working, @ishetnogferre.  I'm going to close this now as it seems like everything is working okay on the Embedded Assets side of things.  Regarding getting the login page information, Embedded Assets can only really work with the data returned by Embed, so I'm going to recommend setting up a test of Embed 3 on the server where that's happening.  If it turns out that the Embed 3 test works fine, then feel free to open a new issue here, but otherwise I'd recommend opening an issue with them.

Did some extra testing because it was still not working. Think I solved it; to have to add "oEmbed" to the products of the Facebook app.

Also, be sure to combine the app key with the client key;

After this we got it working.

@skoften - I'm doing everything you've done and I still can't get all the data. I think it's because you have Instagram Graph API approved in your Facebook Developers app. They keep rejecting my submissions. Any insights into that process?


git-my: init at 1.1.2

Introduces git-my into gitAndTools.
Also takes the luxury beforehand to strongly normalize gitAndTools's naming and sorting. The only package referenced elsewhere in Nixpkgs, git-top, has had its reference updated to not point at the new alias. These are done in separate commits to keep things clean.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)

Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"

[x] Tested execution of all binary files (usually in ./result/bin/)

[x] Determined the impact on package closure size (by running nix path-info -S before and after)
(Which of these are we supposed to post?)

615_240 (derivation)
29_474_872 (output)

[x] Ensured that relevant documentation is up to date

[x] Fits CONTRIBUTING.md.

This change is

Ok, reviews incorporated.

needs a rebase

still needs a rebase, i dismissed my comments

still needs a rebase

Rebsed.

hmm, do you mind separating out the git-my: init at 1.1.2 into a separate PR (that looks read). And make this into the name refactor (more controversial).
I don't want to step on the toes of those maintainers who are having their packages forcefully edited without their knowledge.

darcs-to-git doesn't have a maintainer & hubUnstable is currently throw "use gitAndTools.hub instead".
fast-export is @koral, git-remote-gcrypt is @ellis & @montag451, and top-git is @MarcWeber.
If they're fine with the renames & aliasing, then this shouldn't need to split?

@bb010g I cool merging this after the previous comment is applied.
@jonringer It's good to look out for what other maintainers think about their packages, but this is more of a thing the committers (curators) of the codebase can decide on.

(Huh, GitHub is ordering hub-unstable as the last commit in both the conversations & commits views, even though it's topologically before all the other commits, because of its (severely?) newer author date.)

(Huh, GitHub is ordering hub-unstable as the last commit in both the conversations & commits views, even though it's topologically before all the other commits, because of its (severely?) newer author date.)

I've viewed your stuff locally :+1:


Fix linter links in docs

As described in #137

@kytrinyx there you go

w00t! So fast :) You've been killin' it over here lately! Love it.

Just keeping my brain happy.


feat/crypto: switch to using safe_crypto

LGTM, let's see if build passes, when safe_crypto is published.

Obsoleted by #70.


Info badge colour

This is the first time I've reported something so I'm not sure if I'm doing it in the right place or not.
I note that the new info badge doesn't provide sufficient colour contrast per AA guidelines in the V5 Alpha 2.
https://v5.getbootstrap.com/docs/5.0/components/badge/
The contrast is 1.96 apparently when used with white text as seen in the docs. I use it with black text myself, which does provide the contrast required.
So the docs either need changing or the colour needs changing :)

Thanks for the issue, but please don't grab random PR URLs 😛
The v5 docs are on https://v5.getbootstrap.com/ or the main branch on https://twbs-bootstrap.netlify.app

Noted for future! :) I have checked on the main branch just to make sure and the issue exists on there too :)

@calvinkarpenko For your information, this is already documented here: https://v5.getbootstrap.com/docs/5.0/getting-started/accessibility/#color-contrast
There are probably related issues you should find and read first to understand the rationale behind this choice before proposing to change it 😊

@Mx-Glitter - Thanks for the reply - I'm aware of that and was just mentioning it because it wasn't AA compliant in the docs for Alpha 2 - and figured it may have been slightly overlooked. I'm under the impression that the aim is for the docs to be AA compliant as well :)

We are working towards this, and v5 has already many improvements. I have a WIP PR we plan to land at some point, #29315 .

We are working towards this, and v5 has already many improvements. I have a WIP PR we plan to land at some point, #29315 .

I'm well aware and have been following with interest :) I just didn't know how else to notify you of this and was under the impression it may have been missed, that's all :)

@ffoodd don't we have another issue/PR we track this stuff already?

We do (#30548) and we somehow solved this with #31276 — however badges don't have color variants, they're only relying on our color utilites.
The info badge example should simply use |.ètext-darkalongside.bg-info` to comply, just like warning and light badges. PR incoming :)


update slirp4netns (v1.1.11) and fuse-overlayfs (v1.5.0)

slirp4netns:    v1.1.8 -> v1.1.11 (https://github.com/rootless-containers/slirp4netns/releases)
fuse-overlayfs: v1.3.0 -> v1.5.0  (https://github.com/containers/fuse-overlayfs/releases)

Contains the fix for CVE-2021-{3592,3593,3594,3595} (libslirp v4.5.0 -> v4.6.0) https://gitlab.freedesktop.org/slirp/libslirp/-/blob/v4.6.0/CHANGELOG.md

@AkihiroSuda do you know what the severity is on those CVEs? I see that they're called out in the release notes, but the CVE entries don't appear to have been populated with any info yet; they're still just 'reserved'.

Going forward, please communicate CVE/security concerns via email to<EMAIL_ADDRESS>

Going forward, please communicate CVE/security concerns via email to<EMAIL_ADDRESS>
Sent an email.

Thanks, but I meant in the future before submitting a PR mentioning CVEs or a security issue, email first instead of just submitting a public PR here or in other projects K3s uses. Other projects like RKE2 would use<EMAIL_ADDRESS>fyi

Sorry, I'll send an email next time


Broken image url for OP Tililuotto

Describe the bug
The url for OP Tililuotto logo seems to be broken at checkout page.
To Reproduce
Go to checkout page with Op Payment Service payment method enabled. Broken image is in the div element called "checkout-payment-method-op-tililuotto4".
Screenshots
https://i.imgur.com/qN4o5uD.png

Thanks for reporting!
That payment method is still in development and should not be visible on the test account. We have disabled it now.


Accessing Dapr secret store early on during Startup

I want to use Dapr secret stores to initialize a DB connection with the connection string retrieved from a store. I want to understand the startup sequence and determine the point where secrets are available.
This is my Main:

                var host = CreateHostBuilder(args).Build(); // An OOB host builder, calling `Startup` as usual
                host.MigrateDbContext<MyDBContext>((context, services) => {
                    // DB migration code first. Host is not up ATM.
                });
                host.Run();

Normally, in Startup.cs:ConfigureServices, connection strings are supplied as follows:
        public void ConfigureServices(IServiceCollection services)
        {
            services.AddControllers().AddDapr();

            services.AddDbContext<MyDBContext>(options =>
               options.UseSqlServer(Configuration.GetConnectionString("DefaultConnection")));

            // ... others
        }

To access Dapr through DI, tried this:
            services.AddDbContextPool<MyDBContext>((IServiceProvider sp, DbContextOptionsBuilder options) =>
            {
                var dapr = sp.GetRequiredService<DaprClient>();
                var keys = dapr.GetSecretAsync("localsecrets", "DBConnectionString").AsTask().Result
                options.UseSqlServer(keys["DBConnectionString"]);
            });

In this case, Dapr is called pretty early on, before host.Run(). And I receive "Error starting gRPC call: Connection refused".  GetSecretAsync executes without a problem when called in a controller in my test project.
I can see that Dapr blocks while waiting to discover the application on given port and without host.Run(), that'll never going to happen.
Should we assume that any Dapr service is not available during Startup sequence?

Transferring to dapr repo and closing as duplicate of https://github.com/dapr/dapr/issues/1493


Use an alert key that we control

As far as I can tell (maybe I'm wrong), Namecoin Core is using Vince's alert key, which we don't have control of.  If that is correct, we should create a new alert key.

Yes, I just copied the keys from the old client.  It is on my todo list to change them at some point in the future.  If you want to suggest a way to move forward, feel free to go ahead.

I'd really like this to be fixed before we try to deploy the BIP 9 hardfork.  Hardforks are already hazardous enough; trying to do one without an alert key gives us no ability to take action in case of an emergency.

I have no objections, although I'm not sure it is really that important and useful for the fork.  We have usually been quite able to communicate important things in the past, no?  Nevertheless, we should do that, obviously.
Do you know how to generate a key and how to send alerts with it?  I have no idea so far, that's part of the reason why I didn't bother to change things.

Do you know how to generate a key and how to send alerts with it? I have no idea so far, that's part of the reason why I didn't bother to change things.

So I asked on #namecoin-dev IRC, and turns out that this info is not well-documented, nor well-known even by Bitcoin developers.  I'm still in the process of gathering info on this topic, and I'll post here a summary of what was discussed on IRC once the discussion concludes.

Upstream Bitcoin just removed the P2P alert system entirely.  This matches my previous comment that I think we should rely on other channels for communication instead.  This probably makes the issue here obsolete (but I'll leave it open for now in case @JeremyRand or someone else disagrees).

I'm okay with closing this, but I'd like to wait a week before closing this, so that other developers have a chance to voice any disagreement they may have.

Since no one else has objected and it's been 8 days, I'm marking this as wontfix and closing it.


Look at warnings on Qt wallet start

We get some warnings when we start the Qt wallet:
$ ./MintCoin-Qt
libpng warning: iCCP: known incorrect sRGB profile
libpng warning: iCCP: known incorrect sRGB profile
libpng warning: iCCP: known incorrect sRGB profile
libpng warning: iCCP: known incorrect sRGB profile
libpng warning: iCCP: known incorrect sRGB profile
Could not parse application stylesheet
QIODevice::read (QFile, "style/stylesheet.qss"): device not open

These make us look sloppy, and are probably easy to fix. Check it out.

PNG fixes merged with https://github.com/MintcoinCommunity/Mintcoin-Desktop-Wallet/pull/59

Stylesheet warnings fixed with https://github.com/MintcoinCommunity/Mintcoin-Desktop-Wallet/pull/60


Changing ERD Setting Print Page Mode disables Printing

System information:

Operating system (distribution) and version - Win 10
DBeaver version <IP_ADDRESS>101032114
Additional extensions  n/a

Connection specification:

Database name and version - N/A. This is an ERD issue
Driver name - N/A. This is an ERD issue
Do you use tunnels or proxies (SSH, SOCKS, etc)?  - N/A. This is an ERD issue

Describe the problem you're observing:
When changing the ERD Settings. I changed the Page Mode to Fit Page under the Print section. I hit Apply and Close, then the print icon disables, so I can't print the ERD. If I close the ERD and reopen it, I'm able to print.
Steps to reproduce, if exist:
described above
Include any warning/errors/backtraces from the logs
none I see

thanks for the report

As a dirty workaround you can use Refresh Diagram button in the toolbar menu.

Print button is disabled for Schema/Table ERD until custom diagram creation. But even when it becomes highlighted it doesn't work

Fixed

Still reproducible for non-custom diagran (e.g. for tables). Button is active, but do nothing after click on it.

Fixed

verified


tensorflow is not a supported wheel on this p latform

I have python 2.7.13 (Anaconda 4). When i try install tensorflow its raise error.
tensorflow_gpu-1.1.0-cp35-cp35m-win_amd64.whl is not a supported wheel on this p
latform.

I try use gpu version - same result.

Only Python 3.5 packages are available on Windows -- so use Python 3.5 instead of 2.7.

The comment by @foxik is correct.
On windows, TF only has support for Python 3.5


ci: check list of changed files correctly before scheduling helm chart tests

Description
This pull request changes the following:

check list of changed files correctly before scheduling helm chart tests

Related Issues

Closes #730

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Crashes when EnderIO and EnderCore in libs folder.

Issue Description:
When i put EnderIO and EnderCore in the libs folder, my game just crashes on startup.
What happens:
Game crashes when launching.
What you expected to happen:
Not to crash
Steps to reproduce:
1.Install EnderIO and EnderCore in libs folder
2.Run
3.Crash

Affected Versions (Do not use "latest"):

EnderIO: 1.12.2-5.1.52
EnderCore: 1.12.2-0.5.73
Minecraft:1.12.2
Forge:1.12.2-<IP_ADDRESS>47
SpongeForge? no
Optifine? no
Single Player and/or Server? N/A

Your most recent log file where the issue was present:
https://pastebin.com/r1FU9if4

You put enderio in the mods folder

Okay. But EnderCore still crashes when in libs.

libs is for Java libraries that you can then pull into your project with gradle dependencies.
 If you want to have them on the classpath to use in your mods, declare them as dependencies in your gradle. In that case, you don't need the jar files at all, as gradle will download an ddeobfuscate them for you. Just be aware that you need to manually put the embedded libraries into your mods folder as forge only supports embedded libraries from mod-folder jars, not from classpath jars.

How can i do that?

I got EnderCore, but i cant find enderio reposetory.


Infinitesimal derivative part of StochasticTriples end up as NaN for some instances of MVNormal

MWE:
using StochasticAD, Distributions, LinearAlgebra, ForwardDiff
ϕ = 0.3
function mysample_q(ϕ)
    q_αβ = MvNormal(zeros(4), ϕ * Diagonal(rand(4)))
    return pdf(q_αβ, rand(4))
end

mysample_q(ϕ)
stochastic_triple(x -> mysample_q(x), ϕ) # 0.019046573788085913 + NaNε
ForwardDiff.derivative(x -> mysample_q(x), ϕ) # works

Note that MvNormal(zeros(4), ϕ * Array(Diagonal(rand(4)))) works fine.

This minimizes to the following issue:
julia> using StochasticAD

julia> st = stochastic_triple(0.5)
StochasticTriple of Float64:
0.5 + 1.0ε

julia> zero(1 / zero(st)) # should be 0.0 + 0.0ε
StochasticTriple of Float64:
0.0 + NaNε

This is triggered by https://github.com/JuliaStats/PDMats.jl/blob/fff131e11e23403931a42f5bfb3384f0d2b114c9/src/utils.jl#L63.
With #80 we should get this right.


NuGet Packages not StrongName signed.

Can the NuGet packages please be strong name signed so that programs referencing the library can also be signed.

Ok, I'll sign nuget package later, and you can found the non-signed version in release list.

Ok, we have signed Nuget packages.


Requested number of low priority VMs are not provisioned.

I tried provisioning a cluster with 2 dedicated and 18 low priority E16 VMs in Central India region. It succeeded on the first attempt and worked perfectly. But when I tried it again couple of days later , I saw that only the dedicated ones have been provisioned and no low priority VMs came up even though the state of the cluster went to steady.
I am using a custom spark image. The issue is intermittent.
Am I missing something.

Azure low priority VMs are not guaranteed to be allocated. Aztk clusters are marked as steady once the Spark master has been selected and is up. There is no guarentee that all VMs are available when the cluster goes to steady. Nodes that come up later will join the Spark cluster dynamically.
The behavior you are describing is expected when low-pri VMs. Low priority VMs may be preempted any time after allocation as well.
Since this is expected, I am going to close the issue, but please feel free to reach out if you have more questions.


Infinite loop in new-errors API endpoint.

In ErrorLog.GetNewErrorsAsync, page is never incremented, so if the requested id is not in the first 10 errors, it will keep reading the same page over and over until it runs out of memory.

Hi @cheese83,
I made a PR to fix this issue. Do you mind give it a look, to give it a like if it fixes your issue? :)


11847 security update vulnerable talkerjs packages

Bumping versions of vulnerable modules

 [Security] Update vulnerable talker.js packages


Can't Install GIT

Multiple app have the id git.git --> impossible to install Git

Try winget install --id git.git

winget install -e --name Git ended up working for me.

or winget install Git --exact

-e works fine for me. Seems that a friendly reminder is needed for such situation.

I had the same problem then I found the -e or --exact option. I think -e is should not be required when the --id option is provided.

Thanks for theses responses 👌


Nintendo and Sony Detection wrong (uses vendor and product instead of just vendor)

Vendor detection is bugged due to combining vendor and product in order to determine the current vendor.
Only vendor should be used
eg (official USB vendor id)
Sony 054c
Nintendo 057e
Xbox 045e and 0738
those get translated in SDL GUID and the two characters sets get swapped
this bug causes many controllers to be detected as the wrong type (eg: Nintendo Switch Combined Joy-Cons with Joycond on Linux since they have a GUID of Joystick GUID:     06004cfc7e0500000820000000000000 end up returning Gamepads.Xbox ). https://github.com/FosterFramework/Foster/blob/72c45061702e9a2cf850edbb86312a4e618a8d2d/Framework/Input/Controller.cs#L121-L140
Please change this to only use the vendor and not the vendor and product id.
also, the entire purpose of the SDL2 gamecontrollerdb mapping system is to abstract that away.

Yeah this makes sense to me. I use gamecontrollerdb in my other projects and have meant to pull it into this for forever, but have not, so that would be good to do.
Are we sure that the vendor is the only thing we want to check? I haven't really looked through them all but if it's fine to do that then it sounds good to me. Alternatively it might make more sense to just use SDL_GameControllerGetType but I haven't yet looked to see if that covers everything we'd want it to.
Regarding joycond, is it needed anymore for modern versions of the kernel now that hid_nintendo is included? I know Arch linux says to remove it now:

If you have joycond, then delete it, because it is useless for such Switch-like gamepads, moreover joycond has a udev rule that disallows Steam to provide its own user-space driver.

(This is a little outside my area of knowledge, I just know someone on Celeste64 had issues with their Switch Pro controller until they removed joycond)

My desire is to by default use the physical locations (north/south/east/west), except where specifically desired (ex. for confirm/cancel prompts which should be swapped for Switch). Things like a "Jump" action should always be the South button, regardless of whether that's A/B/X/Y/etc. If I leave SDL_HINT_GAMECONTROLLER_USE_BUTTON_LABELS to 1 then it will also swap around actions that I do not want to be moved around.
It sounds like how SDL3 is implemented is ultimately what I want: north/south/east/west for inputs, and it's up to the game to swap confirm/cancel based on the controller, and the labels hint no longer does anything.

My desire is to by default use the physical locations (north/south/east/west), except where specifically desired (ex. for confirm/cancel prompts which should be swapped for Switch/sony). Things like a "Jump" action should always be the South button, regardless of whether that's A/B/X/Y/etc. If I leave SDL_HINT_GAMECONTROLLER_USE_BUTTON_LABELS to 1 then it will also swap around actions that I do not want to be moved around. Based on the documentation and my own work with Xbox/Switch/PS4 controllers I want this hint off.
It sounds like how SDL3 is implemented is ultimately what I want: north/south/east/west for inputs, and it's up to the game to swap confirm/cancel based on the controller, and the labels hint no longer does anything.

SDL3 implementation is like the hint being ON. Hence why I suggest leaving it on and implementing everything based on that. It will be easier then when you transition to SDL3 as the only change you will need to do is change from interpreting gamecontrollertype to using GetGamepadButtonLabel to determine what label should be shown in GUI dialogs.
With the hint ON, controls in game are automatically correct (SDL_CONTROLLER_BUTTON_A is always guaranteed to be South). So all you need to do is check gamecontrollertype to switch the icons/functionality for menus. Much easier that way than the other way around, and like I said, it is the way SDL3 will force you to use.

SDL3 implementation is like the hint being ON.

I think you're mistaken/have this backwards. According to the documentation:
The variable can be set to the following values:
 *    "0"       - Report the face buttons by position, as though they were on an Xbox controller.
 *    "1"       - Report the face buttons by label instead of position

If this is enabled ("1"), Switch Pro controllers automatically swaps around A/B on me without me doing anything, which is what I do not want. With this off, they use the north/south/east/west positions, which is what I want.

yes, sorry, I have it backwards. but the rest of what I said still applies. I think the best way of moving forward (if you are to keep using SDL2 for the time being) would be to use gamecontrollertype to interpret the vendor information as a means of assigning on screen button prompts rather than the current https://github.com/FosterFramework/Foster/blob/72c45061702e9a2cf850edbb86312a4e618a8d2d/Framework/Input/Controller.cs#L121-L140

Yeah, I totally agree the current way it gets gamepad is weird and should be fixed. I think I'm in favor of using SDL_GameControllerGetType and either making the Foster enum 1-1 or map those values, and then we don't need to worry about parsing vendor information at all, if that makes sense?

Yeah, I totally agree the current way it gets gamepad is weird and should be fixed. I think I'm in favor of using SDL_GameControllerGetType and either making the Foster enum 1-1 or map those values, and then we don't need to worry about parsing vendor information at all, if that makes sense?

making the foster enum 1-1/mapping makes sense. since you might want to show different icons for sideways joycons (where ABXY doesn't make sense). its not a case that could be reasonably used in Celeste64 (which kinda needs two analog sticks) but I can imagine other games wanting it.

also as a sanity check, I modified https://github.com/Grumbel/sdl-jstest to print that info and can confirm that my switch pro controller and combined joycon return their expected values
Found 2 joystick(s)

Joystick Name:     'Nintendo Switch Combined Joy-Cons'
Joystick GUID:     06004cfc7e0500000820000000000000
Joystick Number:    0
GameControllerTypeForIndex: 13
Number of Axes:     4
Number of Buttons: 22
Number of Hats:     0
Number of Balls:    0
GameControllerConfig:
  Name:    'Nintendo Switch Combined Joy-Cons'
  Mapping: '06004cfc7e0500000820000000000000,Nintendo Switch Combined Joy-Cons,a:b0,b:b1,back:b9,dpdown:b15,dpleft:b16,dpright:b17,dpup:b14,guide:b11,leftshoulder:b5,leftstick:b12,lefttrigger:b7,leftx:a0,lefty:a1,misc1:b4,rightshoulder:b6,rightstick:b13,righttrigger:b8,rightx:a2,righty:a3,start:b10,x:b3,y:b2,platform:Linux,'
  GameControllerType: 13

Joystick Name:     'Nintendo Switch Pro Controller'
Joystick GUID:     050056fb7e0500000920000001800000
Joystick Number:    1
GameControllerTypeForIndex:  5
Number of Axes:     4
Number of Buttons: 18
Number of Hats:     0
Number of Balls:    0
GameControllerConfig:
  Name:    'Nintendo Switch Pro Controller'
  Mapping: '050056fb7e0500000920000001800000,Nintendo Switch Pro Controller,a:b0,b:b1,back:b9,dpdown:h0.4,dpleft:h0.8,dpright:h0.2,dpup:h0.1,guide:b11,leftshoulder:b5,leftstick:b12,lefttrigger:b7,leftx:a0,lefty:a1,misc1:b4,rightshoulder:b6,rightstick:b13,righttrigger:b8,rightx:a2,righty:a3,start:b10,x:b3,y:b2,platform:Linux,'
  GameControllerType:  5

OK awesome, I've refactored this now by using SDL_GameControllerGetType: https://github.com/FosterFramework/Foster/commit/fe672db5ad387d298c9b6d3b8c14018189688b63
I definitely think this is better overall.


Umlauts are not indexed properly

Umlauts at the beginning of a string are ignored. For example go to https://w3id.org/kim/hochschulfaechersystematik/scheme and type in "Über", this is what you get:

Although we do not use a forward tokenizer yet (see #153), words that do not start with but contain an Umlaut are indexed ad two words, resulting e.g. in something like this when typing "ühgesch" or "hgesch" at https://w3id.org/kim/hochschulfaechersystematik/scheme:

#186 is now deployed on test and I let this build anew: https://test.skohub.io/acka47/testing-skohub-vocabs/heads/master/index.de.html
Umlauts now don't work in a different way than before:

Theay are apparantly not ignored anymore but transformed to their corresponding vowel:

As mentioned in https://github.com/skohub-io/skohub-vocabs/pull/197#issue-1326736011 Uppercase umlaute should now work.

Theay are apparantly not ignored anymore but transformed to their corresponding vowel:

This is still the case and requires some adjustments in either App.js or the nestedList.js component. So #197 will not close this issue. But it will bring some improvement.

Functional review for #197:
As @sroertgen says, this is an improvement as upper case Umlauts now also work (I noticed that my "Über" example in https://github.com/skohub-io/skohub-vocabs/issues/164#issuecomment-1202724985 isn't a good one as there eists no "Über" in the Hochschulfächerklassifikation):

The change from #197 can be deployed to production, I will open a PR against the master. Otherwise I agree with @sroertgen that this will need some more work to close the issue by handling Umlaute correctly.

Reopening as #197 doesn't completely fix the problem (see above two comments by me and @sroertgen ).


Unable to get Magnetometer information

Hi all,
I am trying to get the information from the magnetometer using the following line of code:
async for compass_info in drone.telemetry.MagneticFieldFrd():             print(compass_info.forward_gauss)
but I am getting the following error

AttributeError: 'Telemetry' object has no attribute 'MagneticFieldFrd'

Can you help me please?
Thank you in advance

Could it be called like this?
async for compass_info in drone.telemetry.magnetic_field_frd():
   print(compass_info.forward_gauss)

Thank you for your quick response.
I already tried that but I am getting the same error

AttributeError: 'Telemetry' object has no attribute 'magnetic_field_frd'

Odd. I can reproduce it.

Oh, I figured it out:
    async for imu in drone.telemetry.imu():
        print(imu.magnetic_field_frd.forward_gauss)

Oh thank you so much!! Just one last question how do I convert the magnetometer measurements in Gauss to a compass angle? I cannot find the way of converting it.
Thank you

If you need the compass angle, why don't you just use the yaw angle which the PX4 estimator already calculates?
    async for euler in drone.telemetry.attitude_euler():
        print(euler.yaw_deg)

Oh, I figured it out:
    async for imu in drone.telemetry.imu():
        print(imu.magnetic_field_frd.forward_gauss)

I was faced the exact issue as mentioned by OP, but those lines of code didn't help as well. After executing the script seems to stay in an infinte loop, because there were no ouput in command prompt. On the other hand the code
    async for euler in drone.telemetry.attitude_euler():
        print(euler.yaw_deg)

is executed just fine.
I was trying to connect my script to PX4 via USB on Windows 10.

Thanks for the heads-up.


Add incidence flag to SEIRModel

Closes #24 .

@DavAug There is a syntax error in line 87: non default argument follows default argument. I don't know what is wrong with it. Could you please help me have a look?

@DavAug There is a syntax error in line 87: non default argument follows default argument. I don't know what is wrong with it. Could you please help me have a look?

@siting-miao
This basically just means that if you have a function

@DavAug There is a syntax error in line 87: non default argument follows default argument. I don't know what is wrong with it. Could you please help me have a look?

This basically means that default arguments always have to come last in python i.e.
def simulate(self, parameters, times, return_incidence=False, t_i, t_f):

fails because returns incidence has a default and  t_i and t_f do not have a default.

I also realised that we can solve this problem a little easier than I suggested in the issue. I will post some comments in the issue #24 .


Subdirectory support

Hey,
first of all I would like to thank you for your nice work. My question or request is more a pull request but I was not able to create them. Is it possible to add subdirectory support? I plan to use your tool on an internal source server. But it's not possible to get a new IP-Address for each generated source view.
I tried it by my own but I think it's easier if you can arrange that because I first have to understand the whole code beforer I can do it by myself.
Thank you very much
Regards
Raphael

Unfortunately this is really hard with the current architecture and would require large-sweeping changes. I myself won't have time to work on this and I'm unlikely to take a pull request that attempts this (too many things may be broken). However if you're courageous feel free to try it in your own fork.
The biggest problem to solve is the component that's hosted on the webserver (SourceIndexServer). I don't know how to make multiple instances of index co-exist in the same web API (or have multiple API endpoints per host).
Additionally, we generate URLs of the form /assemblyname/... everywhere, assuming it starts from the root. We'd have to go through the entire codebase and make sure URLs are relative to the root of the current index, not the entire website.
Here's some more information:
https://github.com/KirillOsenkov/SourceBrowser/wiki/Architecture
Instead, one workaround (that admittedly won't work for some scenarios) is to merge all codebases into a single index. Have you tried that?


Packing kfp-schedwf fails in CI

Packing kfp-schedwf failed in CI run during Test the bundle job. The issue is intermittent though and rerunning the CI packed the charm successfully.
I also see a lot of those in the CI
httpx.HTTPStatusError: Client error '404 Not Found' for url 'https://<IP_ADDRESS>:16443/apis/kubeflow.org/v1/profiles/kubeflow-user-example-com?fieldManager=kfp-operators'

Closing this since not enough information were provided during the creation of this, while the linked logs have expired. We should upload the logs themselves for issues occuring in the CI.


Setting multiple columns into UpdateByProperties breaks SetOutputIdentity

If BulkConfig looks like the one below, and Prop1 is not unique(but Prop1+Prop2 is unique), then with SetOutputIdentity = true identities will be messed up.
SetOutputIdentity = true,
UpdateByProperties = new List<string>
{
    nameof(Prop1),
    nameof(Prop2),
}

The issue seems to be in TableInfo.GetQueryExpression where only first property is used for ordering in the identity retrieval query
expression = ordered
     ? OrderBy(entityType, expression, PrimaryKeysPropertyColumnNameDict.Select(a => a.Key).First())
     : expression;

I would assume addition of ThenBy will fix this.
v 5.2.8

I've made a fix for this, take the latest source and check if it works for your case, then I'll publish the NuGet.

v5.2.9 published


De-experimentalize generated callback code

This allows the code to no longer use "experimental_async()", "ExperimentalCallbackService", etc, and also changes the method signatures to use the non-experimental versions when possible (under the appropriate macro).
Is this the right approach to how the generated code should look? Hard to tell. I think so. Although having #ifdef's in the generated code may be confusing to users since they may think that they have to override both versions of the code in their service definitions, in practice, nobody is currently using the experimental version except for those that will get to migrate right now. Then we can remove the experimental for everybody else with the gRFC once the OSS EM is ready.

This change is

Thanks for the review! I've made all your requested changes.


My first branch

Created a new pull request by comparing changes between main and my-first-branch.
Able to merge. These branches can be automatically merged.

Step 3: git merge my-first-branch
merge: my-first-branch - not something we can merge
Did you mean this?
origin/my-first-branch
git merge origin/my-first-branch
Merge made by the 'recursive' strategy.
PROFILE.md | 1 +
1 file changed, 1 insertion(+)
create mode 100644 PROFILE.md

Step 4. git push -u origin main
remote: Support for password authentication was removed on August 13, 2021.
remote: Please see https://docs.github.com/en/get-started/getting-started-with-git/about-remote-repositories#cloning-with-https-urls for information on currently recommended modes of authentication.


Trying to replicate an experiment

Hi there,
I train by leaving the defaults. Specifically, I used:
./tools/dist_train.sh configs/fcosrbox/fcosr_rx50_32x4d_fpn_3x_dota10_single.py 0,1,2,3 --seed 129 --deterministic

Then I measure mAP by executing:
python tools/test.py configs/fcosrbox/fcosr_rx50_32x4d_fpn_3x_dota10_single.py work_dirs/DOTA10/FCOSR-M/FCOSR_rx50_32x4d_fpn_3x_dota10_single/latest.pth --eval mAP

My result is:
npos num: 0
{'iou_50': {'mAp': 0.0, 'detail': {'plane': 0.0, 'baseball-diamond': 0.0, 'bridge': 0.0, 'ground-track-field': 0.0, 'small-vehicle': 0.0, 'large-vehicle': 0.0, 'ship': 0.0, 'tennis-court': 0.0, 'basketball-court': 0.0, 'storage-tank': 0.0, 'soccer-ball-field': 0.0, 'roundabout': 0.0, 'harbor': 0.0, 'swimming-pool': 0.0, 'helicopter': 0.0}}}

What am I missing? I have tried with 3 seeds: 98 , 8 and 129. With the same result

After further investigation, here is what I have found:

The default config uses "trainval" for training as well as for validation. Then it uses the original DOTA "testing" for testing.
In my case, I used the original dota "train" partition for training with the intention to use the validation partition for testing. However, I wrongly cropped the validation partition by calling the function "DOTA2COCO_ONLYIMG" within "DOTA_devkit/prepare_dota.py"
After fixing this mistake, I get meaningful mAP values.

Following the default configuration, the testing points to a json in coco format that has no annotations (because the testing partition in dota does not provide annotations). How is then the performance of the model checked? Is it checked against the DOTA server? I don't see any code or reference about that. Any tips are welcome
[https://github.com/lzh420202/FCOSR/blob/master/configs/fcosrbox/dataset/dota10_single.py#L50](testing partition used in default config)

After further investigation I trained with trainval and the testing results ready for pushing to dota testing server are empty for some categories. Whay is that? Is that expected?
I am referring to categories bridge and basketball-court. you can see the 0Bytes for these files:

Did you encounter this issue?

Using the full trainval partition for training and testing on the test partition will result in successful (complete) out files. Thanks!


app_key does not exist

I get response 'app_key does not exist'.
I looked at client.php and noticed that it doesn't encode auth string into base64.
I'm sure that this is an issue.

Hello.
I am a WeChat professional.
I can help you anytime and I trust you with your photo.
My ID is "ZAZA" or "mch316".
Anytime you can contact me.
My email address<EMAIL_ADDRESS>


feat: add .gitignore and clean up ignored files

solve the problem mentioned in #5

@TomasAndersonFang please review it, thx :)

Thanks for your contribution. It does help improve our project.


Nullability and JSON fields

According to Prisma's docs JSON fields cannot be directly set to null. Instead one can utilize Prisma.JsonNull or Prisma.DbNull in order to set the field to a JSON value of null or to NULL the db field respectively.
My current plan is to update the generated zod schemas to match this behavior but I am curious as to what others think about this.

The only issue that I see with this approach is using these schemas to validate forms in the frontend. We could use null on the frontend and add a transform that converts it to Prisma.JsonNull or Prisma.DbNull


Sound recording

Whether to increase the function of recording sound on the basis of the original, which can help us a lot, thank you

Hi @Jamesyang-l
My apologies, I don't quite understand the request. Could you explain a little more about what you're trying to achieve?

I need to record the sound through your encapsulated driver, similar to the NI tape sound acquisition function

One of the included example VIs is Audio Capture Example.vi, which is similar to NI's sound input example. I've created a similified version that doesn't save to wav. You can download the VI here: G-Audio Capture Example.vi

thankyou


Wrapping an external sampler: PolyChord

Recently we had a short discussion on discourse (https://discourse.julialang.org/t/mcmc-landscape/25654/83 and around) about using an external nested sampling implementation (e.g. PolyChord) with Turing. I made a thin wrapper (https://github.com/aplavin/Polychord.jl) around the underlying C/Fortran library half a year ago that works fine, but it would be nice to benefit from Turing model definition language. However, I couldn't find a simple way to add this a a Turing sampler - looking at existing samplers it looks like the control flow has somewhat "inverted" direction in Turing and in PolyChord library, at least regarding priors. The workflow for the latter looks like this:
# `prior` converts from uniform cube to model parameters
# e.g. if we have two parameters, one with Normal(0, 1) prior and the other with Normal(5, 2):
prior(cube::Vector{Float})::Vector{Float} = invcdf.([Normal(0, 1), Normal(5, 2)], cube)

# computes likelihood for given model parameters
loglike(params::Vector{Float})::Float = ...

run_polychord(prior, loglike, options)

Any suggestions on how to wrap it as a Turing sampler?

IS might not be typical because it's not a MCMC sampler. I wonder if we can clean up the MH code a bit and make that as an example.

@cpfiffer Good idea to improve our MH sampler, in terms of modularity, better interface, and advanced features (e.g. adaptive variants).
Some recent related discussions:

Adaptive Metropolis Algorithm (RAM) - https://discourse.julialang.org/t/mcmc-landscape/25654
MH-sampler in BAT.jl

https://discourse.julialang.org/t/ann-dynamichmc-2-0/28314/22
https://github.com/bat/BAT.jl

It looks like David Anthoff implemented RAM last month. This also looks pretty easy to migrate over to the interface.

Yes; I added it here a few days ago. I feel we can have a really flexible/advanced MH sampler in Turing, by falling back to different MH backends (e.g. Emsemble, RAM, vanilla MH).

It should be significantly easier to get the prior probability of parameters, and likelihood probabilities separately after https://github.com/TuringLang/Turing.jl/issues/989 is merged. This might pave the way for integrating/implementing nested sampling algorithms.

Another implementation of nested sampling: https://github.com/joshspeagle/dynesty
Speagle, J. S. (2019) ‘dynesty: A Dynamic Nested Sampling Package for Estimating Bayesian Posteriors and Evidences’, arXiv [astro-ph.IM]. Available at: http://arxiv.org/abs/1904.02180.

Even simpler to implement than MultiNest (dynesty and nestle implement it in Python) and parameter-free is MLFriends. It is illustrated and explained a bit at https://johannesbuchner.github.io/UltraNest/method.html

See https://github.com/TuringLang/NestedSamplers.jl


[5.1] BeanstalkdJob bury function delete job unexpectedly

Job will be deleted whenever bury job function is called.
Due to the release flag is not mark, every job goin to bury will be delete eventually.
        if (! $job->isDeletedOrReleased()) {
            $job->delete();
        }

Can find no indication that releasing is needed before burying. Please send link to documentation that states this is necessary.

Yes. No releasing is needed, it's just a workaround so that $release set to true and the job won't be deleted.
The bug is the job would never be bury, it's just deleted at last.

I'm not sure releasing it is the answer then. We should solve the problem in another way.

Yes. Thanks Taylor. it's affected 5.2 as well.

Should we add a bury and isBuried function in Job class? But there is only beanstalkd got this bury function


Downloading classification data

I saw that #611 closed a little while ago. What's the scoop on getting classification data out? I'd like to see the results of the Season Spotter beta...

There should be a "Request data export" button on the lab/project details page now. It'll send you an email with a link to the classifications.

Perhaps this was implemented after the stand-alone Season Spotter beta got put up? I don't have that option available at:
http://demo.zooniverse.org/seasons/#/lab/403
(But I do see it on my projects on panoptes-staging.)

Yeah that button http://demo.zooniverse.org/seasons/#/lab/403 will work, it's the same backend generating it for you.

@camallen The thing is I don't HAVE that button on http://demo.zooniverse.org/seasons/#/lab/403. The button is not there.

Sorry, i thought (But I do see it on my projects on panoptes-staging.) meant you did.

Can't you just access it via http://demo.zooniverse.org/panoptes-front-end/#/lab/403?

Not sure. Are these the same on the backend?
http://demo.zooniverse.org/seasons/#/lab/403
http://demo.zooniverse.org/panoptes-front-end/#/lab/403

Yep.

Oh. Right. Okay, then yes, I can just do that. Thanks!

Um, when I press the button and retrieve the data via email, I have 4 classifications -- all from 5/1 or 5/6. Likely these are testing classifications that I made, as we didn't launch the beta until 5/14. Where are all the beta testing classifications?! I hope this is a retrieval bug and not a storage bug. @camallen can you easily check how many classifications are associated with project 403?

There are only 4 classifications for that version of your Project.

Project 405 and 406 are the ones that have all the classifications.

Sorry everyone, @edpaget 's totally right.


support https

Currently, only http protocol is supported.
To enable db instances that are only available through https, a few changes have to be made.
Switch to xmlrpc.createSecureClient if https is needed.
https://github.com/eXist-db/node-exist/blob/0b8ef3e9eb6973065c1eff1440586181ec97a6f9/components/connection.js#L17
It is yet not decided on how to trigger switching to https.
Idea:
const ex = require('@existdb/node-exist')

ex.connect({
    'secure': true
})

the property makes sense. Design wise i d say https by default would be a good idea, with http as a fallback, but that is also more work.

@duncdrum not really just a different default. The issue is more with local instances not having valid certs which would make the tool fail by default.

Setting this to https by default for security while also defaulting to trust bad certs for convenience is far worse, in my opinion.

we're on the same page with not trusting bad certs, I meant try https if that fails due to bad cert, port closed, whatever, fallback to http. The goal being to use https whenever its available.
But a fallback might also hide the fact that folks are using http, meh i m torn 🤷♂

@duncdrum revisiting this issue: what about to switch from secure: false to secure: true and  documenting it that for local development you need to specify secure: false explicitly?

This turns out to be a breaking change for the "normal" developer setup:
The options now need to change to include either

{ secure: false, port: 8080 } to continue using http
{ rejectUnauthorized: false } for the usual existdb that comes with an expired cert OOTB

:tada: This issue has been resolved in version 2.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Many terms appear to be filenames

58824d7fb6a6fae3,1,1,4.61273e-07,mp_photo_guidelines.html
65f2c1768fda3aaa,1,0,4.61273e-07,engineering.svg
48f47d14ac8f85df,1,0,0.00314992,cover.jpg
39cf0dcb77b80dd8,1,0,0.000296944,filmposter.jpeg
44cd8472a0872e66,1,0,0.00436699,logo.png

filmposter.jpeg actually appears in https://pt.wikipedia.org/wiki/O_Amor_Natural_(filme).
mp_photo_guidelines.html actually appears in https://en.wikipedia.org/wiki/File:Marshole.jpg.
It seems fairly common for Wikipedia pages to include file names in captions and as link text.


update mission name to SeaLion

The mission name & description in package.json should be updated to SeaLion

closed as per https://github.com/ODU-CGA-CubeSat/mission-conops/commit/45cfbcb21beaffe554502389b0a0e283b2786b72


Add link to Google Podcasts

Lots of people searching "Syntax fm" on google podcast, but that isn't the name of it. We should link it directly from the website.
A few questions for android folk

Is google play for podcasts still a thing?
Is there a link to syntax on android?

@wesbos For your questions

Google will phase out Google Play Music in favor of youtube music. Podcasts will probably be "Google Podcasts" only. For now it still alive
For the links, there's no "Share" button in the app

weirdly "Syntax fm podcast" doesn't work but "Syntax fm google podcast" does for me ¯_(ツ)_/¯

Got it, going to work on it in a bit.


Sample ADO pipeline files fail

In the Azure DevOps example publisher pipeline apim-stage.yaml, the checkout step needs fetchDepth: 0 or else it won't be able to do a comparison of the previous commit with the current commit.

Looks like its related to this change:
steps.checkout definition | Microsoft Learn

This has been fixed in the latest release


Fix a couple of typos in the relationships docs

This eliminates an occurrence of a duplicated "of" and a duplicated "the".

@dotnet-policy-service agree

@roji wow, that was quick! Thanks for merging, have a nice day :)


Update toc.yml

As before @torfjelde , if you're happy to approve, please just merge it, since I don't have commit rights


Json schema JsonPath expression

Hello,
Could not find a json schema which can be used to validate a JsonPath expression. Does it exist?

Hah was not a very clever question. Off-course it doesn't have a schema because it is not json at all 😅


Support for aws secret manager authentication

Added the Support for TriggerAuthentication using aws secret manager
Checklist

[ ] When introducing a new scaler, I agree with the scaling governance policy
[ ] I have verified that my change is according to the deprecations & breaking changes policy
[ ] Tests have been added
[ ] Changelog has been updated and is aligned with our changelog requirements
[ ] A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
[ ] A PR is opened to update the documentation on (repo) (if applicable)
[ ] Commits are signed with Developer Certificate of Origin (DCO - learn more)

Relates to #4628

Looking good! ❤
Could you rebase the main branch (there are some merge conflicts) and also add some e2e test for this feature? We already have some e2e tests for Azure Key Vault that you could use as example: https://github.com/kedacore/keda/tree/main/tests/secret-providers
In order to have the KMS, you need to create it through the testing-infrastructure repo (we can help you with that if you want)
Also, this feature requires changes in docs explaining it 🙏

Any update? We aim to cut a release next week

I am really occupied by other things. I don't have plans to do this in coming week.

Replaced by https://github.com/kedacore/keda/pull/5162