Minetest and openage are not on Top 50 ranking

Provided links:
https://github.com/minetest/minetest (8.6K stars)
https://github.com/SFTtech/openage (11.7K stars)

Thanks. That definitely should be investigated.

I solved the puzzle. Problem was that these two entries had keywords for framework/tools in it and were sorted into https://trilarion.github.io/opensourcegames/frameworks/index.html. Not counting as "games", they could not appear in the top 50 stars list at all. Leave this open until I can check all non-games to see if they are really "game engines / frameworks / tools".

I will also expand the page to top 100 and include Gitlab.


Implement a matrix path variable parser

Approximates what is described at https://www.w3.org/DesignIssues/MatrixURIs.html

I've tried to address the comments and also make what is excepted more permissive.


Tabs are not colored correctly by default.

What vue-mdc-adapter Version are you using?

0.17.0

What browser(s) is this bug affecting?

All browsers

What OS are you using?

All OS

What are the steps to reproduce the bug?
https://codepen.io/k900/pen/GBZeJr
What is the expected behavior?

Tabs should follow the toolbar text color (in this case, white)

What is the actual behavior?

Tabs are black

right MDCWeb Tabs can not know they are in a toolbar, and which is the text color of the toobar...
I need to investigate how I can do this.
in the mean  time, you can use MDCWeb sass mixins to color them
https://material.io/develop/web/components/tabs/tab/#sass-mixins
https://material.io/develop/web/docs/theming/#custom-themes

Yeah, that's what I ended up doing. A separate style for tabs inside a toolbar should be possible as part of the toolbar component, but that seems hacky :(


Let Server Handle 404s

Previously, we would forward all page routes to index.html with status code 200 and let the frontend handle displaying a 404 page to the user.  This PR will only send index.html for existing routes and send an actual 404 for everything else.  When running on the live server, @PeterRJones's code will swap the 404 response for a standardized 404 page.
Note that nonexistent workflow and run pages (/workflow/invalid or /workflow/run/invalid) will still be handled by the frontend.  Also, it may still be possible to hit the unstyled frontend 404 page in the case that we have a bug that causes the frontend routing to direct the user to a nonexistent page without a full page refresh.

Looks good @justinmc


[Keyboard Manager] Key remapped to backspace acquires unexpected behavior

ℹ Computer information

Windows build number: Windows 10 2004 (19041.388)
PowerToys version: 0.20.0
PowerToy module: Keyboard Manager

📝 Provide detailed reproduction steps (if any)

Remap Alt (Right) to Backspace
Write anything on a text editor
Press Alt (Right)

✔ Expected result
Delete only the last character
❌ Actual result
It deletes the whole word
📷 Screenshots

@denisdifazio which keyboard layout are you using? I think if you're not using English US, what's happening is that the right Alt key on your keyboard (also known as AltGr if the keyboard layout is not En-US) is actually Ctrl+Alt, so when you press the AltGr key with the above remappings, it sends Ctrl+Backspace, so the whole word gets deleted. You can confirm this by opening the Remap shortcut window and click 'Type' and press the right Alt button, it should show up as Ctrl+Alt.
As of now we don't support remapping multiple modifiers (Ctrl+Alt) to a key/shortcut. but we plan to support this in the future.

Thank you @arjunbalgovind for the clarification! I didn't know about this behaviour of AltGr. As you suspected, I am not using English US keyboard layout. I am using Brazil ABNT2.
I am closing this issue for now.


[DCOS-54125] Fix DC/OS 1.11 Teamcity test fail issue

What changes are proposed in this PR?
Resolves DCOS-54125
Fix DC/OS 1.11 Teamcity test fail issue
DC/OS 1.11 build was failing due to following reason
Failed to write rendered template from envvar 'CONFIG_TEMPLATE_JMX_CONFIG' to '.ssl/jmx-ssl-setup.sh': open .ssl/jmx-ssl-setup.sh: no such file or directory

because path .ssl doesn’t exist in DC/OS 1.11 hence was failing, in this PR i have moved jmx-ssl-setup.sh to root folder, which would be accessible for all DC/OS versions.

Cassandra Integration Tests DC/OS 1.11 Strict https://teamcity.mesosphere.io/viewLog.html?buildId=2217960&buildTypeId=DataServices_Cassandra_IntegrationTests_DCOS_111_Strict&tab=buildResultsDiv

all CI tests are passed.
https://teamcity.mesosphere.io/project.html?projectId=DataServices_Cassandra_Nightly&branch_DataServices_Cassandra_Nightly_DCOS_111=pull%2F3167


Make temperature field optional in OpenAIConfig due to type handling bug

Changed the type of the temperature field in the OpenAIConfig class from float to Optional[float].
This change was made to handle cases where the temperature field is not provided in the configuration file.

📝 Description
Please include a summary of the proposal, relevant motivation and context.
📸 Overview
Before
{Please add a screenshot here}
After
{Please add a screenshot here}
ℹ Issue
Closes issue #1

Can you also fix g3 configure to autocomplete the CLI with the already saved configs in ~/.g3/config, if existed, before falling back to the defaults? At the moment it starts the dialog with clean state as if there is not saved config.

will look at it now


Add impl of scipy.linalg.lu_solve

This is used by the Glow generative model to parameterize an invertible 1x1 convolution.
A TF implementation in terms of triangular solve is here:
https://github.com/tensorflow/probability/blob/master/tensorflow_probability/python/math/linalg.py#L306

Pretty much exactly the same functionality is already available via jax.numpy.linalg.solve, but I guess there's no harm in adding the scipy variant too. (It supports an additional transpose or conjugate transpose.)


Does the tagger use PoS tags as an input feature for NER ?

Does the tagger use PoS tags available in the Conll dataset as an input feature for NER ?
What are the input features, apart from the word embeddings, used by the tagger?

It does not use the POS tags, only the words. We tried to use POS tags, it was helpful for Dutch and German, but not so much for English and Spanish.

Thanks a lot Sir !


Hide CDB TODOs

Fixes

[ ] undo/redo on create table : does not refresh table list
[ ] add property field on list while some already opened (npc.require*) trigger exception
[ ] double click required to open script (instead of single click for other ui)
[ ] closing a list containing an opened script property does not save it
[x] open script column, click on a ref cell then click back on script : keys input trigger ref dialog
[ ] using MoveUp or Ctrl+Up/Down on a list element does not work (applies to parent!)
[ ] same for Paste on a Property : will paste on the root as well
[ ] using MoveUp on topmost element should move before first separator
[ ] List > Props > List : not cursor selected on click
[ ] have a different selection color of opened subsections whereas you have cursor into it or not (problem when several parallel of them opened)
[ ] when doing Ctrl+Down many times to move a line downwards with separators, sometimes we loose the cursor selection
[ ] moving cursors with arrows in filtered view moves it also on hidden lines

TODOs

[ ] cleanup undo mess : undo handled by view, safe current sheet + scroll pos
[ ] Undo does not persist well when switching table (like CDB)
[ ] Custom Types
[ ] handle propagation of id renames / structural changes to prefabs (with resolve cache)
[ ] think about moving CDB Editor again to CDB, but as a library component used by HIDE -- issue : script editor
[ ] when api.xml changes, refresh all scripts check in current page (not only opened editors)

Not important:

[x] Shift+Enter on id triggers exception

Done Already

[x] full screen view : chrome flex bug hiding tabs (move tab header to bottom like old CDB)
[x] empty database : allow create sheet
[x] fixed columns : not displayed correctly over the current columns
[x] fixed columns : when opening a table with Database > View > resources it will duplicate the ids (not sure why)
[x] fixed columns : bug on subtable (should be disabled)
[x] Ctrl+F for filter : filter not working, pressing escape loose focus
[x] Undo creates #DUP errors (need to switch table)
[x] support sheet popup
[x] create sheet : need undo
[x] add first column on new sheet : no refresh
[x] add an empty List and open : no ability to insert column
[x] open two Properties, change a Reference value on one --> the other one gets focus
[x] new sheet -> should display it
[x] add new column on subtable --> should not add to top level bar
[x] Ctrl+V with clipboard plain string does not work on UID
[x] Ctrl+F not working if we click on a table tab (no focus)
[x] display of Flags list is scrolling down additionally to the table scroll
[x] change Property Reference -> lose focus?
[x] add new Properties field, open, add a new field --> null access
[x] delete a column : no redo ? no save ?
[x] close and restart (or rename sheet) should get back to latest opened sheet(s)
[x] AllEditors should also return embedded editors
[x] when reloading CDB, clear undo (prevent undoing external changes)
[x] bluring after text edition when scroll is below the page bottom will slightly scroll up (annoying)
[x] right click on label should not show browser popup
[x] list in props requires double click to toggle
[x] adding a field in a list in a props trigger exception

Another bug:
Database -> New Sheet when cdb view is open does not refresh the tab list. Have to close and reopen the cdb view to see new sheet tab. Works fine if doing right click on existing sheet -> Add Sheet


"Actions" Dropdown causes transitions to respective routes

I'm gonna go ahead and close this since it seems like we're just fine-tuning the actions now and the basic functionality is all there.


darwin.builder: Instruct user not to disable firewall

It turns out that it's not necessary to disable the firewall because qemu by default binds to all ports (<IP_ADDRESS>), including <IP_ADDRESS>.
Ideally, we would configure qemu to bind to <IP_ADDRESS> so that the user would not even be prompted to disable their firewall, but that doesn't work for unprivileged services on macOS.
You heard that right.  On macOS an unprivileged service can bind to <IP_ADDRESS>:22 but not <IP_ADDRESS>:22. See:
https://developer.apple.com/forums/thread/674179
… so right now the lesser evil is just telling users to not disable the firewall rather than requiring them to run the script with sudo.
Description of changes

Things done

Built on platform(s)

[x] aarch64-darwin

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Fits CONTRIBUTING.md.

Note: The first time you run this macOS will ask you if you want to open your
firewall to accept incoming connections.  You can deny that request because it
is not necessary to open your firewall to use the builder.

I wasn't able to repro this, FWIW. (Is the macOS firewall opt-in?)

I actually do not remember.  I had it disabled on my machine but I wasn't sure if that was because I had explicitly opted out of it before

It looks like the firewall is of by default, so I'll retract this pull request


Fix basicsr conflicts

Fix basicsr incompatibilities
The issue is the basicsr 1.4.2 doesn't work on Windows, and 1.4.1 doesn't work on Mac. However, GFPGAN requires 1.4.2 or higher.
To work around this, we pull in either of two different branches of the invoke-ai fork of GFPGAN, depending on the OS. This seems to be working well with the conda install, but I'm still getting issues with the pip-only install when trying to load the Windows requirements file on Linux ("torch not found" error). I'm not sure if this is a problem that will appear on a real Windows system, but needs testing. The workaround is to run pip install torch==1.12.*

