Only sets AVMetadataObjectTypes that are available

Resolves a crash when passing types that aren’t available.

Current coverage is 5.44% (diff: 100%)

Merging #24 into master will increase coverage by 0.74%

@@            master       #24   diff @@
========================================
  Files            9         9
  Lines          383       386     +3
  Methods         95        95
  Messages         0         0
  Branches        26        26
========================================
+ Hits            18        21     +3
  Misses         363       363
  Partials         2         2

Powered by Codecov. Last update 9fa7910...34bb7f1

Thank you for the fix! 👍


[FLINK-15128][hive][doc] Document support for Hive timestamp type

What is the purpose of the change
Document support for Hive timestamp type
Brief change log

Document support for Hive timestamp type

Verifying this change
NA
Does this pull request potentially affect one of the following parts:
NA
Documentation
NA

cc @JingsongLi @bowenli86 @xuefuz @zjuwangg

LGTM, can you resolve the conflict?

Thanks @bowenli86 , PR rebased.


feat: add typography styles for mobile

Update typography styles for upcoming mobile view tasks.

LGTM, @QuentinSaubadu can you please verify the values before we merge?

FOR:
subparagraph: {
fontSize: '14px',
MISSING:
line-height: 18px
Letter spacing: 0.2px
Then all good.
@gregogun @pradel

LGTM, @QuentinSaubadu can you please verify the values before we merge?

FOR: subparagraph: { fontSize: '14px',
MISSING: line-height: 18px Letter spacing: 0.2px
Then all good.
@gregogun @pradel

@QuentinSaubadu this one is actually already there, it didn't show up in green because I didn't change the value :)


Add sidekiq web admin UI rackup script and required gems

Sidekiq has a nice admin interface that gives us queue status, and can review errors etc.
Rather than using basic auth we are going to put this behind an OAuth2 gateway (run separately) and configured in the load-balancer to live under /sidekiq.
Although sidekiq is sinatra and could be mounted in the PVB app putting just one path behind OAuth would involve installing omniauth and we felt this was a smaller and safer change to make it happen this way. (It also means that there's less scope that a mistake in the config/routes.rb could open it up to the world, but that was not the driving factor behind doing it this way.)

This should be rebased onto #279.

@pdb … and now master.

Closing because this needs to be merged into master now.


Env vars from one configuration bleed into another

Issue Type: Bug
I've seen this multiple times but when I tried to come up with a repro it wasn't consistent.  I've got an environment variable DISABLE_SLOW tests defined as "1" in one configuration but not in a second. After using the first configuration for a while, then switching to the second, I'm finding that DISABLE_SLOW is stilled set to the value "1" from the first configuration even though it's not set at all in the second.
EXPECTED: Env vars set in one configuration shouldn't hang around and affect another.
Here are my configurations from launch.json:
        {
            "name": "Launch Tests",
            "type": "extensionHost",
            "request": "launch",
            "runtimeExecutable": "${execPath}",
            "args": [
                "--extensionDevelopmentPath=${workspaceFolder}",
                "--extensionTestsPath=${workspaceFolder}/out/test"
            ],
            "stopOnEntry": false,
            "sourceMaps": true,
            // outFiles is used for locating generated JavaScript files, see https://code.visualstudio.com/docs/nodejs/nodejs-debugging#_source-maps
            "outFiles": [
                "${workspaceFolder}/out/**/*.js"
            ],
            "preLaunchTask": "npm: compile",
            "env": {
                "MOCHA_grep": "", // RegExp of tests to run (empty for all)
                "MOCHA_invert": "0", // Invert the RegExp
                "AZCODE_ARM_IGNORE_BUNDLE": "1",
                "MOCHA_enableTimeouts": "0", // Disable time-outs
                "DEBUGTELEMETRY": "1", // 1=quiet; verbose=see telemetry in console; 0=send telemetry
                "NODE_DEBUG": "",
                "DISABLE_LANGUAGE_SERVER_TESTS": "0",
                "DISABLE_SLOW_TESTS": "1"
            }
        },
        {
            "name": "Launch Tests (webpack)",
            "type": "extensionHost",
            "request": "launch",
            "runtimeExecutable": "${execPath}",
            "args": [
                "--extensionDevelopmentPath=${workspaceFolder}",
                "--extensionTestsPath=${workspaceFolder}/dist/test"
            ],
            "stopOnEntry": false,
            "sourceMaps": true,
            // outFiles is used for locating generated JavaScript files, see https://code.visualstudio.com/docs/nodejs/nodejs-debugging#_source-maps
            "outFiles": [
                "${workspaceFolder}/dist/**/*.js"
            ],
            "preLaunchTask": "npm: pretest-dev",
            "env": {
                "MOCHA_grep": "", // RegExp of tests to run (empty for all)
                "MOCHA_invert": "0", // Invert the RegExp
                "MOCHA_enableTimeouts": "0", // Disable time-outs
                "DEBUGTELEMETRY": "1", // 1=quiet; verbose=see telemetry in console; 0=send telemetry
                "NODE_DEBUG": "",
                "AZCODE_ARM_IGNORE_BUNDLE": "0"
            }
        }

VS Code version: Code - Insiders 1.40.0-insider (c58aaab8a1cc22a7139b761166a0d4f37d41e998, 2019-10-22T05:27:32.053Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Xeon(R) W-2133 CPU @ 3.60GHz (12 x 3600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.73GB (11.96GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

vscode-typescript-tslint-plugin
ms-
1.2.2

azurerm-vscode-tools
msa
0.7.0

This is related to how we start the EH window since the env vars seems to be preserved.
Thus also assigning to @weinand

Env vars set in one ("extensionHost" debugging) configuration shouldn't hang around and affect another.
If they do, this is a bug.

I was bitten by this bug this week - made our released extension behave in test mode in my vscode.
If it can be fixed soon, is there any easy walkaround to unset/recover polluted env vars automatically after using the configuration?

As a workaround you can unset env variables with the value null, e.g.
            "env": {
                "NODE_DEBUG": null
            }

This problem only occurs on Windows.

Thanks for the fix!
This problem occurred on Mac OS and Linux too.

@hyangah I was not able to reproduce the problem on macOS or linux, but I have no explanation why.
My fix is in no way platform specific.


Create Dockerfile

fixes: #1062
added dockerfile to run danger with plugins.

I'm gonna ignore those travis failures, as they're to do with the new bundler release. Shipping! Thanks.


fixed typos

I'm not allowed to merge. I don't have write access :(

Try again! - have added you as an admin

still no write access :(


Enable configuration cache for all projects generated by gradle init

Fixes #24101.

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

Pull Request Feedback (Trigger) (Check)

The merge queue build has failed. Click here to see all failures.

The merge queue build has failed. Click here to see all failures.


RTL Support

Add rtl support to Element package. #2037

Element is a ui library. It can't satisfy the needs of all users. RTL is a feature but not the most wanted feature. Where to use RTL CSS should be determined by the user.


Docker Mac CE external storage error

Thanks for documenting this.  I have a problem with the final check:
When I run this command:
diff --brief -r $EHOME/Library/Containers/com.docker.docker $HOME/Library/Containers/com.docker.docker
Only in /Volumes/KESU/Users/ohms/Library/Containers/com.docker.docker: ._com.docker.docker
Only in /Users/ohms/Library/Containers/com.docker.docker/Data: backend.sock
Only in /Users/ohms/Library/Containers/com.docker.docker/Data: docker.sock
Only in /Users/ohms/Library/Containers/com.docker.docker/Data: osxfs.sock
Only in /Users/ohms/Library/Containers/com.docker.docker/Data/vms/0: 00000002.000005f4
Only in /Users/ohms/Library/Containers/com.docker.docker/Data/vms/0: 00000002.00001000
Only in /Users/ohms/Library/Containers/com.docker.docker/Data/vms/0: 00000002.00001001
Only in /Users/ohms/Library/Containers/com.docker.docker/Data/vms/0: 00000002.0000f3a5
Only in /Users/ohms/Library/Containers/com.docker.docker/Data/vms/0: 00000003.000005f5
Only in /Users/ohms/Library/Containers/com.docker.docker/Data/vms/0: 00000003.00000948
I ran rsync.  Then I ran mv and same thing.
Any ideas?

@impactcolor I hope you figured this out because I honestly just saw this today. I'm sorry for not getting back to you.

@impactcolor if you ever see this, my first thoughts would be a permission issue.  Mac treats the Library folder differently. If you run the rsync or move function in sudo it may solve your problem.

I'm going to close this in a week unless there is more feedback


Compute DeploymentStatistics on demand, from fresh data

@mpolden please review.

Controllers on old and new version may disagree about confidence for controller and system versions for a short while. I doubt that's a problem.


CLI module for object replication service

Resource Provider
What is the Azure resource provider your feature is part of?
Azure Storage
Description of Feature or Work Requested
Provide a brief description of the feature or work requested. A link to conceptual documentation may be helpful too.
Adding ORS configuration to CLI: Set/Get/Remove-AzStorageObjectReplicationPolicy, New-AzStorageObjectReplicationPolicyRule, New-AzStorageObjectReplicationPolicyRuleFilter - these are the 5 commands we'd like to have available in CLI.
Minimum API Version Required
What is the minimum API version of your service required to implement your feature?
Swagger Link
Provide a link to the location of your feature(s) in the REST API specs repo. If your feature(s) has corresponding commit or pull request in the REST API specs repo, provide them. This should be on the master branch of the REST API specs repo.
Target Date
Mid-October is ideal.
We're preparing a demo for MarkRus at Ignite and need a way to perform ORS operations using public APIs/tools.

Hi @artemuwka, if there is any update for swagger file, please let me know and provide the link in your template.

The private package is delivered, but wait for feedback. In this way, it is fine to close it now


Reduntant type info

Currently, in some cases, the generator emits reduntant type info, like for fields of input types or models. Also, it not generates that type info for @Args in resolvers.
The first iteration would add the emitRedundantTypesInfo generator option, that would generate all the missing redundant types info, so that 3rd party compilers without types metadata support would be able to work correctly with the generated code.
Second iteration would invole detecting when the type info could be ommited, so that we will reduce the ammout of emitted code, that could speed up and save some space when using bundlers.
The redundancy in inputs, args and models would be disabled by default and enable full info with this option enabled.

First iteration done in 36113bee. You can now use emitRedundantTypesInfo generator option.

Really cool!
How would I go about integrating this?
I enabled the emitRedundantTypesInfo flag and did a quick test build with esbuild but I still run into these reflection errors:
/Users/xxx/Projects/boilerplate/apps/server/dist/server.js:69362
        throw new errors_1.NoExplicitTypeError(prototype.constructor.name, propertyKey, parameterIndex, argName);
        ^

NoExplicitTypeError: Unable to infer GraphQL type from TypeScript reflection system. You need to provide explicit type for 'auth' of 'AdminAuthResponse' class.
    at Object.findType (/Users/xxx/Projects/boilerplate/apps/server/dist/server.js:69362:15)
    at /Users/xxx/Projects/boilerplate/apps/server/dist/server.js:69767:53
    at __decorateClass (/Users/xxx/Projects/boilerplate/apps/server/dist/server.js:34:25)
    at Object.<anonymous> (/Users/xxx/Projects/boilerplate/apps/server/dist/server.js:246548:1)
    at Module._compile (node:internal/modules/cjs/loader:1155:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1209:10)
    at Module.load (node:internal/modules/cjs/loader:1033:32)
    at Function.Module._load (node:internal/modules/cjs/loader:868:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:81:12)
    at node:internal/main/run_main_module:22:47

The mentioned AdminAuthResponse class:
@ObjectType()
export class Authentication {
  @Field(() => String, { nullable: true })
  token: string | null;

  @Field(() => Date, { nullable: true })
  expiresAt: Date | null;
}

@ObjectType()
export class AdminAuthResponse {
  @Field()
  auth: Authentication;

  @Field(() => Admin, { nullable: true })
  admin: Admin | null;
}

@nephix
@Field(() => Authentication)
auth: Authentication;

Thanks @MichalLytek !

Second iteration would invole detecting when the type info could be ommited, so that we will reduce the ammout of emitted code, that could speed up and save some space when using bundlers.

Would some of that duplication go away with that or is that only about the emitted code?
Or is TypeScript v5 going to help with this? (https://github.com/microsoft/TypeScript/pull/50820)


Modal not working on Ember 1.13.2 and Liquid Fire 0.20.1+

Modal is working only on Liquid fire 0.20.0 with Ember 1.13.2
code:
router.js
 this.route('test', {path: 'test/:test_id'}, function () {
     this.modal('test-modal', { withParams: 'showTest' });
  });

test controller
showTest: false,
actions: {
showTestModal: function () {
        this.set('showTest', true);
      }
}

test template
<button {{action 'showTestModal'}}>show</button>

This works fine in my app in Liquid Fire 0.20.0
Doesn't work on: 0.20.1, 0.20.2, 0.20.3, 0.20.4
It was a problem because on 0.20.0 liquid-spacer doesn't work, i saw commit from #302 fixing liquid-spacer so i've added it to 0.20.0 for now so i could have both working.

Same issue here as well. Except for me, 0.20.0 doesn't fix the modal issue either. When attempting to launch a modal via queryParam, I get:
Uncaught TypeError: Cannot read property 'detectNamespace' of undefined - ember.debug.js:50354

Attempted to use master, but this is still an issue there as well. Using ember 1.13.3.

Same issue here after updating to ember-cli 1.13.x in liquid-fire 0.20.4
The modals seem to work if I refresh and click the button again, but after navigating a bit, they're not working at other routes, they only seem to work at the route I did the refresh in.
I'm also getting the detectNamespace error in 0.20.0

Yes, I think this is a duplicate of #313. Please reopen if you're still able to reproduce on ember 1.13.5 with liquid-fire 0.21.0 (which I'm about to release).


Getting unwanted file type

I am using v3
var docxIdentifier = (Platform.OS === 'android') ? "application/vnd.openxmlformats-officedocument.wordprocessingml.document" : "org.openxmlformats.wordprocessingml.document"
    var docIdentifier = (Platform.OS === 'android') ? "application/msword" : "com.microsoft.word.doc"
//
const res = await DocumentPicker.pick({
        try: [DocumentPicker.types.pdf, docxIdentifier, docIdentifier]
      })

I still can select other file types (eg. images) when it only should be pdf ,doc and docx. I tested this a while back and it was working fine.
I'm testing on Samsung S9+

I've tried your problem, when i remove docIdentifier it works. I'm testing on IPhone 6

hello, filtering file extensions is documented in the readme, see https://github.com/Elyx0/react-native-document-picker#typestringarraystring
thank you


Moderation commands

WIP, feel free to add to this.

Are we assuming the permissions have already been checked for using the bot commands or does that need to be done within?

Permission checks will be done before executing command, on each command most likely.


The color of the play/pause button applies to all profiles

Description
Changing the color of the play/pause button changes its color across all profiles, not just the selected one.
How To Reproduce
Steps to reproduce the behavior:

Open Settings
Select some profile from the dropdown
Go to Profile Settings > Buttons > Play/Pause Button
Change the text color settings to a different color and hit Apply
Switch to a different profile
The changed color is applied not just to the selected profile, but when you change profiles it is also included in the other profiles

The play/pause button is the only place I've noticed this. Text color of the text labels is set per-profile, but play/pause button color seems to be global. Because of this inconsistency I assume this is not intended functionality
Version

Windows 10 19044.1586
AudioBand v1.1.2

Logs/Screenshots Attachment
AudioBand.log
If applicable, feel free to add screenshots to show us the issue.

Was able to reproduce, thank you for reporting Ill get working on a fix soon!

Issue is resolved, will be in the next release


nixos/luksroot.nix: fallback to interactive password entry when no keyfile found

Motivation for this change
being able to boot if the keyfile is missing but I know the passphrase
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-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 nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Nice, that was something I wanted for my Fileserver.
The only issue I see is, that I can imagine use cases where I want to unlock a device at boot if a keyfile is available, but still continue booting when it is missing. I'm not sure if this is possible at the moment with luksroot but I have this issue with the filesystems.*.encrypted module atm so it would be nice if that'd (still) work.

That's a good point -- I was mostly thinking of the case of unlocking / : if it is not available, the system just doesn't boot.
What should I do ? adding an option required guarding this fallback mechanism ? Adding a timeout to the cryptsetup prompt ?

maybe a enablePasswordFallback oder disablePasswordFallback Option?
If a timeout is chosen there should be a option to configure it IMO, with the possibility of waiting forever. Actually that might be the most flexible way.

Timeout feels somewhat wrong to me (what if you have a long password?). Something better may be to set a number of retries before booting continues. Usual three seems to be enough. What do you think?

@abbradar not sure I understand your point about retries. The main problem is that if the luks device is not so important, and that the key goes missing, the system can't boot unattended anymore.
That the user can ^C is (from that point of view) irrelevant.
By the way, if a key is missing with the current bahavior for a "non-essential" drive, what happens ?

@symphorien Hm, indeed, ^C doesn't help the usecase when you want to continue unattended boot with the key missing but the partition not strictly required.
Maybe then use timeout not for the prompt itself, but for the fallback? In lines of "Key for ${foo} is not found. Press any key in N seconds to enter password phrase instead".  Then one can type the password at their own leisure.

@symphorien I don't remember the code well now, but I think the system will just continue to boot. The drive will stay encrypted.

I pushed a commit implementing an option fallback guarding the interactive prompt. It is set to false by default and thus backward compatible.
I am open to bikeshedding about the name of the option.
I tested all the 4 combination of {key file is present}×{fallback==true} on my computer. It works for me™

I think it would be good to name the option, so it is clear what it does. But I don't have a good idea sadly.
A few ideas, but they might be too verbose

fallbackToPasswordIfKeyFileIsMissing (long, and doesn't have 'keyfile' at the start)
passwordFallback (unclear that it only applies when keyfile is set)
keyFileFallback (unclear what that means. E.g. is it another file?)
keyFilePasswordFallback (Could be interpreted, that is is a string with the password)

I think I'd prefer the last one, if no one has a better idea.

I had also thought of interactiveFallback

Sorry for the delay.
I finally went for fallbackToPassword.
If nobody objects, I think this is ready to merge.

I know there's the 18.03 milestone, but… is there anything missing for this PR to be merged?


[MRG] Add functionality to record somatic voltage from all cells in simulation

closes #181. Starting this PR to discuss how this should be implemented. At the moment I am considering a dictionary indexed by gids, the values would be an array of the somatic voltage recorded. Alternatively it could be implemented as its own object like the Spikes class.
Another point to consider is if we want to record and save voltages by default, or have it enabled when specified by the user.

starting a PR after changing 3 lines, I think you're getting the hang of open source and open science now ;-)
umm ... the cleanest API would probably be to rename Spikes to something like Cell, then rename existing Cell to CellBuilder and then anything related to cells will be part of this Cell object: soma voltage, currents, spikes etc. Do you think it's doable by you or too much?

Just to elaborate a bit. You should be able to do:
>>> net = Network(...)
>>> net.cells[50].spikes
>>> net.plot_spikes()
>>> net.cells[50].vsoma
>>> plt.plot(net.cells[50].vsoma)

I'll give it a shot! I have a bit more free time in the coming weeks so ideally I can make decent progress.

First pass at storing voltages in the Spikes class. Everything seems to be working ok but I'll need to compare against the HNN recorded voltages. Here's an example the voltages from each cell class during the default simulation:

Implemented some basic slicing functionality for the spikes class (the next commit will allow for arbitrary lists/arrays) At the moment the slice operation returns smaller Spikes object with the content filtered by gids that match the slice.
One consideration is that indexing gids that do not exist will return an empty list. Considering the structure of spikes._times and similar attributes I think this makes the most sense. However I'm wondering if a warning should be raised if slice produces a completely empty slice object.

Codecov Report

Merging #190 into master will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #190      +/-   ##
==========================================
+ Coverage   68.07%   68.08%   +0.01%
==========================================
  Files          19       19
  Lines        2039     2040       +1
==========================================
+ Hits         1388     1389       +1
  Misses        651      651

Impacted Files
Coverage Δ

hnn_core/network.py
66.27% <100.00%> (+0.20%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7097b52...a8c6e88. Read the comment docs.

Well this is no fun, the test Travis is failing on seems to run fine on my machine. In any case I restructured storage of vsoma to mimic cell.current by setting up recordings when the cell is instantiated. In the future it may be worth moving cell.current data to the proposed CellResponse class, however the previous MPI errors may reflect an issue with creating h.Vector() recordings in the wrong scope.
@rythorpe @jasmainak  If you have the chance can you try running test_compare_across_backends() from your machine? If there is an error it will likely read "MPI simulation didn't return any data"
In the mean time I may push some variants on handling somatic voltages to try and appease Travis/MPI.

yep I could replicate. The problem is that the you don't have mpi4py installed, so you're not able to replicate. Can you do:
import mpi4py

in the ipython terminal? Also try putting a breakpoint (e.g., just add a line saying "1/0") in your test, and run it with "--pdb" flag and try doing "import mpi4py". I think it won't work.
Now the issue is the following -- we don't get a message that the test was skipped when mpi4py is not installed. I thought this was fixed but apparently we didn't do the proper fix when I looked at it. We should do something like this so the entire test is skipped when mpi4py is not installed. @ntolley want to give a shot at fixing that first? Basically, copy these lines and then rebase this pull request once that's merged into master

also cc-ing @blakecaldwell since he's the MPI expert :-)

By the way, just for your info. When you have something finicky that you can't replicate on your box, there is also the option of ssh-ing into the Travis build and trying there.

I thought this was fixed but apparently we didn't do the proper fix when I looked at it.

The way I did it preserves comparing results when the joblib backend is used. Maybe you want to split this test up like it was before https://github.com/jonescompneurolab/hnn-core/commit/e847137eec6ca75925b8d86e07b7a7a8efb29aaa#diff-c85e4dc16a1586adb223901f0de74fdf

@ntolley I found a problem in how the results are being passed between mpi_child.py and MPIBackend.simulate(). This is a failure of my code to correctly pad the data. So I'll come up with a fix in new PR and you can rebase? It seems that tests past once I fix this.
@jasmainak since you have a specific plan in mind for how the tests should look, what about taking that on in a future PR?

While #192 is still open, you can workaround this by
diff --git a/hnn_core/mpi_child.py b/hnn_core/mpi_child.py
index 5d02ab85..ec5d34b7 100644
--- a/hnn_core/mpi_child.py
+++ b/hnn_core/mpi_child.py
@@ -92,6 +92,7 @@ def run_mpi_simulation():
                                         'base64')

         data_iostream.write(repickled_bytes + b"===")
+        data_iostream.write(repickled_bytes + b"====")

     # flush anything in stderr (still points to str_err) to stdout
     sys.stderr.flush()

Thanks @blakecaldwell!
@jasmainak to finish off this PR I'll work on cleaning up indexing the Spikes object. Depending on how long #192 takes to merge I may start work on storing somatic currents in the same format.
Keeping the integration timeline in mind, do you think the proposed test PR can be merged afterwards?

I thought this was fixed but apparently we didn't do the proper fix when I looked at it.

The way I did it preserves comparing results when the joblib backend is used. Maybe you want to split this test up like it was before e847137#diff-c85e4dc16a1586adb223901f0de74fdf

Just to provide more context, the commit that combined running both backends under a single test was designed to speed up and simplify the test. The idea was that each backend needed two simulations: one for comparing to the default 'ground truth' simulation and one for comparing consistency between the other backends. It looks like we'll need to rethink this, however, given the mpi2py import issue. Maybe we can also consider shortening the default 'ground truth' simulation in order to speed up the tests that require simulations.

we can use pytest fixtures to avoid duplicating the same test.

Keeping the integration timeline in mind, do you think the proposed test PR can be merged afterwards?

I would say bring this PR to a mergeable state as soon as you can. I would also like to work on other PRs which could create merge conflicts and slow things down. Adding the currents should be easy once we have this working, no?

Yeah adding currents would work nearly identically as somatic voltage. A cleaner solution might actually be to combine aggregate_currents() and aggregate_voltages() functions. This is more complicated due to how the dipole is calculated (summation occurs implicitly in the parallel call), but shouldn't be too difficult.

Will dig into the Travis error tomorrow. The error message is actually being printed now since the voltage data isn't passed through stderr, but I am unsure if it is related to how I am storing voltage data or some other issue:

In any case, I've flushed out __get_item__() function for the Spikes class which allows indexing by gids. @jasmainak @rythorpe as we discussed some more thought will need to go into how we want users to interact with the fully fledged CellResponse class, but I think this is a good start.
Once the Travis error is resolved I'd be happy to merge, please feel free to suggest changes to the gid indexing or voltage recording.

should we remove this example: https://jonescompneurolab.github.io/hnn-core/auto_examples/plot_firing_pattern.html#sphx-glr-auto-examples-plot-firing-pattern-py
and instead use your function to plot the soma voltage?

The error message is actually being printed now since the voltage data isn't passed through stderr, but I am unsure if it is related to how I am storing voltage data or some other issue:

Oh no, I've seen this error before and it likely could relate to my last PR. I won't have time to check into it until the weekend, but if you find any reason why it's failing with this version as opposed to a previous version of this PR, that'd be helpful troubleshooting info.

@jasmainak Should I just work on modifying the file? The change really just amounts to running simulate_dipole() at the top.
@blakecaldwell No worries Blake, I'll start looking at your PR as I will eventually need to understand the MPI implementation. Unfortunately I am still failing to reproduce the error locally, but will post an update if I find anything.

@jasmainak Should I just work on modifying the file? The change really just amounts to running simulate_dipole() at the top.

Yeah, that older implementation is not very meaningful. At the time, I still didn't have a good handle on this issue. You can remove/chuck it. What I'd like to see in an example is a plot like the one you shared above.

I've updated the example so that the "same" plots are produced, but the voltage data is accessed via the Spikes object. The plots are rather different since cell.plot_voltage() actually runs a separate simulation with step current injections. @jasmainak  do you think this function is still necessary or can it be removed?

@jasmainak do you think this function is still necessary or can it be removed?

go ahead and remove it.
did you check if the outputs are comparable to hnn GUI outputs?

Also, it would be helpful if you can do a %timeit (in ipython) or just use the time module to check how long does the simulation take with and without the vsoma being returned. Maybe on two different configurations:

a canonical use case for the GUI which would be a single trial parallel across neurons.
Multiple trials so the vsoma has a more significant impact

This way we can make an informed decision whether to give the user an option to turn it off.

@ntolley just a heads up that #196 is about to be merged and should fix the MPI Backend failures.

Ran some quick tests to see impact of storing voltage on simulation time. Below compares joblib and MPI with either 1 or 4 trials of the default parameter set.
somative_voltages (sec)
'JL_1t': 59.2, 'MPI_1t': 22.7, 'JL_4t': 129.8, 'MPI_4t': 87.8
master (sec)
'JL_1t': 58.6, 'MPI_1t': 31.3, 'JL_4t': 129.5, 'MPI_4t': 79.3
The impact seems fairly negligible but that may depend on RAM availability. Once the MPI fix is merged I will test with more trials (padding error only shows up with long simulations or several trials).

@jasmainak I found that one of the tests in test_cell.py utilized plot_voltage() here. Is there another purpose to the function that I'm missing? Otherwise I'll get rid of the test as well.

Is there another purpose to the function that I'm missing? Otherwise I'll get rid of the test as well.

Yep remove the function and the test.

You will need to rebase against master now to get @blakecaldwell 's new changes

Codecov Report

Merging #190 into master will increase coverage by 0.17%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #190      +/-   ##
==========================================
+ Coverage   67.91%   68.08%   +0.17%
==========================================
  Files          19       19
  Lines        2035     2040       +5
==========================================
+ Hits         1382     1389       +7
+ Misses        653      651       -2

Impacted Files
Coverage Δ

hnn_core/network.py
66.27% <100.00%> (+12.55%)
:arrow_up:

hnn_core/parallel_backends.py
13.45% <0.00%> (-1.61%)
:arrow_down:

hnn_core/network_builder.py
92.92% <0.00%> (-0.03%)
:arrow_down:

hnn_core/feed.py
45.18% <0.00%> (ø)

hnn_core/params.py
61.37% <0.00%> (ø)

hnn_core/mpi_child.py
0.00% <0.00%> (ø)

hnn_core/tests/test_network.py
100.00% <0.00%> (ø)

hnn_core/tests/test_compare_hnn.py
96.10% <0.00%> (+0.27%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c299d31...9a4f1c7. Read the comment docs.

@blakecaldwell It looks like your data length check is getting raised, perhaps this is a problem with my implementation?

@blakecaldwell It looks like your data length check is getting raised, perhaps this is a problem with my implementation?

It'll take some debugging to figure this out. Would you be willing to try some things? The spikes object now has somatic voltages. Can you tell if the inclusion of that within net.spikes is causing the error?

@blakecaldwell It looks like the point of failure is spikedata = neuron_net.get_data_from_neuron() which is called from _clone_and_simulate().
Commenting out the lines which converts vsoma from a hoc to python object seems to resolve the error.

Commenting out the lines which converts vsoma from a hoc to python object seems to resolve the error.

Thanks. So it seems when vsoma_py is empty, that it succeeds. I've been able to get it to work on Travis spuriously, but haven't found the exact issue yet.

@ntolley after lost of time searching, this is most likely the culprit.
https://github.com/jonescompneurolab/hnn-core/pull/198/commits/2e425c53ad60fe223ede18050a52d047c2202447
My PR #198 ended up becoming rather large, so it may take longer to merge. Perhaps you can git cherry-pick that commit to move forward.

@ntolley after lost of time searching, this is most likely the culprit.
2e425c5
My PR #198 ended up becoming rather large, so it may take longer to merge. Perhaps you can git cherry-pick that commit to move forward.

I take this back. The whole PR is needed.

I've addressed the comments/suggestions raised above. I'm pretty satisfied with the state of this PR and have changed the status to [MRG]. What remains code review and fixing lingering travis errors.
A bonus of adding the ability to turn off the voltage recording is we don't need to wait for the MPI fix. test_parallel_backends.py passes just fine due to the recording being disabled by default.
Unfortunately a new error has been raised that is rather cryptic:

@jasmainak @rythorpe have either of you encountered this before?

@jasmainak @rythorpe have either of you encountered this before?

just to confirm it's not a problem building the mod files on Travis are you able to reproduce this error locally?

Unfortunately I am not. Is my PR somehow impacting the mod file build or is it unrelated?

okay then your best bet is to ssh into Travis: https://docs.travis-ci.com/user/running-build-in-debug-mode/. You're going to go ninja-level fixing this bug :)

I've addressed the comments/suggestions raised above. I'm pretty satisfied with the state of this PR and have changed the status to [MRG]. What remains code review and fixing lingering travis errors.
A bonus of adding the ability to turn off the voltage recording is we don't need to wait for the MPI fix. test_parallel_backends.py passes just fine due to the recording being disabled by default.
Unfortunately a new error has been raised that is rather cryptic:

@jasmainak @rythorpe have either of you encountered this before?

I was able to reproduce this error on my machine (ubuntu), however, I'm a bit baffled by how the custom mechanisms get loaded when running tests outside of NetworkBuilder on the master branch. I feel like I need to understand this better before I can begin troubleshooting. @jasmainak do you understand how this works?

you load custom mechanisms using this line: https://github.com/jonescompneurolab/hnn-core/blob/a1cc05d9146ebab3bf9623cb9611b722fee55c63/hnn_core/network_builder.py#L159.
By the way, I encountered the same error today while working on #201 . In my case, it was a bit of red herring. The actual problem was that some Neuron variable was being deleted outside the scope of the class (remember our old issues?), so adding it as an attribute of the class solved the problem.

you load custom mechanisms using this line:
https://github.com/jonescompneurolab/hnn-core/blob/a1cc05d9146ebab3bf9623cb9611b722fee55c63/hnn_core/network_builder.py#L159
.

Sure, but how does test_cell.test_artificial_cell() call network_builder.load_custom_mechanisms()?
def test_artificial_cell():
    """Test artificial cell object."""
    event_times = [1, 2, 3]
    threshold = 0.0
    artificial_cell = _ArtificialCell(event_times, threshold)
    assert artificial_cell.nrn_eventvec.to_python() == event_times
    # the h.VecStim() object defined in vecevent.mod should contain a 'play()'
    # method
    assert hasattr(artificial_cell.nrn_vecstim, 'play')
    # the h.Netcon() instance should reference the h.VecStim() instance
    assert artificial_cell.nrn_netcon.pre() == artificial_cell.nrn_vecstim
    assert artificial_cell.nrn_netcon.threshold == threshold

The only reason I care is because if I start a new ipython kernel, import the appropriate modules, and try instantiating an _ArtificialCell() object out of the blue, an error is produced because the custom mechanisms weren't loaded.

By the way, I encountered the same error today while working on #201 . In my case, it was a bit of red herring. The actual problem was that some Neuron variable was being deleted outside the scope of the class (remember our old issues?), so adding it as an attribute of the class solved the problem.

I was wondering about that. The trick is going to be tracing the creation/destruction of references to nrn objects....

Sure, but how does test_cell.test_artificial_cell() call network_builder.load_custom_mechanisms()?

humm indeed, this looks like something that needs to be fixed. Maybe call load_custom_mechanisms at the start of the test?

Never thought I'd see green on this PR. Everything seems to be functioning well on my end, let me know if there's any remaining changes you guys think are necessary. If they're not pressing they can be added to-do list for the current recording PR.

@ntolley looks promising!
I left some minor comments which can be addressed easily I believe. Also when the CIs turn green, can you ensure that all 3 CIs are green. For some reason Travis didn't run on your last push. Not sure why, it can sometimes happen. Just try pushing a new commit and it might just work. Hope to merge it in a couple of days :) There is more work to do, particularly with respect to standardizing the different kinds of recordings (currents, dipoles etc) but we can keep it for future PRs! Thanks for all the hard work so far.

You will need to rebase @ntolley since the MPI PR is now merged

Finished adding tests for the dimensions of that vsoma data. The MPI fix seems to be running flawlessly, thanks for all the hard work @blakecaldwell!

Thanks a bunch @ntolley and for your patience ! 🍻 🍺
let's keep the ball rolling in #199


Running “full” OS with Lithos

Hi,
it seems that Lithos is designed for running services that may be isolated using namespaces, cgroups, capabilities(?), i.e. running them inside “containers.” I wonder, can I use it even to run ”full OS” (Alpine Linux to be specific) with traditional init system (OpenRC) and multiple services, including OpenSSH server for users to connect into? Like with LXC that I currently use. Theoretically it should be possible, but it seems that it’s really not designed for such use case (?), so are there any limitations or design decisions that makes it really bad idea?

Hi!
Technically you can. There are a few quirks, though:

We don't have network isolation yet. We have a short-term plan for basic support of handing network namespace to the container (and setting up it with some scripts outside of the scope of lithos). Also, we're looking towards supporting CNI plugins, but I'm not sure I'll have time to try it anytime soon.

Lithos runs its own process (lithos_knot) as pid1 in container. This helps applications, but it may be a problem for systemd, or any other init. I'm not sure how easy to fix this one (because we use lithos_knot process' command-line to find out which processes are running on restart, this mechanism needs to be changed to run a full-featured init).

It's probably super-tedious to setup. In docker or lxc you have everything writable by default. In lithos, you have nothing writable by default. And you need to add every tiny little directory that needs to be writable to lithos config (and presumably put it outside of the container image). Even if it's just to track ntpd state. Many utilities run as root are trying to write to /etc. There are probably much more quirks for systemd (like setting container_uuid environment variable). And maybe a little bit less for openrc. Adding overlayfs making filesystem kinda writable isn't too hard, though.

We also don't have capabilities support (we want to have, though). So the recommended way is not to run as root in container. Or to use user namespaces, which have it's own root with somewhat non-superuser capabilities. Latter may be problem for systemd/openrc too.

All in all, we can add some features incrementally. But I don't see a reason to make full OS a first-class citizen for lithos. There's neither space where we can improve a lot (any specific issues with LXC/LXD?) nor there are lots of use cases for such containerization in the wild I think.
(on the other if someone reading this wants to use underlying libmount and unshare crates for running full OS images I'm very enthusiastic to this idea, so let me know if you need any help)

Thanks for very thorough answer.
Well, LXC works, but I don’t like that project. LXC is 42k LoC in C, that’s really massive for something that can be (basically) written even in dozens lines of shell with few tiny, single-purpose utilities like unshare, ip etc. Also it has very cryptic error messages, so when something get wrong, it’s quite hard to find the cause.
I’d like to find same sane, lightweight alternative, preferably written in some decent language made for system programming (I mean not in Go…). Vagga and Lithos looks very promising.
I have both use cases. I need to run isolated services, but also some full containers that I can give to users, so they can manage some their services inside.

I need to run isolated services, but also some full containers that I can give to users, so they can manage some their services inside.

Well, technically we have a concept of sandboxes just for that. I.e. you allow users to upload images to certain directory configured in a sandbox. And arrange some script to update their /etc/lithos/processes/<sandbox-name>.yaml. This allows them to freely update their images, add and remove services, but they can't escape a sandbox (basically a set of directories they are allowed to see, and a set of user ids they are allowed to use, etc.).
To be perfectly clear: it doesn't help to run full OS. But it does help to manage services without running a full OS. It's easy to run sshd as a service in the lithos, if you want users to be able to log in for some simple tasks like looking system stats or grepping logs.
For generating processes yaml without allowing users to access host system you may use consul-template or confd, or some wrapper around whatever provisioning system you have (e.g. ansible)
(we're developing a verwalter for this purpose, but it still lacks good auth at the moment)

@jirutka, any questions left here?

No questions left, thank you for the answer! We can close this now.


Make channel log level settable from output view

This PR closes #162262

Choosing the option takes you directly to the second quickpick presented by the existing Developer: Set Log Level... command:

Like all entries on this overflow menu the user can promote the new option to be a button:

For channels that don't support log levels the option gets disabled rather than hidden, so it matches the behaviour of existing overflow options, and so when shown as a button the layout doesn't change as you scroll through channels:

/assign @sandy081
Pinging @mjbvz as author of the related issue. This PR doesn't achieve all that was requested, but I hope it helps.

@gjsjohnmurray Thanks for contributing the PR. I liked the new behaviour. I have one feedback

I would prefer to open the log level options inline - meaning - next to the configure log level action. It can be a menu or sub menu based on the location of the action.

@sandy081 as a (sub)menu I don't think we can achieve both functions of the quickpick, in which you can set the log level for the channel until the next restart and the default level that will apply after restarts (the double checkmark on the right-hand end of a level).
Assuming the (sub)menu item picked will get a single checkmark and set the level only for the current session, maybe the initial button / overflow menu item should be captioned "Set Current Log Level..." rather than how they appear in my previous screenshots ("Configure Log Level...").

That's right, we would need two actions

Set Log Level (Session or Window)
Set Log Level (Default)

I would make first action that can be able to made a primary action and second action just a secondary action.

@sandy081 this is ready for your review. It now uses a submenu, but instead of adding a second submenu for setting the channel's default level I added an option at the end of the submenu which updates the default to match the current choice.

This option is only enabled when the default differs from the current level.

As with other action buttons, this one can be hidden, causing the option to move to the ... menu.


Update self-shadowing function for direct lighting

Description
This PR updates the geometric shadowing function used for direct lighting in physically-based rendering.
Our existing code was using a Schlick-Smith approximation. The limitations of this approximation have been explained by Eric Heitz in a 2014 paper. This PR updates the approach to follow the recommendations of the paper, using code similar to the glTF Sample Viewer.
The effect of the change is subtle, but it makes specular reflections slightly brighter on models with medium roughness.
Issue number and link
Resolves https://github.com/CesiumGS/cesium/issues/12010.
Testing plan
Load the glTF PBR Sandcastle, and select the lighting option "Direct Lighting Only". See especially the "Metal-Roughness Spheres" model. Compared to the same Sandcastle in main, the specular reflection should be slightly brighter and broader in this branch.
Author checklist

[x] I have submitted a Contributor License Agreement
[x] I have added my name to CONTRIBUTORS.md
[x] I have updated CHANGES.md with a short summary of my change
[ ] I have added or updated unit tests to ensure consistent code coverage
[x] I have updated the inline documentation, and included code examples where relevant
[x] I have performed a self-review of my code

I have added or updated unit tests to ensure consistent code coverage

I understand why unit test may not be an ideal way to test this, but is there any kind of test we can put in place to avoid regressions?

But overall the code looks good, and I can detect a slight change from main.

is there any kind of test we can put in place to avoid regressions?

@ggetz as discussed, unit testing is probably not the right choice here. Screenshot comparisons would work better.
These lighting effects can be pretty subtle. We will need to add some end-to-end testing with very tight tolerances to catch any changes or regressions. But this would be out of the scope of this PR. I opened https://github.com/CesiumGS/cesium/issues/12065 to discuss further.

Great, thanks @jjhembd!


UI - Not able to delete assistants from Explore option.

Electron build with commit - e8c58e5aca2
Steps to reproduce the problem:

Create an assistant.
From "Explore" view, select this assistant.
Use "Delete" option to delete the assistant.

Notice that nothing happens on clicking the "delete" option.
Assistant continues to be listed in "Home" and "Explore" view and is usable.
Expected Behavior:
Assistant should get deleted.

@sangee2004 This should now be addressed.


Create test for papi-wrap

Based on review in https://github.com/eth-cscs/production/pull/682#pullrequestreview-137265527
Possible starting point mentioned in the review:
https://madra.cscs.ch/piccinal/perfhackaton/issues/14

done: https://github.com/eth-cscs/hpctools/blob/master/reframechecks/papi/papi_wrap.py


Added support for most syscalls, and framework for interoperability.

Stack:

#2182
#2181 ⬅

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is

crates/cairo-lang-runner/src/casm_run/mod.rs line 316 at r1 (raw file):

                        if !addr_domain.is_zero() {
                            // Only address_domain 0 is currently supported.
                            return Ok(false);

what is this bool?
Code quote:
false

crates/cairo-lang-runner/src/casm_run/mod.rs line 316 at r1 (raw file):
Previously, ilyalesokhin-starkware wrote…
what is this bool?

I'm probably missing something, but it looks like you are returning a bool in a function that returns a uint.


ignore ActiveRecord models that are abstract_class=true

ActiveRecord models that are marked [abstract] (http://railsapi.com/doc/rails-v3.0.8rc1/classes/ActiveRecord/Base.html#M005059), such as:
class Part < ActiveRecord::Base
  self.abstract_class = true
  ...
end

class AutoPart < Part
end

...cause RailsAdmin to throw an exception when visiting /admin (full stack here)
ActiveRecord::StatementInvalid (PGError: ERROR:  relation "" does not exist
LINE 4:              WHERE a.attrelid = '""'::regclass
                                        ^
:             SELECT a.attname, format_type(a.atttypid, a.atttypmod), d.adsrc, a.attnotnull
              FROM pg_attribute a LEFT JOIN pg_attrdef d
                ON a.attrelid = d.adrelid AND a.attnum = d.adnum
             WHERE a.attrelid = '""'::regclass
               AND a.attnum > 0 AND NOT a.attisdropped
             ORDER BY a.attnum
):

The patch just skips over models that are abstract.

Moved to AbstractModel.lookup and updated the tests.
It took me 3 reads of the original expression to understand it, so I simplified it.

Thanks!


Upgrade version maps should be available offline

Version maps specify which versions are supported from which operator. The source of truth is master but we need a local copy built into operator for offline upgrade cases.

https://github.com/istio/operator/pull/395, seems like we can load it from VFS now.

We can but we don't use it yet. We should be able to fall back if we cannot get it from github.


Godot LineEdit Autocompletion for Custom Game Commands

not sure if possible, but would like autocomplete for the commands in game to make the experience more playable.
in addons folder there is a custom line edit autocompletion node I downloaded from https://godotengine.org/asset-library/asset/3490
see: https://github.com/EsotericShadow/hacker_game_v1/blob/main/hacker_game_v1.0/addons/auto_complete_menu_node/line-edit-complete-godot-1b2123827600f01b01ea6eb851153ac092eacee7/README.md
still figuring it out, any advice appreciated

The line edit autocompletion for in-game commands is now fully implemented in HackerGame v1.1. Using the Auto Complete Menu Node Plugin by Lenrow, I successfully added dynamic autocompletion for commands, target IDs, techniques, and vendor items.
What Was Done
Installed & Configured Plugin: Integrated the Auto Complete Menu Node Plugin from the Godot Asset Library and connected it to the game's terminal input field.
Populated Terms File: Created a terms.json file to list all possible command inputs, ensuring smooth autocompletion for every valid input, including commands like help, status, vendor.buy VPN, and all target IDs.
Tested & Refined: Debugged and tested command suggestions to ensure accurate and consistent autocompletion.
This update makes the game significantly more accessible and user-friendly, allowing players to easily discover valid commands without needing prior knowledge. Thanks to Lenrow for the Auto Complete Menu Node Plugin, which made this possible.
The issue is now resolved and reflected in HackerGame v1.1.


Usage with stripe connect

I was reading documentation about connect (store-builder recipe) and saw mention that I could use Elements or Checkout for collecting the payment details.
So ended up trying to set this up but run into a small hiccup... if I understood correctly I should be creating the token with connected user’s API key which means I should be changing the apiKey to StripeProvider based on the seller/user. While checking on how to do this I found this:
https://github.com/stripe/react-stripe-elements/blob/master/src/components/Provider.js#L45
Does this mean that this is not supported? or is the some better way of doing this?

Hi! Wouldn't you know the API key you want to use at the Provider's render time? Do you need to switch out the API key dynamically?

The issue is that for each API key, we need to create a new instance of stripe.

That's true...I was wrapping my root component with stripeProvider without thinking too much but I guess makes more sense to just wrap the view where the payment happens.
Thanks 👍

@tume also, if you're using tokens instead of sources, you should just be using your platform's publishable key :).

I ended up bumping into this problem as well as I was dynamically changing out the publishable key based on which store location was accepting the order.
In order to completely re-create the stripeProvider component and avoid the error, simply pass a unique key prop to the provider component
   <StripeProvider key={stripeSettings.publishable_key} apiKey={stripeSettings.publishable_key></StripeProvider>


Piechart

I created a "Pie chart widget" a while ago, maybe you want this (demo is in _example).
I'm not a go veteran, so I'd appreciate feedback.
Best

This is a really nice implementation! I personally like the way you define functions in the middle of your code, a cool idea and not something I use enough. I merged the commit and cleaned up a few things here, but it was mostly minor stuff. Everything else looks good. Thanks for the PR!

Thanks for the nice feedback :)


flutter pub outdated throws: pub finished with exit code 69

Steps to Reproduce

flutter pub outdated throws: pub finished with exit code 69
Expected results:
Actual results:

Code sample

Logs

[✓] Flutter (Channel stable, 2.10.1, on macOS 12.2.1 21D62 darwin-x64, locale zh-Hans-CN)
    • Flutter version 2.10.1 at /Users/ying/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision db747aa133 (3 days ago), 2022-02-09 13:57:35 -0600
    • Engine revision ab46186b24
    • Dart version 2.16.1
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/ying/Library/Android/sdk
    • Platform android-31, build-tools 30.0.2
    • ANDROID_SDK_ROOT = /Users/ying/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)

[☠] IntelliJ IDEA Ultimate Edition (the doctor check crashed)
    ✗ Due to an error, the doctor check did not complete. If the error message below is not helpful, please let us know about this issue at https://github.com/flutter/flutter/issues.
    ✗ FormatException: Unexpected extension byte (at offset 5)
    • #0      _Utf8Decoder.convertSingle (dart:convert-patch/convert_patch.dart:1789:7)
      #1      Utf8Decoder.convert (dart:convert/utf.dart:351:42)
      #2      InputStream.readString (package:archive/src/util/input_stream.dart:207:30)
      #3      new ZipDirectory.read (package:archive/src/zip/zip_directory.dart:40:30)
      #4      ZipDecoder.decodeBuffer (package:archive/src/zip_decoder.dart:19:30)
      #5      ZipDecoder.decodeBytes (package:archive/src/zip_decoder.dart:14:12)
      #6      IntelliJPlugins._findPluginXml (package:flutter_tools/src/intellij/intellij.dart:130:44)
      #7      IntelliJPlugins._readPackageVersion (package:flutter_tools/src/intellij/intellij.dart:141:40)
      #8      IntelliJPlugins.validatePackage (package:flutter_tools/src/intellij/intellij.dart:63:35)
      #9      IntelliJValidator.validate (package:flutter_tools/src/intellij/intellij_validator.dart:103:15)
      #10     asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:111:32)
      #11     asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:109:18)
      #12     _rootRun (dart:async/zone.dart:1426:13)
      #13     _CustomZone.run (dart:async/zone.dart:1328:19)
      #14     _runZoned (dart:async/zone.dart:1861:10)
      #15     runZonedGuarded (dart:async/zone.dart:1849:12)
      #16     runZoned (dart:async/zone.dart:1780:12)
      #17     asyncGuard (package:flutter_tools/src/base/async_guard.dart:109:3)
      #18     Doctor.startValidatorTasks (package:flutter_tools/src/doctor.dart:205:9)
      #19     Doctor.diagnose (package:flutter_tools/src/doctor.dart:309:47)
      #20     DoctorCommand.runCommand (package:flutter_tools/src/commands/doctor.dart:50:48)
      #21     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1320:12)
      <asynchronous suspension>
      #22     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1161:27)
      <asynchronous suspension>
      #23     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
      <asynchronous suspension>
      #24     CommandRunner.runCommand (package:args/command_runner.dart:209:13)
      <asynchronous suspension>
      #25     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
      <asynchronous suspension>
      #26     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
      <asynchronous suspension>
      #27     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
      <asynchronous suspension>
      #28     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
      <asynchronous suspension>
      #29     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
      <asynchronous suspension>
      #30     main (package:flutter_tools/executable.dart:94:3)
      <asynchronous suspension>

[✓] VS Code (version 1.63.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[!] Proxy Configuration
    • HTTP_PROXY is set
    ! NO_PROXY is not set

[✓] Connected device (5 available)
    • MI 9 (mobile)        • a423bb1a                             • android-arm64  • Android 11 (API 30)
    • WY's iPhone (mobile) • 00008020-000A1D110A32002E            • ios            • iOS 15.3.1 19D52
    • iPhone 13 (mobile)   • BA8AFB07-A790-407C-AED9-36F4F447D775 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-2 (simulator)
    • macOS (desktop)      • macos                                • darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)         • chrome                               • web-javascript • Google Chrome 98.0.4758.80
    ! Error: WY's iPhone is busy: Preparing the watch for development via WY's iPhone. Xcode will continue when WY's iPhone is finished. (code -10)

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 2 categories.

Steps to Reproduce

flutter pub outdated throws: pub finished with exit code 69
Expected results:
Actual results:

Code sample

Logs

[✓] Flutter (Channel stable, 2.10.1, on macOS 12.2.1 21D62 darwin-x64, locale zh-Hans-CN)
    • Flutter version 2.10.1 at /Users/ying/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision db747aa133 (3 days ago), 2022-02-09 13:57:35 -0600
    • Engine revision ab46186b24
    • Dart version 2.16.1
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/ying/Library/Android/sdk
    • Platform android-31, build-tools 30.0.2
    • ANDROID_SDK_ROOT = /Users/ying/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)

[☠] IntelliJ IDEA Ultimate Edition (the doctor check crashed)
    ✗ Due to an error, the doctor check did not complete. If the error message below is not helpful, please let us know about this issue at https://github.com/flutter/flutter/issues.
    ✗ FormatException: Unexpected extension byte (at offset 5)
    • #0      _Utf8Decoder.convertSingle (dart:convert-patch/convert_patch.dart:1789:7)
      #1      Utf8Decoder.convert (dart:convert/utf.dart:351:42)
      #2      InputStream.readString (package:archive/src/util/input_stream.dart:207:30)
      #3      new ZipDirectory.read (package:archive/src/zip/zip_directory.dart:40:30)
      #4      ZipDecoder.decodeBuffer (package:archive/src/zip_decoder.dart:19:30)
      #5      ZipDecoder.decodeBytes (package:archive/src/zip_decoder.dart:14:12)
      #6      IntelliJPlugins._findPluginXml (package:flutter_tools/src/intellij/intellij.dart:130:44)
      #7      IntelliJPlugins._readPackageVersion (package:flutter_tools/src/intellij/intellij.dart:141:40)
      #8      IntelliJPlugins.validatePackage (package:flutter_tools/src/intellij/intellij.dart:63:35)
      #9      IntelliJValidator.validate (package:flutter_tools/src/intellij/intellij_validator.dart:103:15)
      #10     asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:111:32)
      #11     asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:109:18)
      #12     _rootRun (dart:async/zone.dart:1426:13)
      #13     _CustomZone.run (dart:async/zone.dart:1328:19)
      #14     _runZoned (dart:async/zone.dart:1861:10)
      #15     runZonedGuarded (dart:async/zone.dart:1849:12)
      #16     runZoned (dart:async/zone.dart:1780:12)
      #17     asyncGuard (package:flutter_tools/src/base/async_guard.dart:109:3)
      #18     Doctor.startValidatorTasks (package:flutter_tools/src/doctor.dart:205:9)
      #19     Doctor.diagnose (package:flutter_tools/src/doctor.dart:309:47)
      #20     DoctorCommand.runCommand (package:flutter_tools/src/commands/doctor.dart:50:48)
      #21     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1320:12)
      <asynchronous suspension>
      #22     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1161:27)
      <asynchronous suspension>
      #23     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
      <asynchronous suspension>
      #24     CommandRunner.runCommand (package:args/command_runner.dart:209:13)
      <asynchronous suspension>
      #25     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
      <asynchronous suspension>
      #26     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
      <asynchronous suspension>
      #27     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
      <asynchronous suspension>
      #28     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
      <asynchronous suspension>
      #29     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
      <asynchronous suspension>
      #30     main (package:flutter_tools/executable.dart:94:3)
      <asynchronous suspension>

[✓] VS Code (version 1.63.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[!] Proxy Configuration
    • HTTP_PROXY is set
    ! NO_PROXY is not set

[✓] Connected device (5 available)
    • MI 9 (mobile)        • a423bb1a                             • android-arm64  • Android 11 (API 30)
    • WY's iPhone (mobile) • 00008020-000A1D110A32002E            • ios            • iOS 15.3.1 19D52
    • iPhone 13 (mobile)   • BA8AFB07-A790-407C-AED9-36F4F447D775 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-2 (simulator)
    • macOS (desktop)      • macos                                • darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)         • chrome                               • web-javascript • Google Chrome 98.0.4758.80
    ! Error: WY's iPhone is busy: Preparing the watch for development via WY's iPhone. Xcode will continue when WY's iPhone is finished. (code -10)

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 2 categories.

@realkimmy
Can you provide the entire stacktrace that shows the error pub finished with exit code 69 ?
I just tried the same and didn't replicate it for me:
dhs@Dhss-MacBook-Pro testdemo % flutter pub outdated
Showing outdated packages.
[*] indicates versions that are not the latest available.

Package Name                Current  Upgradable  Resolvable  Latest

direct dependencies:
cupertino_icons             *1.0.2   1.0.4       1.0.4       1.0.4
shared_preferences          *2.0.12  2.0.13      2.0.13      2.0.13
url_launcher                *6.0.18  6.0.20      6.0.20      6.0.20

transitive dependencies:
shared_preferences_android  *2.0.10  2.0.11      2.0.11      2.0.11
shared_preferences_ios      *2.0.9   2.0.10      2.0.10      2.0.10
shared_preferences_linux    *2.0.4   2.1.0       2.1.0       2.1.0
shared_preferences_macos    *2.0.2   2.0.3       2.0.3       2.0.3
shared_preferences_windows  *2.0.4   2.1.0       2.1.0       2.1.0
url_launcher_android        *6.0.14  6.0.15      6.0.15      6.0.15
url_launcher_ios            *6.0.14  6.0.15      6.0.15      6.0.15
url_launcher_linux          *2.0.3   3.0.0       3.0.0       3.0.0
url_launcher_macos          *2.0.3   3.0.0       3.0.0       3.0.0
url_launcher_web            *2.0.6   2.0.8       2.0.8       2.0.8
url_launcher_windows        *2.0.2   3.0.0       3.0.0       3.0.0
win32                       *2.3.8   2.4.0       2.4.0       2.4.0
xdg_directories             *0.2.0   0.2.0+1     0.2.0+1     0.2.0+1

16 upgradable dependencies are locked (in pubspec.lock) to older versions.
To update these dependencies, use `flutter pub upgrade`.

Also check or try on a different network provider / connection.

This problem is caused by my proxy


Performace info for a LOT of queries on a page?

Would be good to know its limits.  How far can it go as a general guideline, what browsers maybe can go further, does IE tank earlier than others and all that.

Care about generating this information?

?
Asking for help on it?

Yes :p


ML prediction for DAC combination therapies

[x] Build knowledge graphs
[ ] Define ML tasks
Analysis:
https://github.com/GilbertLabUCSF/Decitabine-treatment/blob/main/DAC_combo.ipynb

Related tools:

https://github.com/mims-harvard/TxGNN
https://github.com/mims-harvard/TDC/blob/master/tutorials/TDC_104_ML_Model_DeepPurpose.ipynb

Igor F Tsigelny, Artificial intelligence in drug combination therapy, Briefings in Bioinformatics, 2019, https://doi.org/10.1093/bib/bby004
Jafari, M., Mirzaie, M., Bao, J. et al. Bipartite network models to design combination therapies in acute myeloid leukaemia. Nat Commun 13, 2128 (2022). https://doi.org/10.1038/s41467-022-29793-5

https://github.com/GilbertLabUCSF/Decitabine-treatment/commit/40988f9898884433875bb63cdf14d699088d6253

I think the evidence I found regarding current combinations of DNMT + DHODH inhibitors can be a proof of concept for this drug treatment model using multi-omics readouts. Thus, it might be useful to think about it for defining new tasks.

Related to https://github.com/mims-harvard/TDC/pull/207 and https://github.com/mims-harvard/PrimeKG/pull/12

https://tdcommons.ai/benchmark/drugcombo_group/overview/


docs: typo on Snowflake provider page

Small typo fix on Snowflake provider page.

please reopen when ready


Bun edition

See bun branch

[X] Initial prototype
[ ] Fixing

[ ] Firefox IPC is broken

[ ] Bun examples
[ ] Publish somewhere?

MacOS Ventura.
Browsers installed: chrome, brave, safari
Bun versions tested: 0.5.7, 0.5.8


Added verida custom button

Added verida custom button,  so that developers can pass and id verida-button to an html input tag

resolves #65

I'm not able to give any useful feedback on this, sorry. I don't know TS or how the library works.
Happy to verify it works if there is a test harness, but beyond that I don't think I'm useful.
@tahpot will need to review.

@cmcWebCode40
Any reason addVeridaButtonImage() is called three times?
I would have thought the image would be injected once when the page loads?

@cmcWebCode40
Any reason addVeridaButtonImage() is called three times?
I would have thought the image would be injected once when the page loads?

ok, I noticed that the verida button image gets removed from the DOM whenever the modal is closed.
what way I can resolve this to call the function once?

addVeridaButtonImage()

@tahpot  here is a short demo ,
https://www.loom.com/share/fb97130c9cfc4018acf56510a9785e9f
i called the function once on-load ,but whenever i open and close the QR modal the button unmount , that was why i called the addVeridaButtonImage() , when the modal is clicked .

As discussed, let's undo this and just provide a static image asset for developers to copy and past and include that in the docs.

As discussed, let's undo this and just provide a static image asset for developers to copy and paste and include that in the docs.

@taphot, this PR has been updated with the requested changes.

@tahpot , thanks
changes requests noted
i will resolve them

@cmcWebCode40 do you know what the status of this is?

@cmcWebCode40 @tahpot Does this need merging?

Looks like this is redundant.


The example in the PDF uses an internal URL

The example should be moved to a regular Bluemix URL

After you push the sample as one bluemix app, and bind one PA service instance to it, the URL will be obtained from OS env automatically.


Must restart Ferdium to have click links to work

Archlinux with Dusk Window Manager (DWM deriviate)
When i start my system i have Ferdium autostarted along with other programs.
1st time when i click links from Gmail or Facebook nothing happens.
When i quit Ferdium and start it again clicking those links work again and are opened in my default Browser Firefox.

Solved it: After  sudo pacman -S kde-gtk-config it works.


Support for different prompt types

Have added support for date, email, number, and tel prompts in addition to the original password prompt.
Had to make it so that setSelectionRange would only happen if it was a password, text, or url input field otherwise it errors out.
Probably better ways to do some of it, but it works for my use case on a Honeywell CT60 Android barcode scanner.

Coverage increased (+0.2%) to 89.744% when pulling<PHONE_NUMBER>87ea098f51c8955efbb2e6f8a8343b on ray73864:master into 65f558632bf01811da7ca96c96295d1e1f1e9c8c on coderaiser:master.

Could you please run npm run fix:lint to fix lint errors and provide test for this code :). Is this code works on desktop browsers?

Should work fine on desktop browsers too, the only difference with desktop browsers is they won't see a customised on-screen keyboard but it will still limit what can be put into the prompt input field.
As for tests, I have no clue how to do them, and looking at the test/smalltalk.js file, it's kinda way over my head :(


Pluralization of properties that are lists

This issue is in particular about two properties in Collections, but speaks to a general best practice that should be used throughout the spec.
In collections, the extent object has a properties bbox and interval. The singular nature of those property names indicates that there is a single bbox and interval; however, the properties are a lists of bboxes and a list of intervals, respectively.
I propose that any property that has a list value should have a pluralized name. So this:
  "extent": {
    "spatial": {
      "bbox": [
        [
          39.28919876472999,
          -5.878778696206506,
          39.356865475223195,
          -5.722212794937691
        ]
      ]
    },
    "temporal": {
      "interval": [
        [
          "2016-08-28T00:00:00Z",
          null
        ]
      ]
    }
  }

should actually be:
  "extent": {
    "spatial": {
      "bboxes": [
        [
          39.28919876472999,
          -5.878778696206506,
          39.356865475223195,
          -5.722212794937691
        ]
      ]
    },
    "temporal": {
      "intervals": [
        [
          "2016-08-28T00:00:00Z",
          null
        ]
      ]
    }
  }

We inherit your example from WFS. We can't change it or we would loose WFS compatibility. Therefore I'm strongly in favor of not applying the proposed changes.

@m-mohr that makes sense, thanks for the clarification - I agree if that's what WFS wants, that's what we'll need to keep.
For something like the label:overview property mentioned in #584, would it make sense to modify the pluralization as this is not part of WFS? In general, would it make sense to keep pluralization consistent unless it breaks from WFS?

We probably could have suggested a change to WFS, but they've just locked in 1.0.0, so we're probably set with it.
I'm cool with keeping pluralization consistent unless it breaks from WFS.

Agreed. Are there more places to change except for the label extension?

Should we change it to be pluralized before the 0.8 release?

I would be for changing it to be consistent before 0.8.0 final release unless it breaks from WFS.

Me too. Let me try to fire off a PR today.


Students'answers pages : Edition vs Read mode

Increasing the Header's differences between read mode and edition mode on students'answers pages.
Choosing a icon more meaningful
On Edition Mode :
Changing the Save Icon for a Pencil Off icon

Changing the Page's title from : "Category title" to "Category title : Edition". For example : "Matières et produits : Édition"

Done


Update tsmn_water_quality_saturday.yaml

changed the time interval to ongoing

Dear @chorng cc: @Schpidi @gmilcinski
i went trough the yaml files in public collections, and I was updating the time intervals. During this change I noticed that the yaml file has an edc browser link, and also the type and collection ID and endpoint refer to old informations.
EDCBrowser: https://browser.eurodatacube.com/?zoom=7&lat=43.14975&lng=7.16013&collectionId=tsmn_water_quality_saturday&layerId=Total Suspended Matter Map&type=sentinel-hub-edc&fromTime=2020-04-17T00%3A00%3A00.000Z&toTime=2021-04-17T00%3A00%3A00.000Z
and
EndPoint: https://shservices.mundiwebservices.com
Type: byoc-45ce0fb2-fdaf-481e-b834-f728a8677e59
CollectionId: 45ce0fb2-fdaf-481e-b834-f728a8677e59
what i see is that even if the EDC browser link works the collection ID and type refer the old collections IDs(the one on mundi and not on the new ones that have been migrated.
This is valid for almost all the collections we handle. Is there a way for you to update these fields and check if the EDC Browser link is pointing to the right layer name?
the other collections that need an update of the mentioned fields are:

2mt_2020_monthly_average_from_cds.yaml
cams_glc_2017.yaml
chl_water_quality_saturday.yaml
e12c_motorway.yaml
e12d_primary_corrected.yaml
iceye_e11.yaml
iceye_e11a.yaml
iceye_e13b.yaml
iceye_e3.yaml
jaxa_wq_chla.yaml
jaxa_wq_tsm.yaml
population_density.yaml
s5p-no2-tropno-daily-check.yaml
tsmn_water_quality_saturday.yaml
wind_10m_u.yaml
wind_10m_v.yaml

Thanks for all the support you could provide
kr
Alessandro

Dear @AlessandroScremin ,
There is a pull request which updates collection ids. Could you take a look at this PR and confirm that it does what you requested.
BR,
Chung

Dear @chorng
Thanks a lot...it is what I needed. Very appreviated
Alessanadro

Of the above mentioned collections almost all got update in this PR: https://github.com/eurodatacube/public-collections/pull/189
@chorng Can you please check what is with cams_glc_2017.yaml as this one didn't get updated and is still on mundi.

@dthiex Max and I ingested the cams_glc_2017 and Jaxa-alos2-Activity-cars-containers collections. I updated the cams_glc_2017.yaml with PR 192. Note that Jaxa-alos2-Activity-cars-containers is not included in the edc public collection atm.

I believe this was duly  approved


DOC: building: specify .ps1 for windows example

The docs have the pwsh line executing the activate file instead of the .ps1 file.
$ sed -n '331,355p' scipy/doc/source/building/index.rst
    .. tab-set::

      .. tab-item:: Linux
        :sync: linux

        ::

          python -m venv venv
          source venv/bin/activate

      .. tab-item:: macOS
        :sync: macos

        ::

          python -m venv venv
          source venv/bin/activate

      .. tab-item:: Windows
        :sync: windows

        ::

          python -m venv venv
          .\venv\Scripts\activate
$

doc/source/building/index.rst
-           .\venv\Scripts\activate
+           .\venv\Scripts\Activate.ps1

as per recent output from pwsh on Windows
PS C:\Users\bird> python.exe --version
Python 3.12.6
PS C:\Users\bird> python.exe -m venv venv
PS C:\Users\bird> Get-ChildItem venv\Scripts

    Directory: C:\Users\bird\venv\Scripts

Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
-a---           9/17/2024  5:27 PM           2034 activate
-a---           9/17/2024  5:27 PM            990 activate.bat
-a---           9/17/2024  5:27 PM          26199 Activate.ps1
-a---           9/17/2024  5:27 PM            393 deactivate.bat
-a---           9/17/2024  5:27 PM         108399 pip.exe
-a---           9/17/2024  5:27 PM         108399 pip3.12.exe
-a---           9/17/2024  5:27 PM         108399 pip3.exe
-a---           9/17/2024  5:27 PM         270104 python.exe
-a---           9/17/2024  5:27 PM         258840 pythonw.exe

PS C:\Users\bird>

Peripheral to the foregoing is noted PowerShell isn't to date a Windows only program.
And the docs aren't specific when commands are provided as to Command Prompt or PowerShell.
The pyproject.toml file being updated with some frequency, the following might be considered to implement as being more informative; uppercasing set (though cmd.exe is case-insensitive) may emphasise it as a Command Prompt "internal command".
- # do this instead (which will override this setting)
- # set CIBW_ENVIRONMENT_WINDOWS=PKG_CONFIG_PATH=PWD.replace('\\', '/')
+ # in cmd.exe do this instead (which will override this setting)
+ # SET CIBW_ENVIRONMENT_WINDOWS=PKG_CONFIG_PATH=PWD.replace('\\', '/')

@lucascolley
cf: How venvs work

@vsajip
https://github.com/python/cpython/commit/1a6bacb31f7b49c244a6cc3ff0fa7f71a82412ef
[doc] Refresh the venv introduction documentation, and correct the st…
…atement about VIRTUAL_ENV (GH-98350)
pelson and CAM-Gerlach committed on Oct 19, 2022
cpython/Doc/library/venv.rst
suggested update (below) changes PowerShell to pwsh
and (as rendered) for pwsh adds POSIX ./  and Windows .\
+-------------+----------+-----------------------------------------------------+
| Platform    | Shell    | Command to activate virtual environment             |
+=============+==========+=====================================================+
| POSIX       | bash/zsh | :samp:`$ source {<venv>}/bin/activate`              |
|             +----------+-----------------------------------------------------+
|             | fish     | :samp:`$ source {<venv>}/bin/activate.fish`         |
|             +----------+-----------------------------------------------------+
|             | csh/tcsh | :samp:`$ source {<venv>}/bin/activate.csh`          |
|             +----------+-----------------------------------------------------+
|             | pwsh     | :samp:`$ ./{<venv>}/bin/Activate.ps1`               |
+-------------+----------+-----------------------------------------------------+
| Windows     | cmd.exe  | :samp:`C:\\> {<venv>}\\Scripts\\activate.bat`       |
|             +----------+-----------------------------------------------------+
|             | pwsh.exe | :samp:`PS C:\\> .\\{<venv>}\\Scripts\\Activate.ps1` |
+-------------+----------+-----------------------------------------------------+

Feel free to send a PR @musvaage , following CPython sounds like a good idea to me 👍

following CPython

this should be the fix
-           .\venv\Scripts\activate
+           venv\Scripts\Activate.ps1

cf: How venvs work

update to table corrected in previous comment
and (as rendered) for pwsh adds POSIX ./ and Windows .\
POSIX
to execute example.ps1 in the local directory type the following
./example.ps1
the ./ isn't used when executing a .ps1 file in another directory
/tmp/example.ps1

The table over in the CPython repo has been modified using the executable name only for POSIX.
cf: https://github.com/python/cpython/pull/124315#issuecomment-2383794430
It'd be more convenient for packagers if activate.sh replaced activate as a file name allocating for a transition period.


Stylelint no-empty-source

Rails version: Rails 5.2.0
Ruby version: ruby 2.5.1p57 (2018-03-29 revision 63029) [x86_64-darwin17]
Node version: v10.5.0
Clone for references

Thanks for the help 😊

Thanks for the report, @mcdave029.
Any thoughts on this, @Ruffeng?

@mcdave029 seems it's a rule from stylelint https://stylelint.io/user-guide/rules/no-empty-source/ . However, it doesn't seem to respect the single line compact format, the error is correct but should show the name of the file. Because it's multiline, it doesn't match the regex to know what's going on there.
I'll have a look about the output

I could reproduce the error. Didn't notice you are showing as well the output of the stylelint at the bottom of the picture.
I'll investigate as soon as i am available :)

@mcdave029 This is the app you provided me to reproduce the error:

In the default.yml, I assumed you are using npm server, and I specified there the necessary flags to make it work with it. Since you append directly the node_modules folder, you invoked the executable but no the flag format.
To make it work:
PreCommit:
  Stylelint:
    enabled: true
    required_executable: 'node_modules/stylelint/bin/stylelint.js'
    command: ['node_modules/stylelint/bin/stylelint.js', '**/*.css', '-f', 'compact']

I hope that helps.
@sds If from his side it's correct, we can close this topic. If you want I can do a small PR with extra instructions inside the file explaining situations like this

@Ruffeng thanks for the solution! 👍 it helps but i think the downside of this is. It will lint all the .css files whether its edited or not which i think its not necessary. should i use global stylelint instead of node_modules? but i produce same errors with stylelint global too. thanks! 😊


Abracadabra Ethereum: Indexing Error

https://okgraph.xyz/?q=messari%2Fabracadabra-money-ethereum

@dmelotik Is this issue closed?


tests: Rename uuid.test.ts to uuid-coercion.test.ts

Test gave false impression of testing uuid, but really tests something related to coercion from uuid to strong.

Looks like it's about making sure that what goes into String[] doesn't blow up during validation because of a uuid, so actually testing the runtime validation for String[]

Replaced by https://github.com/prisma/prisma/pull/17969


WTF with the spam?

Installed extension, immediately, a window pops up with

Hrm, odd, uninstall and install again. Then get a new one:

Repeated and got the last one again. I lost trust with this extension, and this is after I had gained trust by seeing it was here on GitHub. What is up with this?

@ElijahLynn thanks for flagging - apologies for the poor experience. I had forgotten to renew the domain and as a result some automated system snatched it up. The extension was previously referencing relevant material that was hosted on my website at the time. Looks like the new owner has now automated redirects to certain sites unfortunately.
I updated my other extension but unfortunately haven't/forgot to update this one. I can totally understand how it looks like spam. Will find time to update the extension with updated urls.

@ElijahLynn thanks for flagging - apologies for the poor experience. I had forgotten to renew the domain and as a result some automated system snatched it up. The extension was previously referencing relevant material that was hosted on my website at the time. Looks like the new owner has now automated redirects to certain sites unfortunately.
I updated my other extension but unfortunately haven't/forgot to update this one. I can totally understand how it looks like spam. Will find time to update the extension with updated urls.

Phew, thanks so much for that explanation! I trust this again and will go ahead and give this a go! Thanks for your work on this.

Just happened again when I had to "repair" the extension under manage extensions. Please update soon, this can cause serious harm to some from phishing.

This is going to get the extension to be reported by many, if it hasn't already.

As a quick fix I've removed it from the store until I could properly update it.

Thanks Peter!

Hey, I was wondering if there was any work done on fixing this spam issue. I noticed the popups after loading the extension using the files here. I restarted my computer which got rid of extension since it was temporarily loaded but the spam issue still persists. I tried cleaning the registry which didn't fix it.
Any thoughts?

I may have been mistaken in attributing my spam issues to this extension. My previous complaint can be disregarded. Still, it would be awesome if we could get this extension back on the Chrome store.
Sorry for the confusion. I appreciate the work done on this extension.


Deprecated warning on Ember 2.6.1

I'm using this great addon on Ember 2.6.1, but I find a deprecation warning in console.
vendor.js:22593 DEPRECATION: [DEPRECATED] didInitAttrs called in <udesk-crm@component:bs-switch::ember436>. [deprecation id: ember-views.did-init-attrs] See http://emberjs.com/deprecations/v2.x#toc_ember-component-didinitattrs for more details.
        at logDeprecationStackTrace (http://crm.ud.com/site/assets/vendor.js:22575:19)
        at HANDLERS.(anonymous function) (http://crm.ud.com/site/assets/vendor.js:22682:7)
        at raiseOnDeprecation (http://crm.ud.com/site/assets/vendor.js:22605:12)
        at HANDLERS.(anonymous function) (http://crm.ud.com/site/assets/vendor.js:22682:7)
        at handleDeprecationWorkflow (http://crm.ud.com/site/assets/vendor.js:73769:9)
        at HANDLERS.(anonymous function) (http://crm.ud.com/site/assets/vendor.js:22682:7)
        at deprecationCollector (http://crm.ud.com/site/assets/vendor.js:73794:5)
        at HANDLERS.(anonymous function) (http://crm.ud.com/site/assets/vendor.js:22682:7)
        at http://crm.ud.com/site/assets/udesk-crm.js:4335:21

Is it on your plan to upgrade it to newer version of Ember in order to remove that?

Yep, released a new version last week to remove this warning. Update to version 1.13.3 of this addon. 😄


Radix Sort In C

Introduction to the Sort
Time Complexity Information Of All Scenarios
Code For The Algorithm

I want to work on this issue. Please assign me @ST1LLWATER


feat(client): cache is supported in web/browser platform

This is an ongoing effort to make this package compatible to web/browser #295.
In web/browser platform, Cache is supported by localStorage.
The naming InMemoryCache is also debatable, as both io and html
implementation of this class is based on "persistent storage", not at all
"in memory".
There are refactor works to do to consolidate in_memory_* files.
Breaking changes
InMemoryCache no longer accept storageProvider,
however this param has always been using for testing only.

Codecov Report

Merging #325 into beta will decrease coverage by 0.05%.
The diff coverage is 82.35%.

@@            Coverage Diff             @@
##             beta     #325      +/-   ##
==========================================
- Coverage   45.68%   45.63%   -0.06%
==========================================
  Files          28       28
  Lines        1055     1054       -1
==========================================
- Hits          482      481       -1
  Misses        573      573

Flag
Coverage Δ

#graphql_client
46.13% <82.35%> (-0.06%)
:arrow_down:

#graphql_flutter
29.03% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

packages/graphql/lib/src/cache/cache.dart
0% <0%> (ø)
:arrow_up:

packages/graphql/lib/src/cache/optimistic.dart
70.21% <0%> (ø)
:arrow_up:

...es/graphql/lib/src/cache/normalized_in_memory.dart
85.71% <100%> (ø)
:arrow_up:

packages/graphql/lib/src/cache/in_memory_io.dart
85.41% <85.41%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d46c00c...afe29bb. Read the comment docs.

This is related to #297 - I really think caching is worth breaking into a separate module https://pub.dev/packages/caches
anyways @truongsinh I'm guessing rebasing will resolve these conflicts and then I can re-approve and merge

@micimize please be aware that I have a new commit, now the test is running in headless Chromium as well

:tada: This PR is included in version 1.1.1-beta.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Master to Live March 23 1:58 pm

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/case-study-scaling-datascape-across-devices-with-different-performance.md
:white_check_mark:Succeeded
View

mixed-reality-docs/coordinate-systems-in-unity.md
:white_check_mark:Succeeded
View

mixed-reality-docs/focus-point-in-unity.md
:white_check_mark:Succeeded
View

mixed-reality-docs/gestures-and-motion-controllers-in-unity.md
:white_check_mark:Succeeded
View

mixed-reality-docs/input-porting-guide-for-unity.md
:white_check_mark:Succeeded
View

mixed-reality-docs/interactable-object.md
:white_check_mark:Succeeded
View

mixed-reality-docs/locatable-camera-in-unity.md
:white_check_mark:Succeeded
View

mixed-reality-docs/mixed-reality-250.md
:white_check_mark:Succeeded
View

mixed-reality-docs/performance-recommendations-for-immersive-headset-apps.md
:white_check_mark:Succeeded
View

mixed-reality-docs/persistence-in-unity.md
:white_check_mark:Succeeded
View

mixed-reality-docs/porting-guides.md
:white_check_mark:Succeeded
View

mixed-reality-docs/shared-experiences-in-unity.md
:white_check_mark:Succeeded
View

mixed-reality-docs/spatial-mapping-design.md
:white_check_mark:Succeeded
View

mixed-reality-docs/spatial-mapping-in-unity.md
:white_check_mark:Succeeded
View

mixed-reality-docs/spatial-sound-in-unity.md
:white_check_mark:Succeeded
View

mixed-reality-docs/tracking-loss-in-unity.md
:white_check_mark:Succeeded
View

mixed-reality-docs/voice-input-in-unity.md
:white_check_mark:Succeeded
View

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


When login to boylove ACC manga cannot be read

Source information
Boylove / 香香腐宅1.4.10
Source language
Chinese
Steps to reproduce

Open extension in Webview
Login to my account
Go back to library all manga from boylove cannot be read

Expected behavior
Manga should be readed even I login to my ACC

Actual behavior
The login page have error now. Once I login ACC, the error will show. I have tried the troubleshooting guide still cannot solve.
Mihon/Tachiyomi version
0.15.3
Android version
Android 14
Other details
When not login to the acc, it is normal. After Login, all manga cannot be read.
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] I have admitted that I am a clown by having checked this box, as I have not read these acknowledgements.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Hi, may I know why this issue keep being closed ?

Now the issue this, why keep saying me not read the acknowledgements? Before is normal, this issue is happen just now.

Just now I alr open 1 issue but keep closed mine. I want to which I didn't follow the acknowledgement?

The bot already told you what you need to do.

The bot told me troubleshoot? I have mention in the post there, i have try. But still not success. Pls check the SQL STATE hy000


Can UIL deal with the images in webview?

Webview ia also a place that consumes a lot of memory. Can UIL do something with it ?

Do something what?

i mean, can UIL control the image shown in the webview, like downloading, caching, and controlling the size of bitmpa?


Sync eng/common directory with azure-sdk-tools for PR 8413

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/8413  See eng/common workflow

This is expected to be a breaking change for container registry tests that utilize manifestv2 json bodies. Confirmation build


Update Bulgarian translation

Translation by @RacerBG

Updates to the following anchors:
&compatPatch
&essentialFiles
&includesX
&patchOutdated
&patchUnavailable
&renameXtoY
&requiresMCM_X
&runAnimFramework
&versionOldX
&reqManualFix


Move to the newest version of VPP & vpp-agent

Coverage increased (+0.07%) to 77.706% when pulling ec29b71d431be05d5233ff2005477706ae47f785 on rastislavszabo:master into e4e0528b420171806f0bd42744c4103e3004f31c on contiv:master.


Fixing the redis DB

Codecov Report

Merging #2277 into remove-moi will not change coverage.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           remove-moi    #2277   +/-   ##
===========================================
  Coverage       93.78%   93.78%
===========================================
  Files             161      161
  Lines           18249    18249
===========================================
  Hits            17115    17115
  Misses           1134     1134

Impacted Files
Coverage Δ

qiita_db/environment_manager.py
45.74% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 22d52f0...27de0af. Read the comment docs.


Implement a banner art fetching option

Currently one can only get the avatar, but another option be added to fetch the banner art of the profile. For example Twitter, the structure is practically the same to the profile picture so adding that functionality wouldn't be to hard.

It isn't the target of the project, sorry!
You can build your own data rules using Microlink, check this:
https://microlink.io/recipes/twitter
🙂


⚠ Bofrost.fr has degraded performance

In 92940be, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1214 ms

Resolved: Bofrost.fr performance has improved in 667246e.