@Vargol See if this fixes the basicsr problems on Mac and let me know the result in a comment.

Looks good.
ln -sf environments-and-requirements/requirements-mac-mps-cpu.txt requirements.txt
pip --prefer-binary install -r requirements.txt

installed without issue, and I was able to start the GUI and run a prompt after finishing the setup.

Hello. Sorry if I suddenly barge in into the discussion.

To work around this, we pull in either of two different branches of the invoke-ai fork of GFPGAN, depending on the OS. This seems to be working well with the conda install, but I'm still getting issues with the pip-only install when trying to load the Windows requirements file on Linux ("torch not found" error). I'm not sure if this is a problem that will appear on a real Windows system, but needs testing. The workaround is to run pip install torch==1.12.*

It did appear at #1409. I still experienced it when re-install InvokeAI recently using  93cdb476d919b779e32a638aae15e56cc46b7adb using pip-based install method. The workaround is to install torch and torchvision manually using pip before installing the required packages from requirements.txt file.

@Vargol and @addianto,  I have implemented a different fix that involved tinkering with our fork of GFPGAN. If you have a chance, could you go to the main or development branch and try updating your environments?
Thanks.

Will do. Thanks for the update. 👍

Hello @lstein,
I tried running pip install on two instances of InvokeAI. First is the instance based on main branch (58f65d49b6cd17c8fc70e86b32a41ca7a98ce304) with torch dependency installed manually, i.e. not through pip install -r requirements.
I called python -m pip install --prefer-binary -r .\requirements.txt to update the environment.
There is no error whatsoever. The updated GFPGAN seems successfully installed as well based on the following log:
Collecting clip
  Cloning https://github.com/openai/CLIP.git (to revision main) to c:\users\REDACTED\appdata\local\temp\pip-install-dqtvhdv0\clip_8abb61d2a34b48a595d197c9980bff08
  Running command git clone --filter=blob:none --quiet https://github.com/openai/CLIP.git 'C:\Users\REDACTED\AppData\Local\Temp\pip-install-dqtvhdv0\clip_8abb61d2a34b48a595d197c9980bff08'
  Resolved https://github.com/openai/CLIP.git to commit d50d76daa670286dd6cacf3bcd80b5e4823fc8e1
  Preparing metadata (setup.py) ... done
Collecting k-diffusion
  Cloning https://github.com/Birch-san/k-diffusion.git (to revision mps) to c:\users\REDACTED\appdata\local\temp\pip-install-dqtvhdv0\k-diffusion_c1af56a85e014732b38fa664471be9de
  Running command git clone --filter=blob:none --quiet https://github.com/Birch-san/k-diffusion.git 'C:\Users\REDACTED\AppData\Local\Temp\pip-install-dqtvhdv0\k-diffusion_c1af56a85e014732b38fa664471be9de'
  Resolved https://github.com/Birch-san/k-diffusion.git to commit 9bfd658ba7190f6e1403d8c77f6c8e26a4a9a7dc
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Installing backend dependencies ... done
  Preparing metadata (pyproject.toml) ... done
Collecting clipseg
  Cloning https://github.com/invoke-ai/clipseg.git (to revision relaxed-python-requirement) to c:\users\REDACTED\appdata\local\temp\pip-install-dqtvhdv0\clipseg_4dbad855101a451d9c3490686686f0b3
  Running command git clone --filter=blob:none --quiet https://github.com/invoke-ai/clipseg.git 'C:\Users\REDACTED\AppData\Local\Temp\pip-install-dqtvhdv0\clipseg_4dbad855101a451d9c3490686686f0b3'
  Running command git checkout -b relaxed-python-requirement --track origin/relaxed-python-requirement
  branch 'relaxed-python-requirement' set up to track 'origin/relaxed-python-requirement'.
  Switched to a new branch 'relaxed-python-requirement'
  Resolved https://github.com/invoke-ai/clipseg.git to commit 1f754751c85d7d4255fa681f4491ff5711c1c288
  Preparing metadata (setup.py) ... done
Collecting gfpgan
  Cloning https://github.com/invoke-ai/GFPGAN to c:\users\REDACTED\appdata\local\temp\pip-install-dqtvhdv0\gfpgan_8282f53a5f5a4c92aae88d4c7045190f
  Running command git clone --filter=blob:none --quiet https://github.com/invoke-ai/GFPGAN 'C:\Users\REDACTED\AppData\Local\Temp\pip-install-dqtvhdv0\gfpgan_8282f53a5f5a4c92aae88d4c7045190f'
  Resolved https://github.com/invoke-ai/GFPGAN to commit 51593f7bb5a9a86c58dd91ae2e7fd6b20aa486b4
  Preparing metadata (setup.py) ... done
Obtaining file:///D:/REDACTED/workspace/ai-art/frontend/InvokeAI-fork (from -r .\requirements.txt (line 8))
  Preparing metadata (setup.py) ... done

The second instance is probably more related to the dependency issue that supposed to be addressed by this PR.
I tried to perform fresh installation of InvokeAI based on latest commit on development branch (d998b2f80654cfddc4df3a05cec9f6b5a72b89bf). It still fails when the installation process attempted to install basicsr.
Here I attached stdout and stderr output from  pip install -v -r requirements.txt invocation:
pip_error_development.log

Much obliged for the testing and the log. So it looks like the path of
least resistance here is to modify the installation instructions to have
the user run the command:
pip install pytorch==1.12 torchvision torchaudio

Just prior to running pip install -r requirements.txt. Will this work?
Lincoln
On Thu, Nov 17, 2022 at 9:56 AM Daya Adianto @.***>
wrote:

The second instance is probably more related to the dependency issue that

supposed to be addressed by this PR.
I tried to perform fresh installation of InvokeAI based on latest commit
on development branch (d998b2f
https://github.com/invoke-ai/InvokeAI/commit/d998b2f80654cfddc4df3a05cec9f6b5a72b89bf).
It still fails when the installation process attempted to install basicsr.
Here I attached stdout and stderr output from pip install -v -r
requirements.txt invocation:
pip_error_development.log
https://github.com/invoke-ai/InvokeAI/files/10032492/pip_error_development.log
—
Reply to this email directly, view it on GitHub
https://github.com/invoke-ai/InvokeAI/pull/1478#issuecomment-1318758441,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA3EVNPVKQGPU5SLVQWFPLWIZBRFANCNFSM6AAAAAASAJ3UF4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Much obliged for the testing and the log. So it looks like the path of least resistance here is to modify the installation instructions to have the user run the command: pip install pytorch==1.12 torchvision torchaudio Just prior to running pip install -r requirements.txt. Will this work?

I think it will work, at least on manual, pip-based installation. The pip commands in docs/installation/INSTALL_MANUAL.md may need to be modified to this example:
pip install --extra-index-url https://download.pytorch.org/whl/cu116 --trusted-host https://download.pytorch.org torch==1.12.1 torchvision==0.13.1
pip install --prefer-binary -r requirements.txt # Copy-pasted/symbolic linked to requirements-win-colab-cuda.txt


⚠ Bofrost.be has degraded performance

In 0ffd3bd, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1651 ms

Resolved: Bofrost.be performance has improved in 2bb137b.


TypeError: _webpack2.default.optimize.OccurenceOrderPlugin is not a constructor

We've recently updated npms in our project. Storybook is now complaining with this error:
><EMAIL_ADDRESS>start:storybook /Users/davidvass/Work/Showmax/website/react
> start-storybook -p 9001

/Users/davidvass/Work/Showmax/website/react/node_modules/@kadira/storybook/dist/server/config/webpack.config.js:34
  plugins: [new _webpack2.default.optimize.OccurenceOrderPlugin(), new _webpack2.default.HotModuleReplacementPlugin(), new _caseSensitivePathsWebpackPlugin2.default()],
            ^

TypeError: _webpack2.default.optimize.OccurenceOrderPlugin is not a constructor
    at Object.<anonymous> (/Users/davidvass/Work/Showmax/website/react/node_modules/@kadira/storybook/dist/server/config/webpack.config.js:34:13)
    at Module._compile (module.js:556:32)
    at Object.Module._extensions..js (module.js:565:10)
    at Module.load (module.js:473:32)
    at tryModuleLoad (module.js:432:12)
    at Function.Module._load (module.js:424:3)
    at Module.require (module.js:483:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/davidvass/Work/Showmax/website/react/node_modules/@kadira/storybook/dist/server/middleware.js:55:17)
    at Module._compile (module.js:556:32)

The apparent solution should be this (https://github.com/webpack/webpack/issues/1964):

OccurenceOrderPlugin has been renamed to OccurrenceOrderPlugin.

I know that in your package.json there is webpack in version 1.13.1, but it would be nice to support version 2 as well.
I've tried to rename problematic plugin name and after renaming it works fine. I suggest using some condition there so it will be compatible with older versions of webpack as well the newest one. What do you think?

What's the storybook version you are running. Try to use the latest which is: v2.3.0. I hope it has the fix for this.

May be this seems like a bug. Let me try to fix this.

I've updated to v2.3.0 and the problem is still present. I think the only two options here are a) upgrade webpack or b) put simple condition there (e.g. new (webpack.optimize.OccurenceOrderPlugin || webpack.optimize.OccurrenceOrderPlugin)())

BTW If my suggested solution b) is OK for you, I can create a PR myself since it's just one line modification.


Pass sizes to temporaries from gt4py program

Description
Requirements

[ ] All fixes and/or new features come with corresponding tests.
[ ] Important design decisions have been documented in the approriate ADR inside the docs/development/ADRs/ folder.

If this PR contains code authored by new contributors please make sure:

[ ] All the authors are covered by a valid contributor assignment agreement provided to ETH Zurich and signed by the employer if needed.
[ ] The PR contains an updated version of the AUTHORS.md file adding the names of all the new contributors.

@DropD Would you please review

src/gt4py/next/program_processors/codegens/gtfn/gtfn_backend.py
src/gt4py/next/program_processors/codegens/gtfn/gtfn_module.py
src/gt4py/next/program_processors/formatters/gtfn.py
These file contain the changes we discussed last week.


problem while deleting

Hi!
I am using this connector in a project and i have an issue when i delete an item from cosmosDb : "BadResponseException: Unsupported content-type: null"; i just don't know where it comes from...
The code i use in flutter is pretty straight forward :
Future<Result> delete(String id) async {
    try {
      await CosmosService().entityCollection.delete(id);
      return Result.success();
    } catch (e) {
      logger.wtf(e.toString());
      return Result.fail();
    }
  }

The item is actually deleted from cosmosDb but i always fall in the catch part... i tried to print the boolean result expected from the delete method but never had a chance to see it...
All the other CRUDL methods work great but this one is resisting... Can you please tell me if i am doing something wrong or if there is something on your side that could help me?
Thanks!

Hello @Xebache,
I will need some more context to help you with this. I can trace the origin in _client.dart:
    if (contentType != MimeType.json) {
      throw BadResponseException('Unsupported content-type: $contentType');
    }

I'd be interested in checking the HTTP request/response. You can enable a debug HTTP client by importing azure_cosmosdb_debug.dart instead of azure_cosmosdb.dart. Then you can provide a debug HTTP client when connecting with the database :
  final server = CosmosDbServer(
    'https://localhost:8081',
    masterKey: masterKey,
    httpClient: DebugHttpClient(),
  );
  server.useLogger(print, withBody: true, withHeader: true);

Please locate the DELETE command in the log and post the request/response here so I can investigate.
Make sure you don't advertise your master key or any other sensitive data when posting this material ;-)
Regards

BTW I'm working on a version 2. It will have some breaking changes but also provide improvement like batch processing and (experimental) support of MultiHash partition keys and general improvements for partition keys in general (at least in my opinion). I'll hopefully post something this week-end.

Hi @d-markey !
here is what i get in the log:

[2023-01-06T22:35:31.518Z] --> DELETE https://.../docs/2pyLxW50iuNtYz6B4vciBEwmiOh1
[2023-01-06T22:35:31.518Z] --> authorization = ...
[2023-01-06T22:35:31.518Z] --> x-ms-date = Fri, 06 Jan 2023 22:35:31 GMT
[2023-01-06T22:35:31.518Z] --> x-ms-version = 2018-12-31
[2023-01-06T22:35:31.518Z] --> user-agent = AzureCosmosDb.Dart/1.8.0
[2023-01-06T22:35:31.518Z] --> x-ms-documentdb-partitionkey = ["2pyLxW50iuNtYz6B4vciBEwmiOh1"]
[2023-01-06T22:35:31.518Z] --> body =
[2023-01-06T22:35:31.518Z] <-- COSMOS DB Status = 204
[2023-01-06T22:35:31.518Z] <-- access-control-allow-credentials = true
[2023-01-06T22:35:31.518Z] <-- access-control-allow-origin = http://localhost:8000
[2023-01-06T22:35:31.518Z] <-- cache-control = no-store, no-cache
[2023-01-06T22:35:31.518Z] <-- content-length = 0
[2023-01-06T22:35:31.518Z] <-- pragma = no-cache
[2023-01-06T22:35:31.518Z] <-- strict-transport-security = max-age=31536000
[2023-01-06T22:35:31.518Z] <-- x-ms-activity-id = 3e9aa140-c981-4e9c-95b4-507ae5f86822
[2023-01-06T22:35:31.518Z] <-- COSMOS DB 204 Payload =
<<<<<<<<<<<<<<<<
👾  UserService - DELETE dbs/MilesLegalDB/colls/Users/docs/2pyLxW50iuNtYz6B4vciBEwmiOh1: BadResponseException: Unsupported content-type: null
i also noticed in the browser's inspector that the request has no response data available... but the item is deleted from the db.
Another matter, not related at all: i have a list of 205 items that i have to import at once; i had to import your connector locally to add Paging to the Context and allow to specify a max length for the list() method. It would be nice to allow user to fetch more than 100 items if needed :-))

Hello, I've just published version 1.8.1 which should fix your problem. Let me know if everything's OK now!

Yup, it worked! Thank you !!!


Ensure the ptr is pointing to reflect.Slice in ExtractList

LGTM
Is this just caution, or did it manifest in some way?

@thockin the code in this PR is not ready yet, I just found that if items is *[]runtime.Object, it prints the error... still debugging...

@thockin done :-) to answer the question, this is just a caution, there is no use-case for this in the code (neither in openshift codebase).

@smarterclayton @csrwng fyi.

LGTM


Option convert to native

Description

Added option on the context menu to not convert to native for the Sync Receive Component

Type of change

New feature (non-breaking change which adds functionality)

How has this been tested?

Manual Tests (please write what did you do?)

Docs

Will be updated ASAP

Hey @psarras!
Thanks for this! We've gone over your changes and it's a great (and necessary) addition.
There's one change i'd like to suggest to keep things aligned with the Object Management nodes.
The way other nodes work is that the "Do not convert" option is part of the KitSelection process. When this option is selected, the Converter instance in that component is set to null, and, whenever it's used, we just perform a null check and pass-through the value instead of converting.
In your node, both options can be active at the same time (i.e. you can have a Kit selected AND the "Do not convert" option active). Is there a specific reason you'd need both active? It would be preferable that all nodes that use this option have a consistent behaviour, but we can organise a video call if you prefer to discuss this things in detail if you prefer.
Just for guidance:
Here's how we're adding the do not convert option.
https://github.com/specklesystems/speckle-sharp/blob/56c0b700d953020ef6c8ad9e06d97dfb22e17b3b/ConnectorGrasshopper/ConnectorGrasshopper/Objects/SelectKitTaskCapableComponentBase.cs#L87-L92
And here's the Kit selection logic.
https://github.com/specklesystems/speckle-sharp/blob/56c0b700d953020ef6c8ad9e06d97dfb22e17b3b/ConnectorGrasshopper/ConnectorGrasshopper/Objects/SelectKitTaskCapableComponentBase.cs#L46-L63
And then when using the converter:
https://github.com/specklesystems/speckle-sharp/blob/56c0b700d953020ef6c8ad9e06d97dfb22e17b3b/ConnectorGrasshopper/ConnectorGrasshopper/Objects/CreateSpeckleObjectTaskComponent.cs#L160
I also noticed that I didn't add this feature on the original Receive node. Would you be willing to add this functionality there too?
PS. Sorry for delay in getting back to you, I was certain I had answered last week... not sure what happened with that message! 😓


Backgroun compression job error

With Postgresql 12 and those extensions:
extname    |extowner|extnamespace|extrelocatable|extversion
-----------|--------|------------|--------------|----------
plpgsql    |      10|          11|false         |1.0
promscale  |      10|       17523|false         |0.1.1
timescaledb|      10|        2200|false         |2.0.0

and PromScale 0.1.4:
level=info ts=2021-01-06T17:21:29.005Z caller=runner.go:149 msg="Version:0.1.4; Commit Hash
I am getting the error below in the logs. Despite this the job status in the job_status table is Success - should the error be ignored?
job_id|last_run_started_at|last_successful_finish|last_run_status|job_status|last_run_duration|next_start         |total_runs|total_successes|total_failures|
------|-------------------|----------------------|---------------|----------|-----------------|-------------------|----------|---------------|--------------|
1000|2021-01-08 09:29:36|   2021-01-08 09:29:36|Success        |Scheduled |  00:00:00.073815|2021-01-08 09:59:36|        26|             19|             7|

2021-01-08 07:30:07 UTC [23445]: [5ff809fe.5b95-1] @,app= [00000] LOG:  job 1000 threw an error
2021-01-08 07:30:07 UTC [23445]: [5ff809fe.5b95-2] @,app= [42703] ERROR:  record "r" has no field "id"
2021-01-08 07:30:07 UTC [23445]: [5ff809fe.5b95-3] @,app= [42703] CONTEXT:  SQL statement "SELECT _prom_catalog.lock_metric_for_maintenance(r.id)"
        PL/pgSQL function execute_compression_policy() line 28 at PERFORM
        SQL statement "CALL _prom_catalog.execute_compression_policy()"
        PL/pgSQL function execute_maintenance() line 5 at CALL
        SQL statement "CALL prom_api.execute_maintenance()"
        PL/pgSQL function execute_maintenance_job(integer,jsonb) line 3 at CALL
2021-01-08 07:30:07 UTC [43]: [5ff5f18c.2b-17] @,app= [00000] LOG:  background worker "User-Defined Action [1000]" (PID 23445) exited with exit code 1

@franck102 thank you for opening an issue for this. @cevian could you please take a look at this?

@franck102 thank you for opening an issue for this. @cevian could you please take a look at this?

While we are here, is it normal to have 2 background jobs scheduled for execute_maintenance?
   1000 | User-Defined Action [1000]                 | 00:30:00          | 00:00:00    |          -1 | 00:05:00     | _prom_catalog         | execute_maintenance_job             | postgres | t         |                                                                                | 2021-01-22 13:25:21.44217+00  |                       |
   1001 | User-Defined Action [1001]                 | 00:30:00          | 00:00:00    |          -1 | 00:05:00     | _prom_catalog         | execute_maintenance_job             | postgres | t         |                                                                                | 2021-01-22 13:50:25.154563+00 |                       |

Installation is through the timescaledb-single and promscale helm charts:
Image:         timescaledev/timescaledb-ha:pg12-ts2.0-latest
Image:         timescale/promscale:0.1.4

While we are here, is it normal to have 2 background jobs scheduled for execute_maintenance?
   1000 | User-Defined Action [1000]                 | 00:30:00          | 00:00:00    |          -1 | 00:05:00     | _prom_catalog         | execute_maintenance_job             | postgres | t         |                                                                                | 2021-01-22 13:25:21.44217+00  |                       |
   1001 | User-Defined Action [1001]                 | 00:30:00          | 00:00:00    |          -1 | 00:05:00     | _prom_catalog         | execute_maintenance_job             | postgres | t         |                                                                                | 2021-01-22 13:50:25.154563+00 |                       |

Installation is through the timescaledb-single and promscale helm charts:
Image:         timescaledev/timescaledb-ha:pg12-ts2.0-latest
Image:         timescale/promscale:0.1.4

Found this in the logs, the 2 identical jobs seem intentional, and it looks like the error simply happens when there is nothing to do:
--add 2 jobs executing every 30 min by default for timescaledb 2.0
        DO $$
        BEGIN
            IF _prom_catalog.get_timescale_major_version() >= 2 THEN
               PERFORM add_job('_prom_catalog.execute_maintenance_job', '30 min');
               PERFORM add_job('_prom_catalog.execute_maintenance_job', '30 min');
            END IF;
        END
        $$;

Found this in the logs, the 2 identical jobs seem intentional, and it looks like the error simply happens when there is nothing to do:
--add 2 jobs executing every 30 min by default for timescaledb 2.0
        DO $$
        BEGIN
            IF _prom_catalog.get_timescale_major_version() >= 2 THEN
               PERFORM add_job('_prom_catalog.execute_maintenance_job', '30 min');
               PERFORM add_job('_prom_catalog.execute_maintenance_job', '30 min');
            END IF;
        END
        $$;

@franck102 Thanks for reporting this. We will aim to fix this in next release

@franck102 Thanks for reporting this. We will aim to fix this in next release


[Choose Your Fighter] Change follower appearances

Add a feature to CYF that makes it possible to change the appearance of followers.
We can probably have magic mirrors + gutsmongers open up an option list containing you and your followers whenever you try to change appearance.

Added in CYF 0.5.0.


update nestjs peer dependencies to v10

PR Type
Update peer dependencies of @nestjs to the latest version (v10).

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior?
NestJS recently released version 10 approximately two weeks ago, resulting in our project encountering the following warning:
─┬ @liaoliaots/nestjs-redis 9.0.5
│ ├── ✕ unmet peer<EMAIL_ADDRESS>found 10.0.4
│ └── ✕ unmet peer<EMAIL_ADDRESS>found 10.0.4

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

@liaoliaots could you have a look on this?

Hi @duysolo
I have a PR that I opened eighteen days ago. It has not been checked yet.
You can follow PR #517

@mahsumurebe it looks like we also need to upgrade node.js required version to 16 based on NestJS Migration Guide

Actually, it looks like !518 is actually more acurate beacause it does update everything needed and introduces major update to package

I’ve already changed the nodejs required version. Please check this

Hi @liaoliaots
Can you check the pull request?

Hey @liaoliaots , can you take a look here?

Looking forward to this one, it's blocking us from upgrading to v10

How is the state of this ?

any update? I need this upgrade

@Beni312 The repo owner is not making updates, I've removed nestjs-redis from my dependencies and rolled my own RedisService using ioredis:
import { Injectable, Logger, OnModuleDestroy } from '@nestjs/common';
import { Redis } from 'ioredis';
import { RedisException } from './redis.exception';

@Injectable()
export class RedisService implements OnModuleDestroy {
  private client: Redis | undefined; // undefined bc it might not exist due to lazy connect

  private createClient() {
    this.client = new Redis({
      host: process.env.REDIS_HOST,
      port: Number(process.env.REDIS_PORT),
      password: process.env.REDIS_PASSWORD,
      retryStrategy: (times: number) => {
        if (times > 10) {
          return undefined;
        }
        // retry at longer intervals based on the number of retries so far, max 3 seconds
        return Math.min(times * 100, 3000);
      },
    });

    if (!this.client) {
      throw new RedisException('Could not connect to Redis');
    }

    this.client.on('error', (error) => {
      Logger.error(`Redis error: ${error}`);
    });

    return this.client;
  }

  getClient(): Redis {
    if (!this.client) {
      this.createClient(); // lazy connect
    }
    return this.client;
  }

  async onModuleDestroy() {
    if (this.client) {
      return new Promise((resolve, reject) => {
        this.client?.quit((err, reply) => {
          if (err) {
            return reject(err);
          }
          resolve(reply);
        });
      });
    }
  }
}

...usable like so
import { RedisService } from 'src/redis/redis.service';

export class YourClass {
  constructor(
    private readonly redisService: RedisService,
  ) {
  // ...
  }
}

 async yourFunction() {
   const redis = this.redisService.getClient();
   await redis.set(
     // see ioredis documentation for basic usage
   );
 }

Hope that helps.

@Beni312 The repo owner is not making updates, I've removed nestjs-redis from my dependencies and rolled my own RedisService using ioredis:
import { Injectable, Logger, OnModuleDestroy } from '@nestjs/common';
import { Redis } from 'ioredis';
import { RedisException } from './redis.exception';

@Injectable()
export class RedisService implements OnModuleDestroy {
  private client: Redis | undefined; // undefined bc it might not exist due to lazy connect

  private createClient() {
    this.client = new Redis({
      host: process.env.REDIS_HOST,
      port: Number(process.env.REDIS_PORT),
      password: process.env.REDIS_PASSWORD,
      retryStrategy: (times: number) => {
        if (times > 10) {
          return undefined;
        }
        // retry at longer intervals based on the number of retries so far, max 3 seconds
        return Math.min(times * 100, 3000);
      },
    });

    if (!this.client) {
      throw new RedisException('Could not connect to Redis');
    }

    this.client.on('error', (error) => {
      Logger.error(`Redis error: ${error}`);
    });

    return this.client;
  }

  getClient(): Redis {
    if (!this.client) {
      this.createClient(); // lazy connect
    }
    return this.client;
  }

  async onModuleDestroy() {
    if (this.client) {
      return new Promise((resolve, reject) => {
        this.client?.quit((err, reply) => {
          if (err) {
            return reject(err);
          }
          resolve(reply);
        });
      });
    }
  }
}

...usable like so
import { RedisService } from 'src/redis/redis.service';

export class YourClass {
  constructor(
    private readonly redisService: RedisService,
  ) {
  // ...
  }
}

 async yourFunction() {
   const redis = this.redisService.getClient();
   await redis.set(
     // see ioredis documentation for basic usage
   );
 }

Hope that helps.

I put it in my application, works well, thanks :)

Please check out the new version: version 10 of @liaoliaots/nestjs-redis.


Pretrained Tokenizer Loading Fails: 'PosixPath' object has no attribute 'rfind'

I was trying to work through the toy tokenization example from the main README, and I hit an error on the step of loading in a pre-trained BERT tokenizer.
~/bert_transfer$ python3 test_tokenizer.py
Traceback (most recent call last):
  File "test_tokenizer.py", line 10, in <module>
    tokenizer = BertTokenizer.from_pretrained('bert-base-uncased')
  File "/usr/local/lib/python3.5/dist-packages/pytorch_pretrained_bert/tokenization.py", line 117, in from_pretrained
    resolved_vocab_file = cached_path(vocab_file, cache_dir=cache_dir)
  File "/usr/local/lib/python3.5/dist-packages/pytorch_pretrained_bert/file_utils.py", line 88, in cached_path
    return get_from_cache(url_or_filename, cache_dir)
  File "/usr/local/lib/python3.5/dist-packages/pytorch_pretrained_bert/file_utils.py", line 169, in get_from_cache
    os.makedirs(cache_dir, exist_ok=True)
  File "/usr/lib/python3.5/os.py", line 226, in makedirs
    head, tail = path.split(name)
  File "/usr/lib/python3.5/posixpath.py", line 103, in split
    i = p.rfind(sep) + 1
AttributeError: 'PosixPath' object has no attribute 'rfind'

~/bert_transfer$ python3 --version
Python 3.5.2

Exact usage in script:
from pytorch_pretrained_bert import BertTokenizer

test_sentence = "When PyTorch first launched in early 2017, it quickly became a popular choice among AI researchers, who found it ideal for rapid experimentation due to its flexible, dynamic programming environment and user-friendly interface"

if __name__ == "__main__":
    tokenizer = BertTokenizer.from_pretrained('bert-base-uncased')

I am curious if you're able to replicate this error on python 3.5.2, since the repo states support for 3.5+.

Ok @hzhwcmhf fixed this issue with #113 and we will be compatible with Python 3.5+ again in the coming release (today probably). Thanks @hzhwcmhf!


nick variable always null

$guild = $this->discord->guilds->get('id', $this->id);
$member = $guild->members->get("id", $discordID);
$nickName = $member->nick;
Always returns null
Var_dumping $member shows the variable but always lists it as null

Thanks!


[RAML to Code] Move Generation system used to generate Endpoints to a Templating Engine/ JCodeModel (Java Only)

Since we are already using Code Models in the project, it might be wiser to move to Code Model based generation, rather than templating for the improved engine

Hi Kurt,
I just pushed a first stable version of the feature to my repo.
Still have to do some code documentation but you can take a sneek preview if you want. ;)
The RamlGenerator.generateClassForRaml(..) method becomes obsolete.
My idea is to use instances of com.phoenixnap.oss.ramlapisync.generation.rules.Rule instead.
At the top level we have three rules that operates on top of a JCodeModel and a ApiControllerMetadata instance. These top level rules are plugged together out of more low level Rule instances, each one concerning another part of the generated artefact (e.g. package declaration, class declaration, class annotations, comments, method signatures, parameters ....).

Finally I created a new maven parameter  where the user can configure, which code generation rule should be used.
It should be easy for you to create completely new code generation rules to support older Spring-Versions or whatever.
Tell me what you think.
I can also create a PR if you prefer.
br,
Armin

Hi Armin
Thanks for taking the time to move this project forward :)
I started looking at your code and i like the approach you took. My only comments would be to move some of the methods currently in the plugin to the parser so that the parser is self sufficient and can be used not through maven
From my understanding so far, it's going to be trivial to generate unit tests, rest clients ec just by adding rules to the system - correct?
br
Kurt

Hi Kurt,
great to here that you like the approach. =)
I should indeed be very easy to setup new Rule classes and assemble other code generation variants (e.g. Rest-Clients). Unit tests for rules based on JCodeModel can follow the exisiting unit tests I already wrote.
One could even use other code abstraction models than JCodeModel. The Rule interface is very generic.
The best way to find out if the approach is flexibel enough would be to create another code generation variant. You spoke about older Spring-Versions. Perhaps you should go ahead and write some of those.
I'm not sure what you mean with "move some of the methods from plugin to parser".
Do you mean parts of the SpringMvcEndpointGeneratorMojo.generateEndpoints() stuff?
Cheers,
Armin

yes that's right: specifically maybe we can abstract away lines 130-142 into a method which offers to handle source generation.
I'll definitely try out writing a new rule. Shall you do a pull request and i'll put the code in a feature branch and code the new rules against it?

Hey Kurt,
I just executed an extract method refactoring on those lines.
Currently I'm not sure where to put the functionality. The RamlParser seems not the right place, because it would lack the single responsibility principle. In fact some of the main classes in parser module does. Perhaps I can do some refactorings here to clean up the code when I've time again.
Should I create a PR of the current version to master branch?

Thanks armin - re PR yes please :)

Ready to be released with 0.4.1. Massive Thanks to @aweisser


Make snarkyjs a monorepo by putting mina-signer inside it

Developer experience is much more sane in one repo, so we're pushing for it. Pending @jasongitmail giving feedback.

we aligned to move forward with this, initial work is covered by https://github.com/MinaProtocol/mina/issues/11855


Data table : Bug expandable with  selectable

I used data table and used expandable with selectable. When click to expand column of expand not correct.

See this REPL for a datatable that is both expandable and selectable.
Please provide a minimal repro and I'd be happy to investigate.


FRoadMapPixelData in RoadMap

what is mean of Data stored in a road map pixel in FRoadMapPixelData?
thanks alot

Hi @abdoly,
The road map is like an image of the whole city as taken from above. Each pixel (FRoadMapPixelData) stores the information of whether there is a road in that point, and if so, the direction of the road. Since this image gets serialized with each map, the information is encoded into an uint16 to optimize the memory usage.


Give --help example of using grep on stderr of rustc

It is very useful in cases like this but it is a bit tricky to figure out how to do it if you are new to using cargo-bisect-rustc.
By adding this example to --help we can make it easier for others to discover this trick.

I'm wondering, have you seen the documentation at https://rust-lang.github.io/cargo-bisect-rustc/? I'm a little reluctant to add lots of examples here, since there are many different workflows. What do you think of linking to that documentation in the help instead?

Thanks for taking a look at my PR.
Yep, I am aware of those docs. If strongly prefer a summary of examples in --help over being redirected to a web page with one example per page, however. So I think it's great that --help already has a couple of examples.
I think we should make it as easy as possible for casual users of cargo-bisect-rustc to use the tool. Because that means we are more likely to get help with bisecting in e.g. the context of issue triaging. The first time someone encounters the tool, I think the web documentation is probably a good entry point. But once they have used to tool a few times, they just need a brief refresher on how to use the tool. That they already have the tool installed I think is a sufficiently strong indication that they are not encountering the tool for the first time. And if they type --help, why whould we "hold back" information and force them to take one (or actually several, since the web docs are multi-paged) steps to see some examples? Examples are a great way to quickly demonstrate usage without a user having to read the docs for each individual option and figure out how to combine them.
And I think my example is actually a great example to have in --help, because it demonstrates something non-obvious but important, namely that that you can use --script bash -- -c "..." to unlock the use of grep for bisecting. Without having to create a separate .sh-script which also has the downside of requiring extra steps, which unnecessarily detracts users of the tool.

@Enselic I am also a great fan of offline documentation but at this time the offline docs for this tool are a bit clunky to use (see also #359). I agree that for now it's probably quicker to just update the online documentation.
But.
If your intent is to expand offline documentation, how about giving this tool a man page that replicates and expands the online help? I didn't investigate deeply but clap supports auto-generating man pages using the roff crate. Or like bat does.
Ideally a workflow that generates both offline and online help (can I dream? :slightly_smiling_face: )


Objects Posted with PreSignedPOST urls do not trigger events

Relevant packages and versions:
    "serverless-offline": "^5.10.1",
    "serverless-s3-local": "^0.5.0",

I managed to get Presigned URLs working with serverless-s3-local by manually installing<EMAIL_ADDRESS>The problem is now my events no longer trigger off of simple requests.
Example configuration to reproduce:
Handler configuration
...
    events:
      - s3:
          bucket: my-bucket
          event: s3:ObjectCreated:*
          rules:
            - prefix: uploads/
            - suffix: .mp4
          existing: true

Example code
const AWS = require('aws-sdk');
const fs = require('fs');
const request = require('request-promise-native');

const S3 = new AWS.S3({
  accessKeyId: 'S3RVER',
  endpoint: new AWS.Endpoint('http://localhost:9393'),
  s3ForcePathStyle: true,
  secretAccessKey: 'S3RVER',
});

function uploadFile() {
  console.log('Uploading file...');
  const genParams = {
    Bucket: 'my-bucket',
    Expires: 3600,
    Fields: {
      key: 'uploads/test/source.mp4',
      'Content-Type': 'video/mp4',
    },
  };
  S3.createPresignedPost(genParams, (err, result) => {
    if (err) {
      console.error('Failed to make, err=', err);
      return;
    }
    console.log('Pre-signed result:', result);
    // Upload the file
    const formData = {
      ...result.fields,
      file: fs.createReadStream('/path/to/video.mp4'),
    };

    const opts = {
      formData,
      uri: result.url,
      method: 'POST',
    };
    request(opts)
      .then((res) => console.log('Upload result:', res))
      .catch((err1) => console.error(err1));
  });
}

uploadFile();

As a sanity check, the following aws-cli command functions as expected, triggering my S3 event:
aws s3 cp --profile s3local --endpoint http://localhost:9393 /path/to/video.mp4 s3://my-bucket/uploads/test/source.mp4

Is there something I am able to do to continue complete local development or is this something that's not easily fixed/corrected with this module?
Thanks, let me know if there's any information I'm missing or forgot to provide, or if this issue really belongs in s3rver's issues list.

Upon further investigation, I've realized this is an issue better suited for s3rver. I discovered that the postObject method specifically doesn't trigger an event, so I will recreate my issue there.

@randonia
Thank for your comment! if your request has been merged into s3rver, I will make serverless-s3-local  use it.

Thanks @ar90n - I've got it in a PR on the main repo. Judging by their release cadence we should expect it sometime in the next week or three.


unexpected 'Engine'

Hello,
I'm trying to install flightphp to create an API endpoint, however when I try to access the page it gives me error 500 and the log says this:
PHP Parse error:  syntax error, unexpected 'Engine' (T_STRING), expecting function (T_FUNCTION) or const (T_CONST) in /var/www/html/list/flight/Flight.php on line 68
I am currently on:
PHP 7.3.31-1~deb10u1 (cli) (built: Oct 24 2021 15:18:08) ( NTS )
Copyright (c) 1997-2018 The PHP Group
Zend Engine v3.3.31, Copyright (c) 1998-2018 Zend Technologies
    with Zend OPcache v7.3.31-1~deb10u1, Copyright (c) 1999-2018, by Zend Technologies

I've used flightphp many times, but never had this issue

Line 68 is:
private static Engine $engine;

Using the example index file btw.

Just read the readme on github and it requires php 7.4 and I am on a lower on, however https://flightphp.com/ says 5.3, maybe update that @mikecao


upward hookshots from the ground can cause you to get launched to space

Fixed by disabling hookshot head collision for a bit at the start


get started jupyter notebook

Hy
I tryed out your exciting wrapper for scopus.
I found some small bugs and was able to correct them. As I work often in Jupyter notebook I copied the conted of the read me in a Get started notebook.
However the rendering of the reports could be improved.
Kind regards
Silas

The reports are not designed for the IPython notebooks. For them, you would want to write different report functions. The functions create org-mode formatted output.

I changed the name of the Notebook


Upgrade to jupyter-echarts 1.2.1

1.2.1 - 03.09.2017
Fixed
#. wrong pinyin(guandong) for 广东
#. 阿拉善盟地图 updated
1.2.0 - 02.09.2017
Updated
#. Reduced echarts maps by 0.1 MB by js minification.
#. Reduced echarts-china-cities-js by 0.1 MB by js minification.
Fixed
#. 阿拉善盟地图 failed to register with echarts

Codecov Report

Merging #153 into dev will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##              dev     #153   +/-   ##
=======================================
  Coverage   97.12%   97.12%
=======================================
  Files          66       66
  Lines        2224     2224
=======================================
  Hits         2160     2160
  Misses         64       64

Impacted Files
Coverage Δ

test/test_constants.py
100% <ø> (ø)
:arrow_up:

test/test_page.py
100% <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 78e7c15...e1c012b. Read the comment docs.

Please let me know when 0.2.4 is to released. I will upgrade github hosted jupyter-echarts to 1.2.1. Otherwise, newly built offline jupyter note books will fail to show maps.

@chfw
那这个 PR 先挂着吧，等我要发布就来 Merge。ok？

Now, you can release any time.


Please revise: GO:0075251 'uredospore formation' is_a 'biosynthetic process'

Hi,
GO:0075251 'uredospore formation'
is currently a subclass of
GO:0043936 asexual sporulation resulting in formation of a cellular spore
and
GO:0009058 biosynthetic process
(http://amigo.geneontology.org/amigo/term/GO:0075251, https://www.ebi.ac.uk/QuickGO/term/GO:0075251).
I suggest to delete the latter please. May have been a mistake as part of the obsoletion of 'single-organism biosynthetic process' (see Change Log in the QuickGO link above).
Thanks,
Paola

I will clean up the others too.


Knockout visible binding does not appear to work on root view element

Hello,
It appears that the knockout visible binding does not work correctly on the root element of a view. For example, if you update the viewComposition/index.html in Durandal.Samples.sln to the following, the view remains visible:

The Views Below Are Composed In Different Ways

This is a view part override of the default content....

It would be nice to know if this working as expected or if this is a bug.
Thank you!
Michael

If you don't want to use the wrappers (e.g. so your team members don't need to know about the bug), you can re-run the update method for Knockout's visible binding after the composition. It's still a workaround and takes a bit of code but it works well.


DataMuxer, revisited

Some long-planned updates to the muxer:

Rather than trying to convey upsampling and downsampling operations in column names (e.g., 'val_linear_mean'), all columns will be called 'val'. Instead, for each DataMuxer.bin_* function, there is now a DataMuxer.plan.bin_* function taking the exact same arguments. It returns an explanation of what resampling will be done for that given muxer object and function arguments. Example:
In [21]: dm.plan.bin_on('Tsam')
Out[21]:
          downsampling_needed upsampling_possible downsample upsample
Tsam                    False               False       None     None
point_det                True               False       None     None
time                     True               False       mean   linear

Binning operations return a DataFrame indexed by bin number, not time. A new time column contains the Event time.

Provide .to_sparse_dataframe(), allowing us to remove or refactor _dataframe in the future.

Remove the notion of bin anchoring, simplifying the bin functions considerably. Now that bins are indexed by number, there is no anchoring.

Simplify API: from dataportal import DataBroker, DataMuxer

N.B. This is passing on Travis, only failing coverage.

Looks good to me


Docs on pypi have internal links that don't work

The 2 links in "For comparison to pipsi, see how does this compare to pipsi? and migrating to pipx from pipsi" in the readme on pypi don't work, because pypi strips out the anchor tags in the real docs.
(I have no idea if this can actually be made to link correctly within pypi, but having the links show up there when they're not going to go anywhere is a bit confusing.)

It appears this is a limitation of the PyPI handling of long_description.  Here's a feature request with more information.

Nice research, @sahensley  :smile: Thank you for sharing.
Sort of a meta-question, but here goes: do people browse PyPI? I personally never search PyPI or read documentation on PyPI. I would be fine just replacing the README on PyPI with a pointer to the github README. Maybe that is just my bias though. Anyone thoughts?

I'll occasionally take a look on PyPI to get a quick overview of a project's maintainers, a link to the homepage, the project's license and review the release history.  My overall goal is to quickly check if a project is still maintained and what license it uses.

This seems to be fixed with the current pypi pipx README.


[libdatachannel ] Update to 0.12.0

Describe the pull request
Update libdatachannel to 0.12.0, it requires libjuice 0.7.1 update before merging this.
https://github.com/microsoft/vcpkg/pull/17076

Use ./vcpkg format-manifest ports/libdatachannel/vcpkg.jsonto make the pipeline happy :)

./vcpkg format-manifest ports/libdatachannel/vcpkg.json

Ok thanks,

Use ./vcpkg format-manifest ports/libdatachannel/vcpkg.jsonto make the pipeline happy :)
And after that, run ./vcpkg x-add-version libdatachannel and commit the result

Ok thanks, wasn't aware of that new thing.

Hi @Nemirtingas
Thanks for your PR.
Have you tested these features in this PR?

Hi @Nemirtingas
Thanks for your PR.
Have you tested these features in this PR?

Hi, I tested the compilation with this feature, but I didn't test it runtime.

LGTM now, thanks for your update. @Nemirtingas


Refactor greedy navigation

Summary
Introduces a new version of the greedy navigation component with greatly simplified code and improved behaviour.
Motivation for change
The greedy navigation component is one of our oldest components and was adapted from an external component. It's long been a pain point of the application as it is hard for the team to modify leading to patch work amends and redundant behaviour being maintained.
The main motivation for change is that the component uses the icon font for the toggle button and we'd like to get rid of this in favour of SVG icons for all components.
This requires change. The component is extremely hard to change so we should make the change easy by restructuring the component to only contain the behaviour we need and to match the rest of our components.
The old component also has a completely different API to the rest of our components so this is a good opportunity to make that consistent.
Improvements
Restructuring the component enables a number of improvements:

Switches the icon to an SVG version (the original motivation for change)
Improves keyboard and mouse support (tabbing and clicking in and out of the menu now behaves more consistently)
Minimises flash of unstyled content (fix ported from the public website)
Bundles welsh translations for UI text. Previously this had to be provided as part of the component initialisation (translations taken taken from the public website).
Fixes mistranslation of aria-labels (in production when translated they are part welsh part english)
Removed significant amounts of redundant behaviour that was brought over from the module we modified.

Keyboard support
The old component had partial keyboard support but it was patchy. You could trigger the menu via keyboard but in a non-default way by tabbing in (but only in one direction). The component used a mouse up event instead of a typical click so you couldn't control the button with the space bar like a regular button.
The new behaviour is:

The more button acts as a regular button and can be controlled with the space bar
When the navigation is open you can tab out of it in either direction and the navigation should close (existing behaviour but should work a bit more predictably in both directions)
Clicking anywhere outside the navigation should close the navigation (existing behaviour)
Pressing the escape key should close the navigation (existing behaviour)

Configuration free
The new component is configuration free. The old component provided a number of configuration options many of which were partially implemented or redundant.
The exception being label text which is now translated as part of the Rails template.
The new component is initialised like all our other components.
import { initNavigation } from '@citizensadvice/design-system/lib';
initNavigation();

Testing strategy
Previously we had a mix of tests split across jest and cypress. The jest tests ran assertions against internal methods rather than using testing library helpers. Using jest for component tests is limited as they have to work against static fixtures and don't run in a real browser. Our cypress tests run in a real browser and against real examples but previously only covered some behaviour but not the actual greedy nav logic.
I've taken the approach of keeping a basic jest test around to make sure the component isn't completely broken and then pushed the majority of the test scenarios into the cypress tests. These now cover the logic around moving items into the greedy navigation, keyboard interaction (within the limit of cypress' abilities), and translation behaviour.
Backwards compatibility
To make it easier to migrate to the new component I've kept the old entrypoint around which wraps the new implementation. This allows us to ship the new version and then remove the old entrypoint in a subsequent version.

The exploratory work for this was done in https://github.com/citizensadvice/design-system/pull/3264 in case you want to see the process. Depends on https://github.com/citizensadvice/design-system/pull/3260

I've brought this back up to date with the intention of merging in the new year.


Lower desired energy buffer for simulator

This simply makes testing using the simulator easier.

This Pull Request has been merged by GitConsensus.
Vote Totals

Yes
No
Abstain
Voters

4
0
0
4

Vote Breakdown

User
Yes
No
Abstain

tedivm
✔

Esryok
✔

dragoonreas
✔

screepas
✔

Vote Results

Criteria
Result

Has Quorum
True

Has Votes
True


oauth pkg: adding OAuthClient resources

@trewest @achuzhoy If you could please review, much appreciated

@achuzhoy

/lgtm


model not learning anything

been training images on my model with masks for earrings and not earrings.
dice_loss = sm.losses.DiceLoss()
focal_loss = sm.losses.BinaryFocalLoss()
total_loss = dice_loss + (1 * focal_loss)

opt = keras.optimizers.Adam(lr=0.001, beta_1=0.9, beta_2=0.999)

metrics = [sm.metrics.IOUScore(threshold=0.5), sm.metrics.FScore(threshold=0.5)]

model = sm.PSPNet(BACKBONE, encoder_weights = 'imagenet', classes = 1,
encoder_freeze=True, activation='sigmoid', downsample_factor=16, input_shape=(960,960,3),
psp_conv_filters=1024, psp_pooling_type='avg')

backbone is inceptionresnetv2 and i've set masks and images to 0...1 range.
the model trains but doesn't learn anything as its output is a matrix full of 0's.
it generated a mask when the input size was 380,380 but does not in this case.
could you point out any error i might be making? thanks in advance
PS: i could attach the colab file that i am using if you want to refer to something else.

Try to learn something with freeze_emcoder=False

in that case it does not make good predictions

i've set masks and images to 0...1 range.
Masks have values of either 0 or 1, and images have floating point values between 0 and 1?

yes @JordanMakesMaps
attaching the code.txt and there are segments which are not currently used in the model or i use them to test at times, they have been labeled as test or unused.
code.txt
please point me at the potential mistakes i might have been making

Okay @jayayy, I have some input that may or may not be useful but I think it would be worth trying out.
Starting at line 60:
for i in range(l):
  imgg = cv2.imread(path + '/'+lis[i])
  gray = cv2.cvtColor(imgg, cv2.COLOR_BGR2GRAY)
  gray = cv2.resize(gray,(960,960))
  mask[i] = np.expand_dims(gray,axis=2)
  #mask[i] = gray.reshape(960,960,1)
  if i%50==0:
      print(i)

Here you're taking in a mask that is currently in RGB format and converting it to Grayscale, then resizing and expanding so that the shape is correct for what you're trying to do.  I think the issue is that the mask needs to be in a binary format, not Grayscale. Read this for info on that (specifically, read "Representing the Task").
Once your masks are in a binary format, you have to one-hot-encode them. You can do this with your own script or use keras.utils.to_categorical().
As a sanity check, the final shape of your masks that you pass into training should be (batch_size, height, width, 2), the 2 comes from the fact that you're doing a binary segmentation. Even though you only have one class that you're looking at, you need to make it clear in your training data that there are really 2 classes, the one that you're interested in, and the background class.
The one-hot-encoded mask's last dimension when indexed should be (960, 960), and for the 0th index should have zeros where the class of interest is, and ones where the background is. The 1st index should be the opposite, where there are zeros where the background is, and ones where the class of interest is (see image below).

I would also recommend you to use the preprocess_input() function provided by @qubvel. Each  preprocess_input() is different but will preprocess in the input image in such a way to mimic how the original backbone was trained. So changing gears and looking at the code starting at line 119:
# data_gen_args = dict(featurewise_center=True,          # consider removing
#                      featurewise_std_normalization=True,     # consider removing
#                      rotation_range=90,
#                      width_shift_range=0.1,
#                      height_shift_range=0.1,
#                      zoom_range=0.2)
# image_datagen = ImageDataGenerator(**data_gen_args)
# mask_datagen = ImageDataGenerator(**data_gen_args)

You have featurewise_center and featurewise_std_normalization which alter the pixel values (other alternatives are rescale and zca_whitening) in an attempt to make is easier for the model to learn. The thing is, those two need to be fit on a the dataset (or a representative sample of the dataset) to be useful. Because you do not fit them, right now I don't think that those lines of code are doing anything for you. If you choose not to remove those to lines of code and instead find a way to make them work properly (see this) DEFINITELY create a different data_gen_args for your mask_datagen because featurewise_center and featurewise_std_normalization would mess up your masks values resulting in bad predictions.
The very last thing is the predictions:
path = 'C:\\Users\\Deepak\\Desktop\\ready\\test\\img'
lis = os.listdir(path)
imager = cv2.resize(cv2.imread(path +'/'+ lis[9]),(960,960))
imager = imager.reshape(1,960,960,3)
raw = model.predict(x=imager)
raw = raw.reshape(960,960)

You can change that, into:
path = 'C:\\Users\\Deepak\\Desktop\\ready\\test\\img'
lis = os.listdir(path)
imager = cv2.resize(cv2.imread(path +'/'+ lis[9]),(960,960))
imager = imager.reshape(1,960,960,3)
raw = model.predict(x=imager).squeeze() # Changing the shape back to (height, width, classes)
cv2.imshow(np.argmax(raw, axis = 0)) # We're finding the max index across the 0th dimension

Hope this helps 👍

so much of appreciation. also due to our frequent communication you seem to be a pen friend to me. ;-)
the masks are in 0 or 1, is that okay? also acc to binary classification example by @qubvel, i am pretty sure that i need only one mask showing the object of interest. i also previously raised an issue (dataset type for binary segmentation #221 ) for the same. nevertheless i still will try one-hot-encoding if that works.
also i was planning to use augmentation and will refer to your text then so your efforts will be pretty much useful then.
ill try the prediction part too after changing the maps.
thanks a lot

@jayayy you're right, I gave you conflicting responses 🤦♂ Sorry about that.

Images should be in range 0..255

@qubvel  your preprocess function makes images' values negative when i use inceptionresnetv2 backbone.
@JordanMakesMaps one hot encoder is no where to be used right here? and despite all, the problem is not solved yet. what am i doing wrong?

Looking at your code, this is where I would do it. But you would need to find a way to convert your RGB masks to some binary format.

path = 'C:\\Users\\Deepak\\Desktop\\ready\\train\\mask'
lis = ns.natsorted(subset_listt)
l = len(lis)
mask = np.zeros((l, 960,960,2)).astype('float')
print(path)

for i in range(l):
  imgg = cv2.imread(path + '/'+lis[i])
  gray = cv2.cvtColor(imgg, cv2.COLOR_BGR2GRAY) # shape = (H, W, 1)
  gray = cv2.resize(gray,(960,960)) # shape = (960, 960, 1)

  # find a way to convert masks to binary form here
  # you might need to use dictionaries to convert the RGB/Grayscale mask to binary mask
  # e.g. background == 0, class of interest == 1
  # then one-hot-encode them

  one_hot_encoded = keras.utils.to_categorical(binary_mask, nb_classes = 2) # shape = (960, 960, 2)
  mask[i] = np.expand_dims(one_got_encoded, ,axis=2) # shape = (1, 960, 960, 2)
  if i%50==0:
      print(i)

See the image below, it's taken from the cityscape dataset. The mask is in RGB format, so there are three channels. To one-hot-encode the masks, they first need to be converted to binary. For example, the car class here is dark-blue, which corresponds to (0, 0, 142) RGB; you could use a dictionary to convert all pixel values with this tuple to a single value for your binary mask.

color_converter = {(0, 0, 0) : 0, # background, ignore class
                              (0, 0, 141) : 1, # Cars
                              (128, 63, 127) : 2, # Road
                              (243, 36, 231) : 3} # Sidewalk

thanks for the reply.

i had previous png files with 3 channels and 255 for interested class and 0 for background. on converting them to grayscale, they became one channel 0 or 255 values (there's no other value except 0 or 255). now dividing these by 255 will give me binary values. so that part is clear.
i am a tiny bit afraid about the
you can close it if there is nothing more that i might be missing on. thanks for your further in depth inputs.


CI: Add TravisCI for code linting with flake8

Although this doesn't implement #16, it addresses the need for checking code style before merging in changes. This PR is not meant as an alternative to Black, but rather as an easy first step towards more comprehensive linting, while offloading the contributors from setting up anything on their end.

Gotcha. Yes, that works. Is there relevant fmriprep documentation about
"plugins" and how those work?
On Wed, Sep 25, 2019 at 3:44 PM Oscar Esteban<EMAIL_ADDRESS>wrote:

@oesteban commented on this pull request.
In setup.cfg
https://github.com/nipreps/dmriprep/pull/18#discussion_r328371098:

@@ -90,9 +90,9 @@ parentdir_prefix =
[flake8]
max-line-length = 99
doctests = True
-exclude=build/
+exclude=build/,/_afq/

Yep, I placed the old code under dmriprep/workflows/_afq/ for when we
reinclude stuff as an AFQ Plugin. Let me know if that's fine with you
all.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/nipreps/dmriprep/pull/18?email_source=notifications&email_token=AAA46NVOBIDWQUWOJURHTXLQLPSWRA5CNFSM4I2SZP2KYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCF6PZNA#discussion_r328371098,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAA46NRJWVAADG5IQ22INX3QLPSWRANCNFSM4I2SZP2A
.

Is there relevant fmriprep documentation about "plugins" and how those work?

That is an idea that sourced from @satra's brain and is gradually taking shape. In this case, I believe this "plugin" would be much easier if we just use BIDS-Derivatives as an interface since we would not need to modify dmriprep's execution graph, just add nodes.

I added a couple lines showing what changes would be necessary to switch to Black for formatting. But I don't think it's necessary to wait for a resolution of #16 to merge this PR. We can always switch over in the future.

Yup, I can see how black --check . tells you which files should be reformatted and exit with 1. However, for the travis build to be useful, then we would need to run black --diff ..
So I'd go ahead, and then add black in #16 (I like switching the pre-commit hook to optional). Possibly I would keep flake8 as enforced and allow the black tests to "fail"


IN operator not working on number multi value field

Description of the bug
When you have a multi value number field and you want to filter with contains or not contains the filter doesn't return anything. (It is working fine for the other fields, it looks like the problem is only with the number field)
Reproduction steps

Create a multi value number field
Go on the public page and create couple multi value records
Try to filter out with the contains filter
Notice that no records are returned

Expected behavior
You should be able to filter multi value number fields as you can with text or other fields.
Video
https://user-images.githubusercontent.com/53232926/211555503-26db0e8d-be6e-4e79-8d1c-0b9550143935.webm
Versions
Branch: 2023.3.x and 2022.9.x

Fixed with #776

Just retested this and can confirm that this is now fixed. @vicpatel just when you can try to add the CL entry.

Fixed
What:

Fixed filtering for multi-value number field with contains or not contains criteria.

How:

By making sure to convert numbers into string format while building a query for the selected filter.


Yield monitor data requirements

I was provided an example yield monitor data file by Bryan and it had a very rich data schema.  Here is the DBF file field definition
Field 0: Type=N/Double, Title=`DISTANCE', Width=18, Decimals=8
Field 1: Type=N/Double, Title=`SWATHWIDTH', Width=18, Decimals=8
Field 2: Type=N/Double, Title=`VRYIELDMAS', Width=18, Decimals=8
Field 3: Type=N/Integer, Title=`SECTIONID', Width=9, Decimals=0
Field 4: Type=N/Integer, Title=`Crop', Width=5, Decimals=0
Field 5: Type=N/Double, Title=`WetMass', Width=18, Decimals=8
Field 6: Type=N/Double, Title=`Moisture', Width=18, Decimals=8
Field 7: Type=C/String, Title=`Time', Width=24, Decimals=0
Field 8: Type=N/Double, Title=`Heading', Width=18, Decimals=8
Field 9: Type=C/String, Title=`VARIETY', Width=3, Decimals=0
Field 10: Type=N/Double, Title=`Elevation', Width=18, Decimals=8
Field 11: Type=C/String, Title=`IsoTime', Width=24, Decimals=0
Field 12: Type=N/Integer, Title=`Machine', Width=9, Decimals=0

When I look at the same CSV files provided on the Box drive, I find a much less rich dataset
harvest_year
harvest_month
record
x
y
swath
mass

In particular for pyritas, those Time and Heading values would be extremely valuable to an algorithm attempting to resolve where the combine is at in the field.  Why aren't these fields included with the cybox files?

Thanks @jarad for the response, lets loop @andyvanl and @bpetersen18 on this discussion.  My opinion is that if we currently know the least common denominator for data attribute availability from the datasets we need to process, lets not make things harder on ourselves than it should be by only using x, y, swath and mass.

As of right now, all of our miscanthus yield monitor data that need to be cleaned will have all those fields listed by @akrherz in the first message.

I wonder if @andyvanl has access to other yield monitor data, e.g. C-CHANGE data, ISU research/production farms, etc.

@mattdstep may also have access to C-CHANGE yield monitor data

@mattdstep may also have access to C-CHANGE yield monitor data

I do not currently have any yield monitor data, although I do expect the Horizon II project will produce some for inclusion in SILPH, if not the Grass2Gas project.

@akrherz - we need to loop in Marissa and Destiny Williams.  They both have YM data and it will likely be a mix of the STRIPS type data and the Commercial Miscanthus data.  My preference would be that RITAS be a living thing that we can keep improving.  With multiple tiers of analysis.  One that works on the minimal data fields like the STRIPS data, and then progressive levels from there.  I also think we should add Fernando Miguez to this effort as well.  He has massive amounts of YM data from research projects an will certainly have a valuable perspective when he has time to provide it.


OpenTelemetry exporter installation error

Package Name: azure-monitor-opentelemetry-exporter
Package Version: 1.0.0b28
Operating System: Windows
Python Version: 3.12

Describe the bug
Installing the above package automatically installs opentelemetry-sdk~=1.26.0 and opentelemetry-semantic-conventions 0.47b0, which will cause incompatibility issues later on.
Error message:
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
opentelemetry-exporter-otlp-proto-grpc 1.24.0 requires opentelemetry-sdk~=1.24.0, but you have opentelemetry-sdk 1.26.0 which is incompatible.
opentelemetry-instrumentation-asgi 0.45b0 requires opentelemetry-semantic-conventions==0.45b0, but you have opentelemetry-semantic-conventions 0.47b0 which is incompatible.
opentelemetry-instrumentation-fastapi 0.45b0 requires opentelemetry-semantic-conventions==0.45b0, but you have opentelemetry-semantic-conventions 0.47b0 which is incompatible.

To Reproduce
Steps to reproduce the behavior:

pip install azure-monitor-opentelemetry-exporter

Expected behavior
Package successfully installed.
Additional context
I am able to install version 1.0.0b24.

@TaoChenOSU
These dependency conflicts are issues in upstream OpenTelemetry Python and unrelated to the azure monitor exporter. Please open up an issue here and someone from the community will get back to you.
As for identifying the actual dependency conflicts, I've addressed them below:

opentelemetry-exporter-otlp-proto-grpc 1.24.0 requires opentelemetry-sdk~=1.24.0, but you have opentelemetry-sdk 1.26.0 which is incompatible.

The issue lies in the otlp exporter dependency on opentelemetry-sdk. The dependency should be opentelemetry-sdk ~= 1.26 instead of opentelemetry-sdk ~= 1.26.0

opentelemetry-instrumentation-asgi 0.45b0 requires opentelemetry-semantic-conventions==0.45b0, but you have opentelemetry-semantic-conventions 0.47b0 which is incompatible.
opentelemetry-instrumentation-fastapi 0.45b0 requires opentelemetry-semantic-conventions==0.45b0, but you have opentelemetry-semantic-conventions 0.47b0 which is incompatible.

Simply upgrade your opentelemetry-instrumentation-asgi and opentelemetry-instrumentation-fastapi to versions 0.47b0 for each.


Change Pulsar to run if user has PINT, but not libstempo

Currently the user gets a printed message that says 'ERROR: Must have libstempo package installed!' if the import call for libstempo raises an ImportError exception. There are a number of user types who do need libstempo installed, including those dealing with gamma-ray or x-ray timing data, since libstempo (really TEMPO2) can't deal with these types of TOAs.
It looks like the only change to the working code would need to be here and would just be changing:
    if pint:
        toas = list(filter(lambda x: isinstance(x, toa.TOAs), args))
        model = list(filter(lambda x: isinstance(x, TimingModel), args))

    t2pulsar = list(filter(lambda x: isinstance(x, t2.tempopulsar), args))

to
    if pint:
        toas = list(filter(lambda x: isinstance(x, toa.TOAs), args))
        model = list(filter(lambda x: isinstance(x, TimingModel), args))
    if tempo2:
        t2pulsar = list(filter(lambda x: isinstance(x, t2.tempopulsar), args))

Besides that we'd just add a truthful error message. Something like: raise ValueError('Must have either PINT or libstempo installed.')
Does anyone (@vallis @paulthebaker @stevertaylor ) know of other places in the code that might require libstempo/TEMPO2?

This is a good idea.  You should also change the import behavior to give warnings if either is missing.
I don't think PINT or libstempo is used outside of loading Pulsar objects.


Implement map projection

Adds support for openCypher Map Projection specification.

@rafsun42 there are warnings in the build...

@MuhammadTahaNaveed
These warnings do not appear with CFLAGS='-O0'. I will switch to -Og and fix the code.


chore: Versioned Alloy Dependencies

Description
Use versioned alloy dependencies.

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

Project coverage is 79.4%. Comparing base (916339a) to head (d8de339).

Additional details and impacted files

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


The admin is able to create a bundle of products without adding products to the bundle products.

Bagisto Version: Master
Steps To reproduce--
1. Log in as an admin.
2. Click on products inside the catalog section.
3. Create a bundle product.
4. Do not add any product to this bundled product.
5. Visit the shop.
6. Try to add to the cart the created product.
Expected result: The bundle items field should be required if the admin is creating the bundle product.
Actual result: The admin is able to create a bundle of products without adding products to the bundle products. Please find the screenshot Here

@jitendra-webkul
As we discuss this is not an issue so I am closing this issue.


Pruning mode

Something similar to https://github.com/ledgerwatch/erigon/pull/2399

may I take this one after i finish unwinds for indexes?

may I take this one after i finish unwinds for indexes?

Yes, sure.


Using the processBeforeResponse flag

Hi @seratch, I am building a slack app on cloud functions (for now... cold starts are annoying...) and I think I should be using the processBeforeResponse as outlined in  https://github.com/slackapi/bolt-js/pull/444 but having issues figuring out how to use it. Is it as easy as just adding the option to my App constructor? Do I need to move by ack() calls to the end of my handler functions or can I keep them at the beginning?

Thanks for flagging this! Yes, this example should be updated.

Is it as easy as just adding the option to my App constructor?

Yes, it is. If you initialize ExpressReceiver manually, you need to set the flag to both App and ExpressReceiver.

Do I need to move by ack() calls to the end of my handler functions or can I keep them at the beginning?

No, you don't. When you turn on the option, your app waits for the whole completion of a listener function before sending an HTTP response to Slack regardless of the position of ack() calls.

Thanks! that worked fine!

I've updated this example to be up-to-date. Let me close this issue now.


Adding env var does not set host env var

let-env FOO = "bar" sets the env var in the Stack but seems like it does not set it for the host. This probably breaks ls_colors which relies on the LS_COLORS env var, probably read from std::env.
The solution: Inside the Stack::add_env_var(), add the env var also into the host environment with std::env
Alternative: Make ls_colors (and any other cases where this is an issue) read Nushell's env var instead

my vote is for everything the gets or sets environment variables to somewhere go through std::env.

Isn't that something that has already being discussed, that the host environment variables should not be touched

Yeah, I think we should load all env vars to engine-q on startup, but then use exclusively the e-q's env vars. I already changed LS_COLORS to behave that way in the PR above, so I'll close this issue.


BMT: Admin module login has disapeared

The top menu login for admin has disapeared.

Same in CI / QA / PROD, but still visible in radix dev.

Fusion upgraded the msal implementation two weeks ago.

The problem:
/fusion-bmt/frontend/src/views/Project/ProjectTabs.tsx

Assigning myself to this.

A pull-request has been created for this.
https://github.com/equinor/fusion-bmt/pull/1002
Moving case to "Review"

A new Pull-Request has been created for this.
https://github.com/equinor/fusion-bmt/pull/1003


Add "Default" AWS credentials object

Describe the Feature
When creating an AWS Clients you are currently able to use the default credential search options by not specifying any credential object.  What I would like to see is a new credential object that follows the AWSCredentials base class.
Is your Feature Request related to a problem?
Not having this is currently causing issues for us where we are trying to use a function to create the credentials object but in some cases we want to fall back on an EC2 Instance Role and currently this involves either catching an error or using an extra check outside of that function.
Additional Context
A quick sudo code example of what we are doing is:
if ( awsProfile != Null || awsAccessKey != Null){
AWSCredentials awsCredentials = GetAWSCredentials(awsProfile, awsAccessKey)
client = new AmazonCloudFormationClient(awsCredentials,region)
} else {
client = new AmazonCloudFormationClient(region)
}

[ ] :wave: I may be able to implement this feature request
[ ] :warning: This feature might incur a breaking change

This is a :rocket: Feature Request

Are you asking for something similar to the DefaultAWSCredentialsProviderChain in Java?

Yeah that looks to be exactly what I would want.

I would like to upvote for this then

@grbartel Just curious whether Amazon.Runtime.FallbackCredentialsFactory.GetCredentials() fits the need. Could you please check?

@ashishdhingra This looks promising, I'll try to make time during this upcoming week to try it out


A VecCheckNan issue

when I enable the Nan Check env
 env = VecCheckNan(env, raise_exception=True)

the system will have a error like:
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/stable_baselines/ppo2/ppo2.py", line 318, in learn
    runner = Runner(env=self.env, model=self, n_steps=self.n_steps, gamma=self.gamma, lam=self.lam)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/stable_baselines/ppo2/ppo2.py", line 442, in __init__
    super().__init__(env=env, model=model, n_steps=n_steps)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/stable_baselines/common/runners.py", line 19, in __init__
    self.obs[:] = env.reset()
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/stable_baselines/common/vec_env/vec_check_nan.py", line 46, in reset
    self._check_val(async_step=False, observations=observations)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/stable_baselines/common/vec_env/vec_check_nan.py", line 58, in _check_val
    has_nan = any(np.isnan(val))
ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()

And I find that is cause by 2d array like following test:
import numpy as np

#val = np.array([0, 0, 1, 0]) # it is run ok
val = np.array([[0, 0, 1, 0], [0, 0, 1, 0]]) # it is run same error as above
has_nan = any(np.isnan(val))
print(has_nan)

and the stable-baselines at vec_check_nan like this:
def _check_val(self, *, async_step, **kwargs):
        # if warn and warn once and have warned once: then stop checking
        if not self.raise_exception and self.warn_once and self._user_warned:
            return

        found = []
        for name, val in kwargs.items():
            has_nan = any(np.isnan(val))
            has_inf = self.check_inf and any(np.isinf(val))
            if has_inf:
                found.append((name, "inf"))
            if has_nan:
                found.append((name, "nan"))

To fix this problem, I think that need change  from 'any' to 'np.any'.
System Info
Describe the characteristic of your environment:

Describe how the library was installed (pip)
GPU models and configuration Mac
Python version 3.6.8
Tensorflow version 1.4.0

Hello,
Thanks for reporting the bug.
In fact there was an (incomplete) PR for solving it: https://github.com/hill-a/stable-baselines/pull/386
We would appreciate if you could complete it ;) (especially writing a test)

Ok I just add test and modify the code, please review
https://github.com/hill-a/stable-baselines/pull/491


Piledriver (and possibly some other maps?) prevent proper server info display

Greetings,
During my server's first playtest I've noticed that server infos do not display on BLREdit as per the following pic:

I suspect this is due to Piledriver being unexpectedly named (and reported in the server_infos endpoint) as piledriver1, as it shows up in the logs after each refresh.
Details
BLREdit version: d5e5d08
BLREdit log: log.txt
server_info JSON (with a .txt extension to make GitHub happy): server_info.txt

Thank you!


Feature icon PHP for Interface

Housekeeping

[X] I'm sure this issue is not a duplicate

Information:
Hi guys!
Would it be possible to create an icon for the "feature" Interface PHP? It could be like the "generic" icon.
Thanks!

We need some more information. Can you fill out the 'Icon Request' template?


Request to transfer java-tracerresolver under opentracing-contrib

@bhs
Can we create a 'java-tracerresolver' under opentracing-contrib to host the ServiceLoader integration that was in the contrib GlobalTracer but has not made it into the util package?
I'd like to transfer the following repo: https://github.com/sjoerdtalsma/java-tracerresolver
See discussion in https://github.com/opentracing-contrib/java-globaltracer/issues/7#issuecomment-292228434
cc: @objectiser

@sjoerdtalsma done: https://github.com/opentracing-contrib/java-tracerresolver/invitations
The name is a mouthful, but I wasn't able to think of an alternative suggestion I actually like. :)


chore(ci): try new builers

rename yaml->yml
update actions versions
Signed-off-by: Ry Jones<EMAIL_ADDRESS>

@petermetz I renamed that one file back to .yml

Hi @ryjones I think you missed updating runner for .github/workflows/test_weaver-data-sharing.yaml, because the runner that used there was buildjet-4vcpu-ubuntu-2204. I don't know if this was intentional, hence asking you here.


Feature/performance transport

What does this PR do?

Supports TLS connections
Closes Idle proxy connections after some time
Improves the performance and monitoring of the transport and round tripper.
Adds some more doc blocks
Adds a new go report badge

:shipit:


button-icon-only class incorrectly added to button that contains data binding

Short description of the problem:
Buttons that have data binding inside of them ({{'data'}}) and an icon incorrectly get the button-icon-only class.
Steps to reproduce:
<button round outline small>
    <ion-icon name="call-outline"></ion-icon>
    {{'testing'}}
</button>

Results in the following rendered html:
<button outline="" round="" small="" class="disable-hover button button-outline button-round button-small button-icon-only" ng-reflect-small="" ng-reflect-outline="" ng-reflect-round=""><span class="button-inner">
          <ion-icon name="call-outline" role="img" class="ion-ios-call-outline" aria-label="call outline" ng-reflect-name="call-outline"></ion-icon>
          testing
        </span><ion-button-effect></ion-button-effect></button>

Notice the button-icon-only class that should not be there.
Which Ionic Version? 1.x or 2.x
Ionic 2.0.0 beta 9
Plunker that shows an example of your issue
http://plnkr.co/edit/jaMbkDfqCurh9TiNAX1E?p=preview

@adamdbradley @jgw96 confirmed this is an issue. we have should refactor the Button class to start a more "angularish" approach by using @ViewChild etc. right now. it does not work if we add/remove icons/text dynamically.

This is not longer an issue since you have to specify manually when a button is "icon-only"
<button ion-button icon-only>
</button>


Publish vendor nothing

When i run php artisan vendor:publish --provider="Benwilkins\FCM\FcmNotificationServiceProvider" it gives me an error Nothing to publish for tag []. how can i fix this?

I am having the same error, Help!

same here
laravel 5.5

Updated documentation. You no longer need to publish the config.
Add your Firebase API Key in config/services.php.
return [

    ...
    ...

    /*
     * Add the Firebase API key
     */
    'key' => ''
];


Records changed on-line will not be overwritten by a file reload

Please add for discussion.
We will still need to be able to replace a PIECE online if an error has been reported with the Transcription and we have amended the VLD.
Currently I send these to Lemon on a 4 weekly basis stating that they are amendments so that he can replace it with the one already online.
This could be arranged to be done at COORD level as we used to be able to!
From: PatReynolds<EMAIL_ADDRESS>Sent: Tuesday, January 26, 2016 6:44 PM
To: FreeUKGen/FreeCENMigration<EMAIL_ADDRESS>Subject: [FreeCENMigration] Records changed on-line will not be overwritten by a file reload (#91)
—
Reply to this email directly or view it on GitHub https://github.com/FreeUKGen/FreeCENMigration/issues/91 .

Requirement for new system.

@Captainkirkdawson to take a look and explore the required specification here.

This is easily done through a file  or piece of similar lock.
However the first set in moving forward with any of these actions is the assembly of a small team of coordinators/ coordinators/ transcribers who will drive forward the migration of Cen2 into a direct build system. No actions should be programmed until such a team exists to verify and validate the approaches.

The locking system is in place on test3


[rust] migrate to stable toolchain

Noew that 1.39.0 has stabilized, remove the rust-toolchain file and
configure ci to use the stable toolchain.

:hourglass: Testing commit 9852eb3916fedcd46d476f41945e5f2e01e8b2f8 with merge da365e1bad49eac6e4688b2bd921fdef77f47e2e...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: metajack
Pushing da365e1bad49eac6e4688b2bd921fdef77f47e2e to master...


Problem with subclassing example?

I've been trying to use attrs in a simple subclass setup shown in the examples.
I thought I had used this functionality a while ago. But now I get a TypeError even when trying to run the example in the [documentation] (https://www.attrs.org/en/stable/examples.html) using version 0.23.2 or 0.23.1 with Python 3.10.11 (haven't tried other Python versions).
I'm getting a TypeError that A's init takes 2 arguments but 4 were given.
Checking MRO, the A and B subclasses show up in the expected order.
Is this happening for others?

Turns out my installation was weirdly broken. Rebuilding my environment fixed this.