When running dbt-meshify split, get exception "Unable to find a docs block with the name int_hcss_dis_jobs_doc" even though doc block exists

Describe the bug
When running dbt-meshify split, get exception "Unable to find a docs block with the name int_hcss_dis_jobs_doc" even though doc block exists
Steps to reproduce
Using Codespaces, ran "dbt-meshify split dm_accounting_pipeline --select +dm_accounting"
Expected results
Expected to split our +dm_accounting pipeline into a new project
Actual results
Got an exception error:
Exception: Unable to find a docs block with the name int_hcss_dis_jobs_doc.
Screenshots and log output
codespace@codespaces-75b401:/workspaces/dbt_cloud$ dbt-meshify split dm_accounting_pipeline --select +dm_accounting
15:12:12 | INFO | Executing dbt parse...
15:12:40 | INFO | Generating catalog with dbt docs generate...
Traceback (most recent call last):
  File "/home/codespace/.local/bin/dbt-meshify", line 8, in <module>
    sys.exit(cli())
  File "/home/codespace/.local/lib/python3.10/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/home/codespace/.local/lib/python3.10/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/home/codespace/.local/lib/python3.10/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/codespace/.local/lib/python3.10/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/codespace/.local/lib/python3.10/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/home/codespace/.local/lib/python3.10/site-packages/click/decorators.py", line 33, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/home/codespace/.local/lib/python3.10/site-packages/dbt_meshify/main.py", line 232, in split
    project = DbtProject.from_directory(
  File "/home/codespace/.local/lib/python3.10/site-packages/dbt_meshify/dbt_projects.py", line 292, in from_directory
    return DbtProject(
  File "/home/codespace/.local/lib/python3.10/site-packages/dbt_meshify/dbt_projects.py", line 315, in __init__
    self.jinja_blocks: Dict[str, JinjaBlock] = self.find_jinja_blocks()
  File "/home/codespace/.local/lib/python3.10/site-packages/dbt_meshify/dbt_projects.py", line 326, in find_jinja_blocks
    blocks[unique_id] = JinjaBlock.from_file(
  File "/home/codespace/.local/lib/python3.10/site-packages/dbt_meshify/storage/jinja_blocks.py", line 63, in from_file
    start, end = cls.find_block_range(file_content, block_type, name)
  File "/home/codespace/.local/lib/python3.10/site-packages/dbt_meshify/storage/jinja_blocks.py", line 36, in find_block_range
    raise Exception(f"Unable to find a {block_type} block with the name {name}.")
Exception: Unable to find a docs block with the name int_hcss_dis_jobs_doc.

System information
Which database are you using dbt with?

[ ] postgres
[ ] redshift
[ ] bigquery
[x] snowflake
[ ] other (specify: ____________)

The output of dbt debug:
codespace@codespaces-75b401:/workspaces/dbt_cloud$ dbt debug
15:17:19  Running with dbt=1.7.13
15:17:19  dbt version: 1.7.13
15:17:19  python version: 3.10.12
15:17:19  python path: /usr/local/bin/python
15:17:19  os info: Linux-6.2.0-1019-azure-x86_64-with-glibc2.31
15:17:19  Using profiles dir at /workspaces/dbt_cloud/.dbt
15:17:19  Using profiles.yml file at /workspaces/dbt_cloud/.dbt/profiles.yml
15:17:19  Using dbt_project.yml file at /workspaces/dbt_cloud/dbt_project.yml
15:17:19  adapter type: snowflake
15:17:19  adapter version: 1.7.1
15:17:20  Configuration:
15:17:20    profiles.yml file [OK found and valid]
15:17:20    dbt_project.yml file [OK found and valid]
15:17:20  Required dependencies:
15:17:20   - git [OK found]

15:17:20  Connection:
15:17:20    account: fc41459
15:17:20    user: dylanmorrish
15:17:20    database: ANALYTICS_DW
15:17:20    warehouse: COMPOSE_WH
15:17:20    role: SYSADMIN
15:17:20    schema: PROD
15:17:20    authenticator: None
15:17:20    private_key_path: None
15:17:20    token: None
15:17:20    oauth_client_id: None
15:17:20    query_tag: None
15:17:20    client_session_keep_alive: False
15:17:20    host: None
15:17:20    port: None
15:17:20    proxy_host: None
15:17:20    proxy_port: None
15:17:20    protocol: None
15:17:20    connect_retries: 1
15:17:20    connect_timeout: None
15:17:20    retry_on_database_errors: False
15:17:20    retry_all: False
15:17:20    insecure_mode: False
15:17:20    reuse_connections: None
15:17:20  Registered adapter: snowflake=1.7.1
15:17:20    Connection test: [OK connection ok]

15:17:20  All checks passed!

The output of dbt --version:
Core:
  - installed: 1.7.13
  - latest:    1.7.13 - Up to date!

Plugins:
  - sqlserver: 1.7.4 - Up to date!
  - snowflake: 1.7.1 - Update available!
  - fabric:    1.7.4 - Update available!

  At least one plugin is out of date or incompatible with dbt-core.
  You can find instructions for upgrading here:
  https://docs.getdbt.com/docs/installation

Additional context
Are you interested in contributing the fix?
Nope

I have a similar issue on Athena, while creating a group and assiging an owner. I did the same tests as the OP.

@dmoditto @Hansk-12 thanks so much for the report! we may need a bit more info to debug here. Would each of you mind providing:

the filepath (including whether or not this comes from a package!) of where the failing doc block is defined
the content of that entire file (you can scrub any sensitive docs, we care mostly about the blocks of jinja in the file)
the name of the failing block (which we have in your logs above @dmoditto!)

hopefully that will be enough for us to get to a root cause here! appreciate your patience!

@dave-connors-3 sure thing! Below are my answers. In gathering that info, I was able to narrow down further what is going wrong. Seems like whatever is parsing the doc block header doesn't like the two cases I outline below.

/dbt_cloud/models/marts/accounting_gl/integrations/acnt_integrations.md

{% docs int_hcss_dis_jobs_doc%}

The 'int_hcss_dis_jobs' view is used to udpate HCSS HeavyJob via DIS for Construction and PRG Jobs from JDE F0006 Job Master.

{% enddocs %}

The Case of the Missing Space
In taking a look at the doc block again, I actually found a change that fixes this: simply ensure there is a space at the doc block tag. So {% docs int_hcss_dis_jobs_doc%} changed to {% docs int_hcss_dis_jobs_doc %} fixes the issue.
Unfortunately, it turns out we have numerous doc blocks that lack that final space. So, I'm off to find and replace all those suckers.
An Odd Dash
One other case I found is with the following doc block:
{% docs snowflake_-_helper_dbt_doc %}

https://github.com/jeremiahhansen/snowflake-helper-dbt

{% enddocs %}

And it is all good when I remove the oddly placed dash
{% docs snowflake_helper_dbt_doc %}

https://github.com/jeremiahhansen/snowflake-helper-dbt

{% enddocs %}

I'll add any more cases if I find any, but hope these two help point towards a root cause and resolution!

Case 3: A Missing Space: Part 2 The Macro Edition
{% macro create_schema_yml_with_sf_tags(
        model_name,
        schema_name,
        database_name = target.database,
        generate_columns = True,
        column_names = []
    ) %}

Adding a space between the name of the macro and the first parenthesis fixes this issue:
{% macro create_schema_yml_with_sf_tags (
model_name,
schema_name,
database_name = target.database,
generate_columns = True,
column_names = []
) %}

@dmoditto Aha! this makes a lot of sense -- we are using some pretty unscientific regex matching to move jinja blocks, and we're likely simply not covering all the valid edge cases of space and special character placement! should be a relatively quick fix! cc: @nicholasyager

I am facing this issue, please when will the above fix be released?


[795] School can order + awaiting on TechSource

Context

https://trello.com/c/mZTOpn2R/795-build-techsource-account-not-yet-ready-page

Changes proposed in this pull request

Minor copy changes and ui tweaks

Guidance to review

Login as a school user who can order_devices true but techsource_account_confirmed_at nil
Make the school allowed to order devices
As school user attempt to order devices
Should see copy related to TechSource account will be ready soon

Is this missing the updated title? I see no new line in en.yml.
Title should be 'Order devices soon'

Is this missing the updated title? I see no new line in en.yml.
Title should be 'Order devices soon'

Good spot. I've made a tweak and also added a screenshot the PR


ORA-12541: TNS:no listener

Hi,
I can connect database with sqldeveloper.
But when I try to connect with doctrine (database ORM for symfony and PHP), I have the following error :
An exception occured in driver: ORA-12541: TNS:no listener
Thanks for any help

What connect string are you using?
This works for me
docker run -d -p 8022:22 -p 1521:1521 -e ORACLE_ALLOW_REMOTE=true wnameless/oracle-xe-11g
sqlplus system/oracle@localhost/XE

Hi,
Don't worry, this works now for me too.
Sorry, can't remember what's going wrong with my config, but now its ok.

Hi,
Don't worry, this works now for me too.
Sorry, can't remember what's going wrong with my config, but now its ok.

Hi! I'm having the same problem, what did you do to fix it? Thanks!!!


Fix feature list and dependencies for package axom.

Summary

This PR is a bugfix: see https://github.com/LLNL/axom/issues/288
It allows Windows TPL and src to build in a freshly-pulled repository.
This PR does the following:

Changes the vcpkg CONTROL file for the axom package so build dependencies and features are properly specified
Sets the default for the source build to 64-bits, to match the TPLs

Thanks @agcapps.
Does this imply that, for Windows, Axom will depend on hdf5 when conduit does?

Yes, because the vcpkg CONTROL file for the conduit package lists hdf5 as a build dependency.


send a command to all buildkite agents

This is more a question than an issue:
How can a command step be triggered on all available agents (or a selected subset)? E.g. some daily cleanup task?

Not at present @MMesch! It's something we've discussed a little bit though. I might close this one for now, but if you wanted to raise an issue with your usecase in https://github.com/buildkite/feedback we'll keep you updated on progress.


Typo in EventHubsConnectionStringBuilder Sample Code

There is a typo in the Sample Code
It says:
var connectionStringBuiler = new EventHubsConnectionStringBuilder(
Is should be:
var connectionStringBuilder = new EventHubsConnectionStringBuilder(

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

ID: b5a8afbe-024a-eff4-d738-35d3c25f851b
Version Independent ID: 43ab484f-1dd3-de05-6eb7-a4df54e42f74
Content: EventHubsConnectionStringBuilder Class (Microsoft.Azure.EventHubs) - Azure for .NET Developers
Content Source: xml/Microsoft.Azure.EventHubs/EventHubsConnectionStringBuilder.xml
Service: event-hubs
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

Thank you for your feedback.  Tagging and routing to the team best able to assist.


[ONNX] Average and Max Pooling

@rkimballn1 @diyessi Please note this line: https://github.com/NervanaSystems/ngraph/pull/1489/files#diff-2fb021160894001e71f8ceffd572efbbR51
Let me know if you're okay with adding this default value.


Preface for Japanese reader

Depending on the country or culture, there may be differences in the issues that people are paying attention to, so it might be helpful to have text that serves as a cushion for those individuals. I tried writing a manuscript tailored to Japan.

Japan is currently in the midst of unprecedented social change. With the declining birthrate and aging population, labor shortage has become a serious problem. This has led to growing expectations for automation technologies such as AI and robotics. The efficiency and productivity improvements brought about by these technologies are essential for Japan's future economy.
However, in the current social structure where the elderly make up a large proportion of the population, democratically elected politicians tend to prioritize the welfare of the elderly over the needs of the working-age population. Perhaps due to the generational conflict of interest between the future benefits of investing in science and technology and the current benefits of elderly welfare, there is a growing sense of disappointment in democracy, particularly among the younger generation, leading to a decline in voter turnout. In order for technological innovation to bring happiness to everyone in society, careful deliberation from the perspectives of all people is necessary on how to apply technology.
This book focuses on the intersection of technology and democracy in this situation, exploring how balanced progress can be achieved. Along with concrete examples from around the world, it discusses how technology can foster cooperation in society. Through this dialogue, readers will gain insights into the roles they should play in both the current and future Japanese society.

I welcome any comments!

 Gov4Git notices
On Friday, 19-Apr-24 17:47:04 UTC by Gov4Git dev
Notice bxge4f
Started managing this issue, using the Plural Management Protocol v1,  as Gov4Git concern 895 with initial priority score of 0.000000.
This project is managed by Gov4Git, a decentralized governance system for collaborative git projects.
To participate in governance, install the Gov4Git desktop app.
Notice ftgc5x
This issue's priority score is now 0.000000.
The cost of priority is 0.000000.
The projected bounty is now 0.000000.
Notice dlb73p
The set of eligible proposals claiming this issue is empty.

 Gov4Git notices
On Monday, 01-Jul-24 18:56:18 UTC by Gov4Git dev
Notice zcnbr7
This issue's priority score is now 14400.000000.
The cost of priority is 14400.000000.
The projected bounty is now 14400.000000.
Notice iwdaao
The set of eligible proposals claiming this issue is empty.


Addition of Lab (task 4) for multitenant lifecycle management workshop (id 656)

Add here your WMS ID 656
General requirements

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

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

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

some additional changes are required to integrate multiple pull requests into one.. hence cancelling this pull request, will raise a new one


fix(ci): Change the logic on junit upload job

First iterate on all xml to sort them by team
Then call upload on each team folder
Added tests
Muted all unittest that logged in stdout, pending a logger
implementation

What does this PR do?
The initial logic was the following:

Extract the tarball
For each xml on it, split by owner

For each splitted xml, call uploader

As a result there was no parallelism at all leading to a duration of 20min to upload files for the kmt_arm64_junit_upload job.
This PR changes the logic of the job:

Extract the tarball
For each xml, split by owner-flavor (for uniform tagging)
Then, for each owner-flavor folder, call the uploader (with all the files at once) using a thread pool.

For the unit-tests jobs it moved from 102 calls to 31 only.
Before:

After:

For the specific kmt_arm64_junit_upload job

Motivation
Job efficiency

Additional Notes
I took advantage of this PR to

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/trigger-ci --variable RUN_ALL_BUILDS=true --variable RUN_KITCHEN_TESTS=true --variable RUN_E2E_TESTS=on --variable RUN_UNIT_TESTS=on --variable RUN_KMT_TESTS=on

/trigger-ci --variable RUN_ALL_BUILDS=true --variable RUN_KITCHEN_TESTS=true --variable RUN_E2E_TESTS=on --variable RUN_UNIT_TESTS=on --variable RUN_KMT_TESTS=on

/trigger-ci --variable RUN_ALL_BUILDS=true --variable RUN_KITCHEN_TESTS=true --variable RUN_E2E_TESTS=on --variable RUN_UNIT_TESTS=on --variable RUN_KMT_TESTS=on

/trigger-ci --variable RUN_ALL_BUILDS=true --variable RUN_KITCHEN_TESTS=true --variable RUN_E2E_TESTS=on --variable RUN_UNIT_TESTS=on --variable RUN_KMT_TESTS=on

/merge


Login with Two-Factor Authentication not available

My Setup

Operating System: Windows 10
App Version: 3.0.2
Installation type: exe

[x ] I have tested with the latest version
[x ] I can simulate the issue easily

Description

During the login the Two-Factor Authentication is not asked.
Current Behavior

Attempting to sign in using the two-factor authentication with Google Authenticator. Successfully enter login name and password but after this the two-factor authentication is not asked.
Expected Behavior

It should be asked during the login to authenticate using two-factor authentication.

I agree this issue persists in version 3.0.4 of the electron client.
In my environment, users are required to log in with certificate based SAML authentication.
The 3.0.4 overrides/covers the server-configured logon page and does not provide an option to select user certificate.
I've included a side-by-side: Electron client on the left, Chrome browser on the right.

Please test if the latest release did solve this. If not please reopen the issue or create a new one and tell us. I will be closing this one for now.
Thank you!


Drone sheet improvements

Fixed an error when creating a fresh drone actor and added a nice display when the drone has no skills available to it.

Looks good, merging.


[RFC] Wrapper/Proxy for Stream

🚀 Feature
Discussed with @NivekT about the wrapper class for all streams:
Pros:

We can add a __del__ method to close the file stream automatically when ref count becomes 0 for wrapper. It would eliminate all warnings.
A wrapper class can unify the reading API for file streams. (For OnDiskCache, I would prefer a unified API to read stream, otherwise I have to handle all different cases)

Local file stream, we can use read() to read everything into memory
When we set stream=True for large file, the requests.Response doesn't support read. It only supports iter_content or __iter__ to read chunk by chunk.

Cons:

Thanks to @NivekT, it needs extra care about magic methods.

Reference: https://github.com/pytorch/data/pull/35#discussion_r728201731, https://github.com/pytorch/data/pull/65#discussion_r730117933
cc: @VitalyFedyunin

@NivekT
IIUC, HTTPRespond and GDriveRespond should also be wrapped by StreamWrapper

Yea, I believe they should, and they should still work as intended given that StreamWrapper delegates the method to the underlying object.


Add footprint class to normalize footprint representations.

Currently footprint names are hard coded in the programs.
The footprints should be converted into a class that is then resolved based upon the tool being output into.
Optionally we could also output the source used to make the footprint for the layout tool.
R_0402_info = {
    KICAD: {'footprint': 'Resistors_SMD:R_0805'},
    PCB_GEDA: {'footprint': "0805",
                           'footprint_source': "Insert M4 footprint source code here.",
    },
}

R_0402 = Footprint(R_0402_info)

I seem to recall that there was a standard for industry standard footprints.
http://www.ipc.org/ContentPage.aspx?pageid=Land-Pattern-Calculator
Not sure of it's use but they claim support for kicad.

To support intelligent footprint selection we should add footprint properties like dimensions etc...
This is to enable if footprint_a >= footprint_b:


[Addon] add velaux argument: nodeport

Signed-off-by: Qiaozp<EMAIL_ADDRESS>
Description of your changes
as title.

How has this code been tested?

Checklist

I have:

[x] Title of the PR starts with type (e.g. [Addon] , [example] or [Doc]).
[ ] Updated/Added any relevant documentation and examples.
[ ] New addon should be put in experimental.
[ ] Update addon should modify the version in metadata.yaml to generate a new version.

Verified Addon Contribution
I have:

A new addon added in this repo should be put in as an experimental one unless you have test for a long time in your product environment and be approved by most maintainers.

An experimental addon must meet these conditions to be promoted as a verified one.

[ ] This addon must be tested by addon's e2e-test to guarantee this addon can be enabled successfully.

This addon must have some basic but necessary information.

[ ] An accessible icon url and source url defined in addon's metadata.yaml.

[ ] A detail introduction include a basic example about how to use and what's the benefit of this addon in README.md.

[ ] Also provide an introduction in KubeVela documentation.

[ ] It's more likely to be accepted if useful examples provided in example dir.

Let's merge this and override v1.5.4 VelaUX. @barnettZQG


admin url issue

when website app runs from subfolder ( ex: yourdomain.com/subfolder ) URLs of admin panel with "admin_base_path" function in view working wrong.
I fixed this bug here -> https://github.com/hesamfarhang/laravel-admin

Thanks.  How can I install it?


Launch Configuration "Attach by Process ID"  Not Showing Any Process IDs

Issue Type: Bug
Steps

Clone https://github.com/mrroll/vscode-attach-process
run yarn && yarn start
Ensure Debugger listening on ws://<IP_ADDRESS>:9230 is present in the terminal output.
Hit F5

Expected Result

See the node process on port 9230

Actual Result

Don't see the node process.

Additional Information

It worked 2 versions ago.
There are no errors in the developer console

VS Code version: Code 1.43.1 (fe22a9645b44368865c0ba92e2fb881ff1afce94, 2020-03-18T07:17:38.324Z)
OS version: Linux x64 5.5.8-1-MANJARO

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4790 CPU @ 3.60GHz (8 x 3673)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_renderer: disabled_off_okvideo_decode: unavailable_offviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
1, 1, 1

Memory (System)
15.59GB (1.84GB free)

Process Argv
--no-sandbox

Screen Reader
no

VM
0%

Extensions: none

Just a quick update. It works again. @weinand  I'm leaving it open for review in case you guys want to check it out further.
VS Code version: Code 1.43.2 (0ba0ca52957102ca3527cf479571617f0de6ed50, 2020-03-24T07:52:11.516Z)
OS version: Linux x64 5.5.13-1-MANJARO

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4790 CPU @ 3.60GHz (8 x 3189)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_renderer: disabled_off_okvideo_decode: unavailable_offviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
1, 1, 1

Memory (System)
15.59GB (4.70GB free)

Process Argv
--no-sandbox

Screen Reader
no

VM
0%

Extensions: none

Since this issue was written we have a new debugger. It sounds like you got it working, but please let us know if you hit it again. Thanks!


Initial commit for fp8 CI

What does this PR do?

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Replaced by https://github.com/huggingface/optimum-habana/pull/995


Feature/grc subdivisions

Changes in this PR:
The General Regulatory Chamber has a number of Jurisdictions (or
'sub-tribunals' as we sometimes call them). These are different to the
separate Lists we see within eg the Chancery Division - Lists are an
administrative distinction, decided on by the court themselves, whereas
Jurisdictions are a statutory one, decided in law. Neither jurisdiction
nor list affects the NCN of the judgment, but they're stored differently
in the XML - Lists are represented in the court code in the uk:court
element, whereas judgments are represented with the uk:jurisdiction
element in addition to the uk:court.
This PR provides accessors for the jurisdiction data, both in isolation,
and combined with the court as a slash-separated string, following the
convention established in the courts metedata in ds-caselaw-utils.
This will allow us to display the jurisdiction in the PUI (and allow
editors to correct it in EUI) with minimal changes
I have also fixed a bug that was showing up in testing for me - the parse and enrich timestamps were an hour out, as i'm in a different timezone :-)
Trello card / Rollbar error (etc)
https://trello.com/c/JDoWjYsa/1602-medium-grcfollow-logic-for-the-courts-and-expose-grc-subdivisions-on-pui-prototype-first

 {Medium} GRC|Follow logic for the courts and expose GRC subdivisions on PUI - (prototype first)


Train on own dataset in COCO format with different class names.

Hi,
I am trying to train YOLOX-nano with my own dataset in COCO format.
So I just need to change:
self.data_dir = "datasets/my_coco_dataset"
self.train_ann = "instances_train.json"
self.val_ann = "instances_val.json"
self_num_classes = 2
My question is how I can change the class names to i.e. ['no_facemask', 'with_facemask']?
Thanks a lot.

For COCO, the class names are predefined:
https://github.com/Megvii-BaseDetection/YOLOX/blob/main/yolox/data/datasets/coco_classes.py

or I just need to add:
self.cls_names = ("no_facemask", "with_facemask")
?

For training and evaluation, you don't need to modify the cls_name if you follow the coco anno format. Class names are only used for visualization and demo.


Suggestions about UI and hotkeys

Hi, this extension is fire! I've been dreaming about this functionality since February.
I have couple of suggestions:
Would be nice to have some dark grey theme for this popout window, like the rest of the interface. It feels too bright.

Also would be cool to have hotkey like Alt+Left Click on keyword to "Disable\Enable Keyword". This is so needed because the disable button is too far on the right, and the workflow will be much faster.
Anyway, thanks for developing such a great extension, you're awesome!

Thank you very much for your suggestion.
I will make some modifications to this window under the dark theme.
Regarding disabling/enabling keywords, I've been considering using a double-click approach. If I were to use hotkeys, it would require setting up and binding the hotkeys, which can be quite complicated at the moment.
Additionally, there are some enhanced features in the expansion style that you can try, such as moving the keyword popup to the right side.

Dark theme pop-up window has been optimized.
New feature added: Double-click to disable/enable keyword.


Forge 1.20.4 installer is no longer able to find entry point

Describe the problem
Originally reported in Minecraft on Docker Discord and discussed with Forge team.
Changed mc-image-helper to locate the shim jar, but fails with a new problem:
+ exec mc-server-runner --stop-duration 60s --named-pipe /tmp/minecraft-console-in java -Xmx1G -Xms1G -jar forge-1.20.4-49.0.7-shim.jar
Missing required library: net/minecraftforge/forge/1.20.4-49.0.7/forge-1.20.4-49.0.7-client.jar
Exception in thread "main" java.lang.IllegalStateException: Missing required libraries! Check log
	at net.minecraftforge.bootstrap.shim.Main.main(Main.java:53)

Container definition
docker run -it --rm -e EULA=true -e TYPE=FORGE -e VERSION=1.20.4 -e FORGE_VERSION=49.0.7 -e DEBUG=true itzg/minecraft-server

Might be related to https://github.com/MinecraftForge/MinecraftForge/issues/9815

The error while running the shim was only a problem with 49.0.7. 49.0.9 of the shim runs fine.


FFlag Editor v2 (Possibly v3.0.0?)

Acknowledgement of preliminary instructions

[X] I have read and acknowledged the preliminary instructions.

What idea do you have?
I've had a few thoughts of this concept.
So my first idea is to make the FFlag Editor the main tab, removing the flag preset tab. If somebody adds a well-known flag (like "DFIntTaskSchedulerTargetFps"), it would put a info symbol next to the flag name, and when hovering the cursor over it, it displays info about that flag. (Also the Show Preset button obviously wouldn’t exist.)
The Add Flag button will add a new row to the flag editor instead. If a row’s name section is empty it will show a drop down (or something similar) that displays some well-known flags which you can select and once a flag is selected.
More to the idea, once a well known flag is selected. It would make you select a value in which a dropdown (or something similar) would appear, which shows options for the specific flag. For example, if "DFIntTaskSchedulerTargetFps" is selected it would show the following options: Default (0), Synchronize (Your refresh rate), infinite (9999).
Next idea is the FFlag Editor being able to recognize flag types: Booleans, Integers, and String FastFlags, and putting their respective value types (toggles for FFlags, a text field or something else for FInts, and text fields for FStrings).
My changes to the importing feature (#1092) would be included.

I have too many ideas tbh, whenever I come across something I like (e.g. Bloxstrap) I think of a whole lotta ideas to make it better.

(A little edit to my first idea if you think it's a bit to much, is to just swap the so the Flag Editor would be first and the preset tab would just be in a sub-tab)

removing the flag preset tab.

I'm sorry? Why would you want to do such a thing?
This entire suggestion is terrible and heavily compromises on user experience. Flag presets being shown prominently and upfront while the flag editor is quietly hidden away is done for a reason. How is a user being presented with "DFIntTaskSchedulerTargetFps" for a setting's name within a dense list format and important info being hidden away an improvement over "Framerate limit" with important information being shown upfront?

Well @pizzaboxer are you gonna consider the other idea...? The only things you've mentioned is about half of my request.

As I said in the other issue you mentioned, I am already planning to improve how FastFlags can be added/imported, and that included checking for valid flag types as you mentioned. Though, it's not clear if it's possible for me to change what type of input can be given here.

As I said in the other issue you mentioned, I am already planning to improve how FastFlags can be added/imported, and that included checking for valid flag types as you mentioned. Though, it's not clear if it's possible for me to change what type of input can be given here.

Alright, thank you for responding.


make service account optional

This PR makes the use of a GCP service account optional.
Fixes #4
Signed-off-by: Johan Siebens<EMAIL_ADDRESS>

Hi @jsiebens I've merged and released this in 0.3.2, could you reference it downstream where required please?
https://github.com/inlets/cloud-provision/releases/tag/0.3.2
Alex


V: Show V version does not work

Hi,
I install the vlang extension in vscode and I tried to ctl+shift+p then V: Show V version.
I got this error:

Unable to get the version number. Is V installed correctly?

My v is installed from git and If I open an cmd window and type v version:

D:\github>v version
V 0.3.4 34163ae

My v.exe's full path is D:\github\v\v.exe
Then I tried to set the full path at the setting:

I'm not sure I should use D:\github\v\v.exe or D:\github\v so I tried both, it does not help to make any difference.
I'm opening a folder in vscode, and the .v file is the active open file in vscode.

after reading the commit history, it seems that this issue is fixed about a year ago (https://github.com/vlang/vscode-vlang/pull/475) but never released.
Please release a new version.


Too many pack/unpack string functions?

I assume there are valid reasons to have all of these unpack combos as functions:

unpackTxt
unpackTxtString
unpackString
unpackStringTxt

but I'm having trouble wrapping my head around it.  Could you add doc strings?

That's mainly my fault when I merged #360. Ive added unpackStringTxt, that just called unpackTxt.

unpackStringTxt - unpacks []string
unpackString - unpacks string
these mirror the struct tags in types.go - at least that is the plan

unpackTxt and unpackTxtString are nominated to be killed in the ongoing (but almost finished!!) effort to fix #291. I expect half of msg.go to remain, maybe even less.

I look forward to that cleanup.  Any idea when to expect it to be committed?

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/dns] Too many pack/unpac..." ]

I look forward to that cleanup.  Any idea when to expect it to be committed?

Whenever I can make the time :) Anywhere between a couple of days to a week,
don't want to have the code linger in this limbo state too long.
/Miek
--
Miek Gieben

This cleanup has been done, still think this might be confusing. Please send PRs for further cleanups :-)


Protractor bug report  - q library

Bug report

Node Version: ``
Protractor Version: ``
Angular Version: ``
Browser(s): ``
Operating System and Version ``
Your protractor configuration file
A relevant example test
Output from running the test
Steps to reproduce the bug
The URL you are running your tests against (if relevant)

error:

/Users/alexamil/.nvm/versions/node/v6.10.2/lib/node_modules/protractor/node_modules/q/q.js:155
                throw e;
                ^

TypeError: Cannot read property 'length' of undefined
    at Function.getSpecs (/Users/alexamil/.nvm/versions/node/v6.10.2/lib/node_modules/protractor/built/configParser.js:88:25)
    at new TaskScheduler (/Users/alexamil/.nvm/versions/node/v6.10.2/lib/node_modules/protractor/built/taskScheduler.js:34:100)
    at helper.runFilenameOrFn_.then.then.then (/Users/alexamil/.nvm/versions/node/v6.10.2/lib/node_modules/protractor/built/launcher.js:175:25)
    at _fulfilled (/Users/alexamil/.nvm/versions/node/v6.10.2/lib/node_modules/protractor/node_modules/q/q.js:834:54)
    at self.promiseDispatch.done (/Users/alexamil/.nvm/versions/node/v6.10.2/lib/node_modules/protractor/node_modules/q/q.js:863:30)
    at Promise.promise.promiseDispatch (/Users/alexamil/.nvm/versions/node/v6.10.2/lib/node_modules/protractor/node_modules/q/q.js:796:13)
    at /Users/alexamil/.nvm/versions/node/v6.10.2/lib/node_modules/protractor/node_modules/q/q.js:604:44
    at runSingle (/Users/alexamil/.nvm/versions/node/v6.10.2/lib/node_modules/protractor/node_modules/q/q.js:137:13)
    at flush (/Users/alexamil/.nvm/versions/node/v6.10.2/lib/node_modules/protractor/node_modules/q/q.js:125:13)
    at _combinedTickCallback (internal/process/next_tick.js:73:7)

the error was generated by this script:
#!/usr/bin/env bash
search_dir="$(dirname "$0")";
project_root="${SUMAN_PROJECT_ROOT:-$(cd $(npm root) && cd .. && pwd)}"
export PROTRACTOR_TEST_PATHS=$(find ${search_dir} -name '*spec.js')
protractor "${project_root}/protractor.conf.js"

This has to do with find returning a newline delimited string, so I put this in protractor.conf.js
let testPaths;
if(process.env.PROTRACTOR_TEST_PATHS){
  testPaths = String(process.env.PROTRACTOR_TEST_PATHS).split(/\s/g).map(function(s){
    return String(s).trim()
  })
  .filter(v => v);
}

...

specs : testPaths || []

now it works, but you should try to fix the bug anyway though?


Use SQL commands instead of createdb and dropdb

Fixes: #245
Removes calls to createdb and dropdb and replaces them with SQL commands. This means the user will not need to install the PostgreSQL client to use this tool.

@Newbie012 I'm not sure what the versioning strategy is here. Should I create a changeset or bump the versions manually?

run pnpm changeset from the root.
Looks great!


fix: add blank target for external urls when using anchors

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Make Express probe work in request mode

Fixes #324

Unit test not required?

LGTM

@sam-github - yes, but we still don't have a good testing story for the probes, we really should do this at some point


warn on filter parameters

Problem 1:
After taking a close look and reading the plugin doc in this github, there is one more problem -> the logs are not even seen on http://localhost:24231/metrics
So, I am guessing the source is also not working
curl http://localhost:24231/metrics does not show anything.
Problem 2:
I am following the steps as per this blog ->
https://blog.treasuredata.com/blog/2016/07/19/routing-data-from-docker-to-prometheus-server-via-fluentd/
The entire filter tag seems to be going as warn and not used. can please help
2017-05-13 19:40:31 -0400 [warn]: parameter 'type' in <filter **>
type prometheus

name test
type counter
desc total count

 is not used.
2017-05-13 19:40:31 -0400 [warn]: parameter 'name' in
name test
type counter
desc total count
 is not used.
2017-05-13 19:40:31 -0400 [warn]: parameter 'type' in
name test
type counter
desc total count
 is not used.
2017-05-13 19:40:31 -0400 [warn]: parameter 'desc' in
name test
type counter
desc total count
 is not used.

Can you paste your fluentd.conf and describe fluent-plugin-prometheus version?

I close this issue because not updated.
Please feel free to reopen if you have a update or comment.


⚠ Samsung has degraded performance

In 1e621d3, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2148 ms

Resolved: Samsung performance has improved in f513898.


Gradle plugin org.codehaus.groovy.runtime.GStringImpl cannot be cast to java.lang.String

Just got exception java.lang.ClassCastException: org.codehaus.groovy.runtime.GStringImpl cannot be cast to java.lang.String with Gradle 1.10 and Flyway plugin (version 2.3).
Workaround from Gradle's issue tracker http://issues.gradle.org/browse/GRADLE-2566 works for me.
Situation:
I have some string property in gradle.properties file (in project's root)
Later I use it in build.gradle to configure flyway plugin, like:
placeholders = ['execUserName' : "${project.execUserName}"]

and it gives exception: java.lang.ClassCastException: org.codehaus.groovy.runtime.GStringImpl cannot be cast to java.lang.String
But
placeholders = ['execUserName' : "${project.execUserName}".toString()]

works like expected.
Not sure if it is Flyway plugin specific error or Gradle's (posted comment to http://issues.gradle.org/browse/GRADLE-2566 too)
May be relevant to https://github.com/flyway/flyway/issues/574

please try this
applicationVariants.all { variant ->
variant.outputs.all {
outputFileName = "${variant.name}-${variant.versionName}.apk"
}
        }

in your build.gradle file


Check for typo and language mistakes

Sometimes typos or incorrect spelling appear in the changelog, although it can be prevented by danger.
I think as a reference we can take a look on this.

Did you try to use danger-prose along with this one? I think it might just work out of the box?

@dblock not really, didn't have time yet. The idea just came up and I decided to write it down before I forget 😄

@Antondomashnev Want to help out with this gem? Make a release? I would be happy to add you to contributors and rubygems! (what's your rubygems email?)

@dblock I'm already 😉

Ah yes, I forgot :)


update packages

:white_check_mark: Build TheIdServer 7.4.4-fix-dependencies1-0008 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/e5fb5b579b by @invalid-email-address)


prost-build forces prost 0.5.0 which does not include r#Self bugfix.

Here's the discussion from the tonic project: https://github.com/hyperium/tonic/issues/225
prost-build version 0.5.0 specifies prost 0.5.0 (Cargo.toml) which does not include the bugfix found in https://github.com/danburkert/prost/commit/f51cbb2d23fe1c044309f8ab14473b0db61c3ffe on the master branch.
I was hoping to write some kind to take advantage of that fix. Any suggestions on how to proceed?

The issue here is that a fix isn't yet released, not that prost-build's dependency version is incorrect (FYI 0.5.0 does not pin to exactly 0.5.0 https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#specifying-dependencies-from-cratesio).  A new version of prost is actively being worked on, and will be released soon.

Pardon the silly question but if I point my Cargo.toml to use master, I would get this fix right?

Yes, that's correct.  It gets a bit hairy with tonic, though.  There's some discussion about this in the prost room of the tokio discord from yesterday.

Alrighty. Gonna close this. Thanks so much for the fast response and all your work with prost. <3


D3 refactor

All the DOM for the SVG replaced with D3 (the SVG is responsive)
Support for D3 update changes - it means no need for recreating the graph
Text display is now part of the SVG using D3 as well as the dividers lines
Tooltip is supported per line / section (on 2d)
Click on section with configurable callback is supported

From the forked code mainly I used the 2d Path creation

By mistake - sorry this aimed for my fork


Add context menu command support

Things to consider:

whether to split UserContextMenuCommands and MessageContextMenuCommands from the get-go (vs. current solution of checking inside a unified handler); see https://github.com/ist-bot-team/ist-discord-bot/pull/100/files#diff-0c14a37a0833070d83b1840afa4f70d830ccf1dd45cfc2f69d783aaea9ae1c1aR37
how to improve command logging format (audit, not debug) to look better / potentially include more information; see utils.stringifyContextMenuCommand

Thoughts, @diogotcorreia?

@RafDevX

I'd split them. When registering the command, it can only be of one type (from what I've seen, if I'm wrong this does not apply then). Therefore, it'd make sense for the handler to be only of that type.
Seems good to me as it is tbh

I've decided against splitting it as there would be no way to actually type-check on runtime.
I can tell TypeScript that the handler accepts a MessageContextMenuInteraction but, in the end, there's nothing stopping someone from registering both a Message and a User command with the same name* - which would be translated into the same handler (and one of them would error, expecting it the interaction to be of type X when it's of type Y).
As such, I think the best way to do this is to just use type guards inside handlers (through isMessageContextMenuCommand and isUserContextMenuCommand), as originally designed.
*as far as I can tell, Discord allows this (I just tested it)


gc-compaction: filter keys not in shard / not in keyspace

So that we can reclaim spaces taken by https://github.com/neondatabase/neon/issues/8623 and collect more garbages.

we still need to figure out a way to deal with keyspace changes in gc-compaction


Restructured Token type

Fixes: #186
Description
The old Token type is a leftover from MultiToken and MultiMint. Both got never really got handled by cashu-ts (it most of the time extracted the first token inside a token object and worked with that). Now that v4 is the new standard it is time to restructure the Token.
Changes

Restructured the Token type
Adjust all methods to work with the new type

PR Tasks

[x] Open PR
[x] run npm run test --> no failing unit tests
[x] run npm run format

FYI: I realised that two things are still missing:

adjust integration tests
adjust token v3 encoding (not sure why we do not have a unit test for this, will add them)


(re-)fire reminders that were missed by only a few seconds

The following happened to me:
matrix-reminder-bot    | 2021-08-02 23:11:38,199 | matrix_reminder_bot.callbacks [DEBUG] Command received: !remindroom tuesday 09:55 ; remindertext
matrix-reminder-bot    | 2021-08-02 23:11:38,199 | matrix_reminder_bot.bot_commands [DEBUG] Parsing command arguments: tuesday 09:55 ; remindertext
matrix-reminder-bot    | 2021-08-02 23:11:38,199 | matrix_reminder_bot.bot_commands [DEBUG] Got time: tuesday 09:55
matrix-reminder-bot    | 2021-08-02 23:11:38,207 | matrix_reminder_bot.bot_commands [DEBUG] Creating reminder in room !myroom:myhost.tld with delta None: remindertext
matrix-reminder-bot    | 2021-08-02 23:11:38,209 | apscheduler.scheduler [INFO] Added job "Reminder._fire" to job store "default"
matrix-reminder-bot    | 2021-08-02 23:11:38,216 | apscheduler.scheduler [DEBUG] Looking for jobs to run
matrix-reminder-bot    | 2021-08-02 23:11:38,216 | apscheduler.scheduler [DEBUG] Next wakeup is due at 2021-08-03 09:55:00+02:00 (in 31401.783110 seconds)
matrix-reminder-bot    | 2021-08-02 23:11:55,823 | matrix_reminder_bot.callbacks [DEBUG] Command received: !remindroom wednesday 09:55 ; otherremindertext
matrix-reminder-bot    | 2021-08-02 23:11:55,823 | matrix_reminder_bot.bot_commands [DEBUG] Parsing command arguments: wednesday 09:55 ; otherremindertext
matrix-reminder-bot    | 2021-08-02 23:11:55,823 | matrix_reminder_bot.bot_commands [DEBUG] Got time: wednesday 09:55
matrix-reminder-bot    | 2021-08-02 23:11:55,829 | matrix_reminder_bot.bot_commands [DEBUG] Creating reminder in room !myroom:myhost.tld with delta None: otherremindertext
matrix-reminder-bot    | 2021-08-02 23:11:55,829 | apscheduler.scheduler [INFO] Added job "Reminder._fire" to job store "default"
matrix-reminder-bot    | 2021-08-02 23:11:55,834 | apscheduler.scheduler [DEBUG] Looking for jobs to run
matrix-reminder-bot    | 2021-08-02 23:11:55,835 | apscheduler.scheduler [DEBUG] Next wakeup is due at 2021-08-03 09:55:00+02:00 (in 31384.164924 seconds)
matrix-reminder-bot    | 2021-08-03 07:55:00,008 | apscheduler.scheduler [DEBUG] Looking for jobs to run
matrix-reminder-bot    | 2021-08-03 07:55:01,001 | apscheduler.scheduler [INFO] Removed job edbec807c6b64451bfc87a15abbe473a
matrix-reminder-bot    | 2021-08-03 07:55:01,001 | apscheduler.scheduler [DEBUG] Next wakeup is due at 2021-08-03 19:00:00+02:00 (in 32699.988727 seconds)
matrix-reminder-bot    | 2021-08-03 07:55:01,005 | apscheduler.executors.default [WARNING] Run time of job "Reminder._fire (trigger: date[2021-08-03 09:55:00 CEST], next run at: 2021-08-03 09:55:00 CEST)" was missed by 0:00:01.005331

Hit another one:
matrix-reminder-bot    | 2021-09-21 17:43:51,648 | matrix_reminder_bot.callbacks [DEBUG] Command received: !r tomorrow noon; first reminder
matrix-reminder-bot    | 2021-09-21 17:43:51,650 | matrix_reminder_bot.bot_commands [DEBUG] Parsing command arguments: tomorrow noon; first reminder
matrix-reminder-bot    | 2021-09-21 17:43:51,650 | matrix_reminder_bot.bot_commands [DEBUG] Got time: tomorrow noon
matrix-reminder-bot    | 2021-09-21 17:43:53,496 | matrix_reminder_bot.bot_commands [DEBUG] Creating reminder in room !room:host.tld with delta None: first reminder
matrix-reminder-bot    | 2021-09-21 17:43:53,505 | apscheduler.scheduler [INFO] Added job "Reminder._fire" to job store "default"
matrix-reminder-bot    | 2021-09-21 17:43:53,518 | apscheduler.scheduler [DEBUG] Looking for jobs to run
matrix-reminder-bot    | 2021-09-21 17:43:53,518 | apscheduler.scheduler [DEBUG] Next wakeup is due at 2021-09-22 12:00:00+02:00 (in 58566.481818 seconds)
matrix-reminder-bot    | 2021-09-21 17:56:52,484 | matrix_reminder_bot.callbacks [DEBUG] Command received: !r tomorrow noon; second reminder
matrix-reminder-bot    | 2021-09-21 17:56:52,484 | matrix_reminder_bot.bot_commands [DEBUG] Parsing command arguments: tomorrow noon; second reminder
matrix-reminder-bot    | 2021-09-21 17:56:52,484 | matrix_reminder_bot.bot_commands [DEBUG] Got time: tomorrow noon
matrix-reminder-bot    | 2021-09-21 17:56:52,489 | matrix_reminder_bot.bot_commands [DEBUG] Creating reminder in room !room:host.tld with delta None: second reminder
matrix-reminder-bot    | 2021-09-21 17:56:52,489 | apscheduler.scheduler [INFO] Added job "Reminder._fire" to job store "default"
matrix-reminder-bot    | 2021-09-21 17:56:52,494 | apscheduler.scheduler [DEBUG] Looking for jobs to run
matrix-reminder-bot    | 2021-09-21 17:56:52,494 | apscheduler.scheduler [DEBUG] Next wakeup is due at 2021-09-22 12:00:00+02:00 (in 57787.505645 seconds)
matrix-reminder-bot    | 2021-09-22 10:00:00,003 | apscheduler.scheduler [DEBUG] Looking for jobs to run
matrix-reminder-bot    | 2021-09-22 10:00:00,011 | apscheduler.scheduler [INFO] Removed job 37ef2a99bfa64e6f9e28273ee4459033
matrix-reminder-bot    | 2021-09-22 10:00:00,012 | apscheduler.scheduler [INFO] Removed job 4a5ac502fe8945779d8adecbc7eddb3f
matrix-reminder-bot    | 2021-09-22 10:00:00,012 | apscheduler.scheduler [DEBUG] Next wakeup is due at 2021-09-22 19:00:00+02:00 (in 25199.991128 seconds)
matrix-reminder-bot    | 2021-09-22 10:00:00,013 | apscheduler.executors.default [INFO] Running job "Reminder._fire (trigger: date[2021-09-22 12:00:00 CEST], next run at: 2021-09-22 12:00:00 CEST)" (scheduled at 2021-09-22 12:00:00+02:00)
matrix-reminder-bot    | 2021-09-22 10:00:00,013 | matrix_reminder_bot.reminder [DEBUG] Reminder in room !room:host.tld fired: first reminder
matrix-reminder-bot    | 2021-09-22 10:00:05,991 | apscheduler.executors.default [WARNING] Run time of job "Reminder._fire (trigger: date[2021-09-22 12:00:00 CEST], next run at: 2021-09-22 12:00:00 CEST)" was missed by 0:00:05.990765
matrix-reminder-bot    | 2021-09-22 10:00:06,603 | matrix_reminder_bot.reminder [DEBUG] Cancelling reminder in room !room:host.tld: first reminder
matrix-reminder-bot    | 2021-09-22 10:00:06,686 | apscheduler.executors.default [INFO] Job "Reminder._fire (trigger: date[2021-09-22 12:00:00 CEST], next run at: 2021-09-22 12:00:00 CEST)" executed successfully

As you can see, I set up 2 reminders to fire at the same time. One worked, the other was missed by 5 seconds.
I was not able to repro. The bot definitely needs some resilience. Probably it should just dump matrix-reminder-bot    | 2021-09-22 10:00:05,991 | apscheduler.executors.default [WARNING] Run time of job "Reminder._fire (trigger: date[2021-09-22 12:00:00 CEST], next run at: 2021-09-22 12:00:00 CEST)" was missed by 0:00:05.990765 into the respective room anyway formatted as @recipient:host.tld $remindertext (scheduled for $time was missed by $delay)


Missing definition of enum 'PointerDeclaratorSort' in the section sec:ifc:SyntaxSort:PointerDeclarator

https://github.com/microsoft/ifc-spec/blob/main/ltx/parse.tex#L609

Thanks.  It would be good to have a LaTeX macro to typeset references to types so that:

we can automatically catch undefined types the way undefined references are caught
references to a type has a hyperlink to its definition/description site


network-manager: replace hardcoded path to ping

This chance fixes the hardcoded path to /usr/bin/ping which fixes things on NixOS but partially breaks network-manager on non NixOS.
Things done:

[X] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
Built on platform(s)

[X] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

More
Fixes issue #14103
cc @abbradar

I think I have a solution that satisfies both worlds -- as what user does NM usually run? If it's root or someone with raw network packet privileges, we can just replace path to ping in Nix store because we don't need SUID bit. For me at least it runs as root.

Sadly I haven't seen those ping error messages. I suppose you can reproduce this -- could you try to replace hardcoded path with ${inetutils}/bin/ping and see if it works?
Also, please rename commit to "network-manager: replace path to ping` (per our guidelines).

Thanks! Did this work?

I'm closing this for now until I have cleaned up the branch. Will then reopen.


bigquery: show base64 encode and records in insertAll snippets.

Showing a raw base64 string isn't great. Instead, show a way to encode
the bytes to a base64 string using Guava's BaseEncoding class.
Also, it wasn't clear how to insert values of type record, so show that,
too.
@lesv @mziccard PTAL

Coverage increased (+0.005%) to 84.22% when pulling a6cc3f126844139cbad168194085ebbdc8f27e22 on tswast:bigquery-insertall into da12d8fd3b190875f5903b1a794916ef3da3cb96 on GoogleCloudPlatform:master.

Using Guava in snippets is generally a bad idea. We shouldn't assume that users are familiar with it nor we should show code that depends on it to work

Okay. I wasn't sure about that. I wish we could show imports in snippets.
What about using classes that are Java 8, only? http://docs.oracle.com/javase/8/docs/api/java/util/Base64.html There wasn't a built-in way to encode Base64 until then.


fix(featureCard): change styles so user can still click

Related Ticket(s)
none
Description
Due to card updates Feature Card in md/sm breakpoints link was broken. This updates styles to fix and allow user to click anywhere on card
Changelog
Changed

styles to remove position absolute

nents looked in great shape, but noticed React sm breakpoint the card lost 2x1 aspect ratio:
Current: 320x144px, expecting 320x160 for both the image and card content piece.

Thanks Shixie, good catch! Looks like this is happening in react production mode and out of scope of this PR. Can we make a seperate bug?

@shixiedesign trying to catch up on this one, looks like what you mentioned could be a separate issue?


Record git project name and sha

This pr adds functions to the dockerfile parser. If the parser
comes across the ADD directive, record the git projeect name and
sha in a comment after the command.
This pr modifies utils/general.py and analyze/docker/dockerfile.py.
In dockerfile.py, function find_git_info recieve a line of ADD
command and the path to the parsed dockerfile. Function
expand_add_command receives a dfobj and iter it with find_git_info.
In general.py, function check_git_folder is used to find git info
in a folder and funtion check_git_tar in a tar file. Function
check_git_src looks into the src_path, and check the file type.
Work towards #509.

I have run some local tests. i.e.
FROM ubuntu
ADD plain_file /tmp
ADD test_git_repo /tmp
ADD test_git_repo.tar /tmp

i got the output
FROM ubuntu
ADD plain_file /tmp # Not a folder or tar file
ADD test_git_repo /tmp # git project name: (not found) sha info: 55cbf5cbc8b201a99ac63f70f6f6f0ff1d1ae630
ADD test_git_repo.tar /tmp # git project name: (not found) sha info: 55cbf5cbc8b201a99ac63f70f6f6f0ff1d1ae630

The dockerfile and git repo are in the sample folder.
I am now working on  adding some tests for these functions.

I have run some local tests. i.e.
FROM ubuntu
ADD plain_file /tmp
ADD test_git_repo /tmp
ADD test_git_repo.tar /tmp

i got the output
FROM ubuntu
ADD plain_file /tmp # Not a folder or tar file
ADD test_git_repo /tmp # git project name: (not found) sha info: 55cbf5cbc8b201a99ac63f70f6f6f0ff1d1ae630
ADD test_git_repo.tar /tmp # git project name: (not found) sha info: 55cbf5cbc8b201a99ac63f70f6f6f0ff1d1ae630

The dockerfile and git repo are in the same folder.
I am now working on adding some tests for these functions.

@ForgetMe17 what is the test_git_repo that you have in your test dockerfile? Could you also test with a git repository where the project name is available?
@nishakm Is the sha info useful if we don't know the name of the git project?  This seems confusing to me but I am also no expert at what .git metadata provides.

I created a local git repository, not cloning from github. So the repo does not have a remote url where I get the name of project.

@nishakm I am thinking we may also want to move find_git_info, check_git_folder, check_git_tar and check_git_src to tern/analyze/docker/common.py instead of leaving them in dockerfile.py and just leave the expand_add_command in dockerfile.py. Thoughts?

In the commit 87b35d6, I add tests for find_git_info and expand_add_command, and move check_git_folder, check_git_tar and check_git_src to tern/analyze/docker/common.py.
I test the plain file, tar file and git folder. For git folder, since a git repo cannot be uploaded unless as a submodule. So in the online ci test, the result for the git folder is Not a git repository.
In the test, the path of dockerfile is relative and the src_path is also relative. For the situations where the git repo is a few folders above the dockerfile, i think in this case the src_path in the ADD command should be like ../../git_repo. I donnot know if it is valid in the dockerfile(probably not i think), but this pr does not work with this situation since the os.path.join will append ../../git_repo to the dockerfile_path. So maybe i will work on locating src_file in this situation.

commit 648427f, Using command line and modified tests
In tern/analyze/common.py:
1.Delete function check_git_tar and check_git_folder.
2.Add get_git_toplevel which uses git command to find the toplevel directory.
3.Add get_git_project_name and get_git_sha to get the git repository information which uses the path_to_toplevel given by the get_git_toplevel.
Modified tests in the tests/test_analyze_docker_dockerfile.py
In the test, i am using the dockerfile in tests/dockerfiles/pin_add_command_test and the ADD command is ADD plain_file /tmp. The plain_file is also in tests/dockerfiles/pin_add_command_test.  plain_file is not a git repo, but it is in our tern git repository, so the project name we get is tern, which is the toplevel folder name of the git repository. But in the circleci test, i find that the project name is project. I think it is becasue in the circleci test environment, the toplevel folder name is change to project. In the Pull Request Lint and Test of the Github, the toplevel folder name is 'tern'. So in the test, i used AssertIn to match the name between the tern and project.
And thanks for your suggestions which really helps me a lot!

I squashed the commits into one commit.

Thanks for your suggestions!


"x" close button on notifications do not have a "close" accessibility label

"x" close button on notifications do not have a "close" accessibility label. When read on a screen reader, it simply says "tap to activate" but people will not know what tapping to activate will do. This is missing on Android.
This also means that I'm unable to close the in-app guide using voice control.

Is this on notifications or on the the in-app guided tours? Both?


PR: Prevent double saving when running a file

Description of Changes
This PR fixes some autoformatting overlaps when a file is run in the iPython console due to duplicate calls to save when autoformatting on save is enabled

[ ] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)

[ ] Included a screenshot or animation (if affecting the UI, see Licecap)

Issue(s) Resolved

Fixes #14653
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct:  @andfoy

Hello @andfoy! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/plugins/editor/plugin.py:

Line 2623:22: E261 at least two spaces before inline comment
Line 2626:33: E127 continuation line over-indented for visual indent
Line 2627:33: E127 continuation line over-indented for visual indent
Line 2628:33: E127 continuation line over-indented for visual indent


spdy issue

Hi,
The following:
const zetta = require('zetta')

zetta()
  .name('Test')
  .listen(1337, function () {
    console.log('Zetta is running')
  })

Fails beautifully with the following message:
> node index.js

util.js:972
    throw new TypeError('The super constructor to "inherits" must not ' +
    ^

TypeError: The super constructor to "inherits" must not be null or undefined
    at Object.exports.inherits (util.js:972:11)
    at Object.<anonymous> (/Users/marvin/Documents/Code/zetta/node_modules/spdy/lib/spdy/connection.js:86:6)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/marvin/Documents/Code/zetta/node_modules/spdy/lib/spdy.js:20:19)

I am on macOS on node 7.7.3.

It looks like Zetta's broken on Node v7.x.  The spdy module has a reference to process.EventEmitter, which has been deprecated as of v7.x.  (See: https://github.com/spdy-http2/node-spdy/blob/3c7e45532a6083e1ae9a4fea26b70e62efa7eda4/lib/spdy/connection.js#L86).
The best option would be to upgrade Zetta's spdy module to the latest version, but last I heard, there were some issues with doing that.

As of the v ^2.0.0 of sdpy indeed, require('events') is used. What's wrong with the latest version?

@AdamMagaluk You looked at this months ago, didn't you?  Any notes from this?

Quick question, why SPDY over HTTP/2?

And, simple workaround:
process.EventEmitter = require('events').EventEmitter

const zetta = require('zetta')

zetta()
  .name('Test')
  .listen(1337, function () {
    console.log('Zetta is running')
  })

@kevinswiber I looked at upgrading the SPDY client a while back and made some progress but never finished. There were a few of places in zetta that needed to change because of internal updates for the spdy client.
@marvinroger We choose SPDY originally because when zetta was first developed HTTP/2 was still in draft and not finalized. We've talked about upgrading. If we were to upgrade to the latest spdy client it would support moving to the http2 spec.

I'm actually taking a look at this upgrading the spdy dependency now.
For a little history...
We started working on what would become Zetta in December of 2013.  It wasn't until December of 2014 that the IETF working group would start work on HTTP/2.  They used SPDY/4 as the base, and to my big surprise, HTTP/2 was a proposed standard by February of 2015.  That's an insanely short amount of time for the IETF.
Of course, by December of 2014, our Z2Z protocol had already been built and working.  It would still take many months after HTTP/2 became a proposed standard before there would be any working libraries or tooling.  Looking back, I still think we made the right choice to stick with SPDY.  We were already heavily invested and didn't have the cycles to spare.  We can still look at upgrading down the road.  We'd probably support HTTP/2 and SPDY/3.1 concurrently for backward compatibility.

FYI: Still working on this when I can.  I'm making progress.   phase, and now I'm looking at a few other broken tests as a result of the upgrade.

Also, one benefit of upgrading the spdy dependency is that we can look to provide an HTTP/2 interface for the Client API when it's done.

Great, thanks for the update. https://github.com/zettajs/zetta/pull/349 will also need the spdy update so that ES2015 classes can be used. The current version does not support it.

Thank you for simple work around ....

... I ran into the same issue with node 8.2.1 , the hack solved it - thanks
@kevinswiber I put the 'process.EventEmitter = require('events').EventEmitter' line at the top of zetta.js which seems to work as well.
It's ugly but would save other users from falling over this - patch wanted?

one more thing: when I link a zetta device on my machine running node 8.2.1 to a server with node 4.2.6 (on ubuntu xenial), I get a RangeError:
`svogl@M:~/proj/zetta-liner$ Jul-28-2017 15:59:27 [server] Server (eyeLiner) eyeLiner listening on http://<IP_ADDRESS>:3338
Zetta is running at http://<IP_ADDRESS>:3338
Jul-28-2017 15:59:27 [peer-client] WebSocket to peer established (ws://akoya.local:3338/peers/eyeLiner)
zlib.js:201
throw new RangeError('Invalid windowBits: ' + opts.windowBits);
^
RangeError: Invalid windowBits: 0
at Inflate.Zlib (zlib.js:201:13)
at new Inflate (zlib.js:520:8)
at Object.Inflate (zlib.js:519:12)
at Object.createInflate (/home/svogl/proj/zetta-liner/node_modules/spdy/lib/spdy/utils.js:46:22)
at Pool.get (/home/svogl/proj/zetta-liner/node_modules/spdy/lib/spdy/zlib-pool.js:41:27)
at Parser.onversion (/home/svogl/proj/zetta-liner/node_modules/spdy/lib/spdy/connection.js:107:19)
at emitOne (events.js:115:13)
at Parser.emit (events.js:210:7)
at Parser.setVersion (/home/svogl/proj/zetta-liner/node_modules/spdy/lib/spdy/protocol/parser.js:208:8)
at Parser.execute (/home/svogl/proj/zetta-liner/node_modules/spdy/lib/spdy/protocol/parser.js:229:12)
`
Any ideas?

I also get RangeError with client running node 8.2.1 and hitting the Apigee Link instance of Zetta (not sure what version that's running but I imagine 0.12).

@kevinswiber I was able to get a working example of our z2z protocol working with the latest spdy dependancy. See: https://github.com/AdamMagaluk/z2z-proxy Haven't tried making the changes in zetta and seeing what breaks yet...
Also looks like http/2 support has landed in Node.js master so could look at using that directly too, though not sure if all the apis will be available for the z2z use case. https://github.com/nodejs/node/pull/14239

@kevinswiber @AdamMagaluk any official patch or walk around ?
I'm on Node.JS 8.4.0 and upgrade to latest SPDY 3.4.7 but still the same error.
how to upgrade the SPDY dependencies to latest ? I'm confused with https://github.com/AdamMagaluk/z2z-proxy
how to apply it ?
thanks.

@hutualive No not yet, the z2z-proxy just demonstrates that it would be possible to upgrade zetta to use the latest spdy library. It shows the number of small API changes needed, though I'm still working through one issue regarding spdy pings as the interface isn't exposed in the new version.
I plan to work on a zetta branch in the next week or two that would apply the updates. I'll keep this channel updated.

Hi,
Is there any potential fix for this yet?

RangeError: Invalid windowBits: 0
zlib.js:202
at Inflate.Zlib (zlib.js:202:13)
at new Inflate (zlib.js:546:8)
at Object.Inflate (zlib.js:545:12)
at Object.createInflate (c:\Repositories\rbg\iot-700-controller\node_modules\zetta\node_modules\spdy\lib\spdy\utils.js:46:22)
at Pool.get (c:\Repositories\rbg\iot-700-controller\node_modules\zetta\node_modules\spdy\lib\spdy\zlib-pool.js:41:27)
at Parser.onversion (c:\Repositories\rbg\iot-700-controller\node_modules\zetta\node_modules\spdy\lib\spdy\connection.js:107:19)

@khinze I've been slowing working on a fix but unfortunately I haven't had much time to commit. Here is the latest "mostly" working version of zetta that fixes the issue. https://github.com/zettajs/zetta/pull/360

kjljkl

At this point, spdy seems beyond deprecated -- mostly dead. Adam did a lot of work to get spdy 4 mostly working with zetta, but now even more bugs are popping up. How much effort is required to replace all references and hooks to spdy with node's native HTTP/2? Also, I believe there are functions relied upon that HTTP/2 can't provide, so some other modules would need to be added in place of spdy.
I don't believe anyone alone has the time/bandwidth to complete all the required work. I'd be willing to work on some chunks if one of the more knowledgeable architects could devise an upgrade path. zetta was a very nice platform, and I'd hate to see it die on the vine.

Hi Scott,
I agree with you on spdy, it's been a nightmare and moving target trying to upgrade. I agree with your assessments on http/2 and i followed the Node.js API when it was being drafted and there were holes that zetta needed. But not sure what the state is I haven't been active in that for 2 years. Its worth re-evaluating.
During the SPDY upgrade i was also working under the constraint that Zetta versions be backwards compatible from a network and protocol perspective. If we're left with the option of making breaking changes and zetta being abandoned we could evaluate that contraint.
I'm always happy to advise and hop on a GVC and review PRs when I can. I agree with you and would love for Zetta to live on, I think it still has plenty of potential.


For a 0.16 release

#490
For the release notes:

(breaking) Modify the output of fitted_params for the following models, to bring them into line with other MLJ models: Standardizer, Discretizer, OneHotEncoder. See the updated doc-strings for the new format (#444, PR #490)

Codecov Report

Merging #491 (7214811) into master (1c813c2) will decrease coverage by 0.30%.
The diff coverage is 25.00%.

@@            Coverage Diff             @@
##           master     #491      +/-   ##
==========================================
- Coverage   77.43%   77.12%   -0.31%
==========================================
  Files          16       16
  Lines        1121     1093      -28
==========================================
- Hits          868      843      -25
+ Misses        253      250       -3

Impacted Files
Coverage Δ

src/builtins/Transformers.jl
95.23% <25.00%> (-1.17%)
:arrow_down:

src/loading.jl
67.53% <0.00%> (-0.42%)
:arrow_down:

src/builtins/ThresholdPredictors.jl
92.62% <0.00%> (-0.12%)
:arrow_down:

src/metadata.jl
77.64% <0.00%> (-0.02%)
:arrow_down:

src/registry/src/update.jl
0.00% <0.00%> (ø)

src/registry/src/activate_registry_project.jl
0.00% <0.00%> (ø)

src/utilities.jl
16.12% <0.00%> (+0.97%)
:arrow_up:

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

Merging despite the check_registry fail, which has known unrelated cause and is tracked: https://github.com/JuliaAI/MLJModels.jl/issues/493 . I have tested the registry locally and it's fine.


文档修正

hello, docs/tutorial/03-learning/05-use-fetcher.md has some error, at line 491
as-is: ## Compare with useRequest and useWatcher
should to-be: ## Compare with useRequest and useFetcher

感谢你对alova做出的贡献🤞


[grid] Add sessionqueue package.

Thanks for contributing to Selenium!
A PR well described will help maintainers to quickly review and merge it
Before submitting your PR, please check our contributing guidelines.
Avoid large PRs, help reviewers by making them as simple and short as possible.

Description

Adding NewSessionQueue and NewSessionQueuer interfaces.
LocalSessionQueue and LocalSessionQueuer are the implementations.
NewSessionRequest and NewSessionRequestResponse events are added to communicate via the EventBus.
The incoming request has a request-id which ties the whole flow together. The request-id is used by GetNewSessionResponse to maintain a local map of request and response and return the response as it receives it.

Queuer is responsible for sending the incoming /session request to the appropriate handler and waiting for the response.
LocalSessionQueue will perform the actual operation of adding, removing and retry adding to the local queue.
Motivation and Context
New session request should not be dropped as it is happening in the current implementation. Instead, it should be queued and handled as the slots are available.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[X] I have read the contributing document.
[X] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Moved the PR upstream.


Please help

Hi,
Recently I found your DbContextScope, and I find project very interesting.
I wanted to test it but I encounter some issue.
Explanation :
BUSINESS LAYER:
"ClientManager"
        public int Insert(Client client)
        {
            using (var context = _dbContextScopeFactory.Create())
            {
                CLIENT object = ClientMapper.Instance.Map(client);
                object = _clientRepository.Add(object);
                context.SaveChanges();
                client.IdClient = objet.ID_CLIENT;
                return object.ID_CLIENT;
            }
        }

"AddressManager"
     public int Insert(Address address)
        {
            using (var context = _dbContextScopeFactory.Create())
            {
                ADDRESS object = AddressMapper.Instance.Map(address);
                _addressRepository.Add(object);

                context.SaveChanges();

                return object.ID_ADRESSE;
            }
        }

I have my console program for test and it's works fine.
But now if I want to add to "ClientManager" another function:
     public int InsertClientWithAddress(Client client, Adresse address)
        {
            using (var context = _dbContextScopeFactory.Create())
            {
                int id = Insert(client)
          address.Id = id; // = 0 bacause no SaveChanges
                _addressManager.Insert(address);

                context.SaveChanges();

                client.IdClient = objet.ID_CLIENT;
                return client.IdClient;
            }
        }

It's not working because my client ID is auto-increment integer in database and it's not initialized because of DbContextScope.SaveChanges
How I can resolve this issue with your framework?
Thanks you in advance
P.S
Sorry for my English :)

Your title needs some work; I assume the actual issue is "Inserted item ID not populated"?
You may need to move the last two lines out of the using block; closing the connection to the database should complete the transaction. Otherwise, Context.Entry(address).Reload() might work.

Sorry, but It doesn't working...
When I use a classic TransactionScope with entity-framework it's works very well because when I call context.SaveChanges() => Id property is populated correctly and at the and I call Commit.
But in this case with DbContextScope SaveChages() Do anything when the are parent scope.

Finally,
I try this to replace DbContextScope with TransactionScope
and it's seems to work but I don't know if it's correct approach ?
public int InsertClientWithAddress(Client client, Adresse address)
        {
            using (TransactionScope scope = new TransactionScope())
            {
                int id = Insert(client)
                address.IdClient = id; // =now it's OK
                _addressManager.Insert(address);

                scope.Complete();

                return id;
            }
        }

You can overcome this issue by calling SaveChanges from the DbContext (not the DbContextScope) in your implementation of _clientRepository.Add


MenuItem needs an accessible name

Write an ESLint rule ensuring that the MenuItem component has an accessible name.          <MenuItem
icon={}
onClick={() => alert("Cut to clipboard")}
>

PR here


add the spin up hub

Adds the Spin Up Hub section with example templates, plugins, libraries and more.
@karthik2804 👏🏻 🙌🏻 :shipit:

Documenting some smaller bugs and improvements to follow up on
Global Menu Style

Despite the same CSS stack, there are some sizing differences between developer.fermyon.com (bartholomew) and developer.fermyon.com/hub (vueJS). Need to tweak things so it does not 'jump' between section loads.
Additional Content Styling

example from spin message trigger
Add table formatting, lists, blockquotes, etc.


Add List Type

Lists are defined like Tuples in python:
(1,)
(1, 2, 3, 4);
(2, 2.5, 3, 3.5);
...
etc.

Update language guide data types

Commit: https://github.com/johneastman/boomerang/commit/550ac07760b6dfa0eab253bc980feb801f838792


Clean up character names for feature engineering/modelling and create new feature engineering script

The character names have a bunch of multiverse and secret identity jargon making them hard to use for for feature modelling and analysis. Split these apart in preprocessing and then add in a feature_engineering.py script into the workflow to support further transformations.

Thanks, new features seem promising. We just need to add description for new features in summary report.

Addressed in the team meeting.


Manage Source Datasets APIs

Approach
Source Dataset Model

id (uuid) PK
Source study id (FK to Source Study)
sd_info

Title
CELLxGENE dataset id
CELLxGENE dataset version
number of cells

Updated
Created

APIs
GET /atlases/id/source-studies/id/source-datasets

source dataset id
source dataset title
source dataset doi
source dataset publication string
source study id
source study title
cellgene dataset id
-cellgene dataset version
number of cells
updated
created

GET /atlases/id/source-studies/id/source-datasets/id
same as above
POST /atlases/id/source-studies/id/source-datasets/id

title

PATCH /atlases/id/source-studies/id/source-datasets/id

title only valid on non-cellxgene records

DELETE /atlases/id/source-studies/id/source-datasets/id

only valid on non-cellxgene records

Backfilling

When refreshing...
Extend the RFS to hold the CXG Datasets by collection id.
Go over every SS (as we usually do)
If SS has a CELLxGENE Link  (matches a collection by DOI), read the collections datasets and save them as SDs in the new table.

@hunterckx I made some additions to the above. LMK if you have questions.

@hunterckx and more updates... added fields to the API responses joined from the source study.

Complete


doc Fix typo component.ngdoc

The wrong field name was being passed into the $ctrl.update call in heroDetail.html resulting in the wrong behavior (name was being updated instead of location)

I signed it!


Integrate DFU flash in bluetooth page

It would be nice if the users could update the firmware of the Nicla Sense Me with a press of a button. Also an option to upgrade from the standard - firmware to the edge-ml one would be nice.

Upgrading the edge-ml firmware with a newer version should also be possible.


npm 3.3.8 and Browserified modules

I have a module with transforms defined in the package.json so that when I require this module elsewhere the necessary transforms get applied. This has broken since I updated to npm 3.3.8. The problem is that when the new package.json file is written after the npm install, the browserify transforms are no longer in the correct order. For example I use
"browserify": {
   "transform": ["jadeify", "babelify", "brfs"]
}

but after installed it is
"browserify": {
   "transform": ["babelify", "brfs", "jadeify"]
}

Which doesn't work because babel can't parse the jade files. It looks like the problem originates from here: https://github.com/npm/npm/blob/master/lib/utils/deep-sort-object.js#L6 and https://github.com/npm/npm/blob/master/lib/install/update-package-json.js#L14
I opened an issue with npm as well but thought you guys might want to know about this since it breaks a sort of important feature of browserify.
Issue with npm: https://github.com/npm/npm/issues/10063

+1 this is breaking browserify bundling all over the place.

Wow this seems like a big bug on npm's part if it's resorting arrays.

PR sent upstream: https://github.com/npm/npm/pull/10185

Same problem here...
Why don't just specify a config file just like babel ( .babelrc ) or webpack(webpack.config.js) does!?
It's way better than just throws all the configurations into the package.json......

So, to use browserify WHY you have to have a package.json? This make no sense for me.

This issue is related to a problem with transforms. It's not a place to gripe about browserify architecture.

This bug is fixed upstream in npm 3.7.0.


Add priority for routes

Hello! Please add priority for routes (aka https://github.com/jesseditson/fs-router)
// all routes are sorted by this property - the higher numbers are matched first.
// kind of like a z-index for your routes.
// note that equal priority will just sort based on the fs in the case of a collision, which is not guaranteed order on OSX/Linux
module.exports.priority = -1

Thanks!

Just being curious, do you have am example codebase where there's a use case for this? Would love to see a demo. Like in Next.js there's no such feature and I never missed it to be honest.

This is necessary when your route layers come with dynamics and it is not clear when [car].js and car.js will respond

Like in this case?
├── routes
    ├── [car].js
    └── car.js

No magic happening here, car.js will be higher priority by ExpressJS's native design https://gabrieleromanato.name/order-of-routes-when-using-parameters-in-expressjs
So I don't really see a need for this code feature (although it's now already merged into the codebase).

@matthiaaas actually, it won't be a higher priority. Please, take a look at the blog post you provided a little more attentive:

However, the above code doesn't work as intended because a parametric path inserted just before a literal one takes the precedence over the literal one.

Bear in mind that in ExpressJS there's no built-in way to declare our routes as a list ordered by priority.

So, @Rysakov1986 meant that file systems in the different OS are able to give you surprises with sorting of the list of the files after you read it. So, an example from a blog post can be written like:
├── routes
    ├── [post].js
    └── page.js
    └── index.js

and your code will define something like:
app.get('/:post', handlerPost);
app.get('/page', handlerPage);
app.get('/', handlerIndex);

and in this case /page path will be matched to the handler from [post].js file.

Ah I see, I'm absolutely wrong on this. But wouldn't it be much more elegant to build in that priority feature into this package as I initially thought Express would do it already?
I don't like that export priority syntax so maybe this would be a much cleaner solution?

Indeed, but to solve this issue in a more elegant way you'll need to adopt additional route matching to this package. Right now, this package just utilizes Express routing and leaves all these questions to it. To become smarter this package will need to do a lot more than now.
Please, take a look at how SvelteKit is fallthrough routes for the most similar case: https://kit.svelte.dev/docs#routing-advanced-fallthrough-routes

I think ordering based on a sum of a depth and specifity metric should work fine. This rule would sort the following file tree
├── routes
    ├── index.js
    └── posts
        ├── [id]
            ├── details.js
            └── index.js
        ├── index.js
        └── test.js

strictly into
/
/posts
/posts/test
/posts/:id
/posts/:id/details

After that maybe some alphabetic ordering needs to be done so there are no differences between different platforms.

The method described above is now live on NPM.


[BUG] yarn reassure not working

Describe the bug
Using this (https://github.com/callstack/reassure-examples/tree/main/examples/native) example project, but getting the below error when trying to run yarn reassure
reassure is present insider ndoe_modules
To Reproduce
Steps to reproduce the behavior:

clone the above repo
yarn install
go insider native in examples
run yarn reassure

Expected behavior
yarn reassure should work
Error
`
yarn run v1.22.18
$ Desktop/reassure/reassure-examples/examples/native/node_modules/.bin/reassure
internal/modules/cjs/loader.js:905
throw err;
^
Error: Cannot find module 'node:fs'
Require stack:

<EMAIL_ADDRESS><EMAIL_ADDRESS>at Function.Module._resolveFilename (internal/modules/cjs/loader.js:902:15)
at Function.Module._load (internal/modules/cjs/loader.js:746:27)
at Module.require (internal/modules/cjs/loader.js:974:19)
at require (internal/modules/cjs/helpers.js:92:18)
at Object. (Desktop/reassure/reassure-examples/examples/native/node_modules/@callstack/reassure-cli/lib/commonjs/commands/init.js:8:15)
at Module._compile (internal/modules/cjs/loader.js:1085:14)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1114:10)
at Module.load (internal/modules/cjs/loader.js:950:32)
at Function.Module._load (internal/modules/cjs/loader.js:790:14)
at Module.require (internal/modules/cjs/loader.js:974:19) {
code: 'MODULE_NOT_FOUND',
requireStack: [
<EMAIL_ADDRESS><EMAIL_ADDRESS>]
}
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

`

@Darsbir which version of node are you using?

@Darsbir which version of node are you using?

node  v14.17.1

Closing as not able to reproduce


Service does not show up when right-clicking on a link in Chrome

It works when you right-click on the contents of the omnibar, but not on a link to a Gist within a page.

Can't get this to work at all in Google Chrome v43.0.2357.124 (2357.124) on Mac OS X 10.10.3 (14D136). Is there an updated / new version of this service available anywhere by chance?


[DOC] install/docker page: versions

What do you want to do?

[X ] Request a change to existing documentation
[ ] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
On https://opensearch.org/docs/2.11/install-and-configure/install-opensearch/docker/ , it says
docker pull opensearchproject/opensearch:2
docker pull opensearchproject/opensearch-dashboards:2

Why 2 and not latest? What does "2" mean? Maybe the latest 2.X version? If I already have a 2.X version, does it update to the latest, or does it keep the one I have?
Also, this command is given for checking that the search server is active, and gets the version:
curl https://localhost:9200 -ku 'admin:admin'
but I don't see the equivalent for the dashboard. It would be good to know how to check the Dashboard version either via API/curl or in the Dashboard UI.
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.

Thanks, @smacrakis ! Latest, 2, and 2.11.1 are the same image so you will get the latest version of the second major release.

Thanks,
To me at least (ignorant as I am of Docker conventions), latest seems clearer than 2, which is pretty obscure.
Maybe that dashboard version command should also be mentioned on the page (especially since it isn't the same as the command for the server)? (I guess it's not a documentation issue that it comes out as unformatted JSON...)
I think that the Dashboard version is what you see by clicking on the question mark in a circle in the upper right of the Dashboards page to the right of the word HELP, but it's not 100% clear that that's the Dashboards version and not the back end server version (shouldn't both be listed somewhere?).

By the way, there is a typo in your curl command.
You wrote: curl http://localhost:5601/api/status -ku ‘admin:admin’.
But it should be: curl http://localhost:5601/api/status -ku 'admin:admin'.
They look almost the same, but the first version uses curly quotes, which give an error!

By the way, there is a typo in your curl command. You wrote: curl http://localhost:5601/api/status -ku ‘admin:admin’. But it should be: curl http://localhost:5601/api/status -ku 'admin:admin'. They look almost the same, but the first version uses curly quotes, which give an error!

Sorry, copy/paste error! I should have written it in code format.

@Naarcha-AWS - can you please work with Stavros to make the updates? Thanks!

I can do this @hdhalter if you like? Assign me and I can do the update if that's helpful. Thanks.

I believe this is not a documentation concern, so I am closing this issue. @smacrakis - if there is anything we can do, let me know.


on Windows 10 it doesnt go further than intro menu

Describe the bug

and restarting does not help
Steps to reproduce
just open app on win 10
Screenshots or additional context
No response

Have you installed the msi package?
I think I already figured out the issue. The msi package installs it to the Program Files folder, and the server tries to create the database there, where there is no permission.
The nsis .exe package should work fine, as it installs to AppData.
For now, I will remove the msi package and address this on the next release.

yes, this was due to .msi installation and yes. exe works fine.
thanks for help.


refactor: extract ACL related functionality

Remove the ACL-Features from the interfaces and instead implement them as extension methods implemented in "TestableIO.System.IO.Abstractions.Wrappers".
This removes the dependency on System.IO.FileSystem.AccessControl from the interface project.
Note: I had to change one test case (MockFile_AfterSetAccessControl_ShouldUpdateLastAccessTime), that the times are adjusted when setting ACL value, but this test was incorrect anyway (I checked against the real file system)!
This is part of the approach discussed in #883

@fgreinacher :
I finished part 1 of the next PR. Could you please have a look?

@fgreinacher : This time the two issues found by codacy are false positives.
I need these parameters as they are interface implementations, right?

I'll have a deeper look at this by the middle of the week.
@siprbaum Do you want to do another pass?

@fgreinacher :
I think it would be good to finish this PR before bumping the major version in #919, as this PR also changes the dependencies...

@fgreinacher : I think it would be good to finish this PR before bumping the major version in #919, as this PR also changes the dependencies...

@vbreuss I'd like to push out a clean v18 first. We can then just bump major again for this change, no problem :)

@fgreinacher : I think it would be good to finish this PR before bumping the major version in #919, as this PR also changes the dependencies...

@vbreuss I'd like to push out a clean v18 first. We can then just bump major again for this change, no problem :)

@fgreinacher: I merged the latest changes from main including the v18 refactoring.
I am not sure, if the change really needs a major version bump; it just removes a reference, from the interface and replaces it with extension methods. However, if somebody only uses the interface project (which didn't exist previously), the ACL-methods are now missing...

I am not sure, if the change really needs a major version bump; it just removes a reference, from the interface and replaces it with extension methods. However, if somebody only uses the interface project (which didn't exist previously), the ACL-methods are now missing...

It will break people that are currently mocking the methods on the interface level, so I'd rather bump major to not surprise folks :)
Could you update the description to include a BREAKING CHANGE notice like in https://github.com/TestableIO/System.IO.Abstractions/pull/919?


Javascript build fails on Master

scene/gui/check_box.cpp:62:13: error: cannot use dynamic_cast with -fno-rtti
if (dynamic_cast< ButtonGroup* >(parent))
^
1 error generated.
Please remove the -fno-rtti in platforms/javascript/detect.py or do not use the dynamic_cast test in this line.

https://github.com/okamstudio/godot/commit/9de37817bcd0fb53384539d1b1a772541cabaca1
Fixes this issue


New API object: /api/clip

Rough outline...
{
  "sessionId": "ABC123"
  "start":<PHONE_NUMBER>,
  "end":<PHONE_NUMBER>,
  "status": "PENDING",
  "url": "https://cdn.livepeer.com/recordings/ff14961b-684f-449b-9134-e82c1645379d/source.mp4"
}

It pushes a clip to RabbitMQ which will be consumed by Mist instances who will bake the clip.

Hey team! Please add your planning poker estimate with ZenHub @victorges @darkdarkdragon @ya7ya


Odd blue line showing up when using the datepicker

For some reason, a blue line (the "highlight" indicator) is showing up across the bottom of my screen on every page that I use the datepicker.

Have you seen this before? It happens after you pick a date, and close the picker.

Happens to me all the time. Similar to the Slider which also has lots of visual artifacts.

Potentially may get resolved with a materialize library upgrade #149

Believe that this was fixed in materializecss. Make sure you're pulling in ~0.97.0 and this should no longer be an issue


dartfix for old-style unnamed constructor references in doc comments

@bwilkerson brought up a great idea for unnamed constructor references in dartdoc:
Given the new way to reference an unnamed constructor (C.new), I think that dartdoc is going to start to recommend this way in doc comments (/// See [C.new] for details.), and deprecate the old ways of referring to an unnamed constructor. I think the old ways are:

[new C], [const C]
[C()], [new C()], [const C()]
[C.C] (I could be mistaken here)

(It's possible that it's desirable to keep around these ways in case they convey information, like using const.)
If dartdoc does deprecate those ways, we should have a dartfix for updating those references.

CC @jcollins-g

The listing of old ways is correct, including the third bullet point.   This looks reasonable to me, including the proposal to preserve the possibility of const.


How can I reference local GiftedMessenger from example?

Trying to change from require 'react-native-gifted-messenger to
var GiftedMessenger = require('../'); so that I can debug and play with GiftedMessenger.js
But I get an error: Unable to resolve module ...
Any ideas?

I tried copying all the files such as GiftedMessenger.js inside the examples folder, but getting other errors...

you need to npm install react-native-gifted-messenger --save into your react-native project. Then you can require as normal.

What I meant by local is my local clone of react-native-gifted-messenger.
I guess I could just use npm link ;)


Do not check DeprecationDate when comparing dependencies

Summary

This might be an alternative to #233 in addressing short comings when comes to comparing the metadata of a dependency.

DeprecationDate does not work with reflect.deepEqual for 2 reasons

After Marshal and Unmarshal, the time.Time cannot be compared anymore, since it will loose some precision because of the missing UnmarshalTOML
Even WITH the fixed unmarshaling, the compare will work, but the reflect.deepEqual will still fail because it compares the exact internal structure.

If the CPEs array is empty, those a arrays are not considered to be reflect.deepEqual

Use Cases

As discussed in #233, comparing the metadata of a dependency is probably not needed at all if sha256 is the same. But at least the comparison should be successful in those cases since the dependency caching currently depends on it.
Checklist

[X] I have viewed, signed, and submitted the Contributor License Agreement.
[X] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[X] I have reviewed the styleguide for guidance on my code quality.
[X] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

This looks like a cost-efficient and compatible alternative until #233 can be continued on a v2 branch. @dmikusa is there anything holding us back from merging it?


Add variable to download archive

Hello,
In some cases, the architecture of some exporters is not 386 (amd64 for example). Do I've added a default parameter that is "386" (for retro compatibility) and updated the download path to use this variable.

Nice! Is it possible to plan a release so we can get the role through Ansible Galaxy please?


Keep non-spatial attributes during SRDD computation

User should have the freedom to decide whether keep the non-spatial attributes during SRDD computation instead of dropping the non-spatial attributes directly. It is hard for users to recover the dropped attributes afterwards.

In JTS or plus, every geometry has those two methods: getUserData and setUserData, which can store non-spatial attributes.  BTW, I have read your paper, in which you do not take into consideration  of ellipsoid when you process the longitude and latitude. I think projection is a better choice when you process gis problem.

Thanks for your comment! It is very helpful. We may also consider the ellipsoid in the future.

This problem has been solved. Now JTSplus spatial objects all carry the original user input data. See this answer: https://github.com/DataSystemsLab/GeoSpark/issues/25.


BUG: Deprecation warning says to use non-existent symbols

Describe your issue.
About a year ago (commit 6b08e087171479b30fc2dc015d2074a24844789e) the optimize.tnc module gained some deprecation warnings, indicating that symbols of the form optimze.tnc.<SYM> were deprecated, use instead optimize.<SYM>.  Since none of these symbols exist in optimize directly, doing as suggested generates ImportError.
Reproducing Code Example
$ python -c 'from scipy.optimize.tnc import RCSTRINGS ; from scipy.optimize import RCSTRINGS'

Error message
<string>:1: DeprecationWarning: Please use `RCSTRINGS` from the `scipy.optimize` namespace, the `scipy.optimize.tnc` namespace is deprecated.

Traceback (most recent call last):
  File "<string>", line 1, in <module>
ImportError: cannot import name 'RCSTRINGS' from 'scipy.optimize' (C:\Program Files\Python310\lib\site-packages\scipy\optimize\__init__.py)

SciPy/NumPy/Python version information
1.9.3 1.24.1 sys.version_info(major=3, minor=10, micro=7, releaselevel='final', serial=0)

xref #17572

Thanks, yup, same thing.  My searches for deprecation, optimize and so on missed those.  Feel free to close as a duplicate, if it doesn't add anything to those already-existing bug reports.

Yes let's close it since we have already an active issue. Still thank you for reporting @efahl!


Please implement get_default_instance()

With Mbed OS 5.10 the network interfaces changed a bit, please implement get_default_instance() implementation for this driver, too.
Example can be found from within Mbed OS in the ESP8266 driver.

@JanneKiiskila is there a test case covering this in Greentea or someplace.  I've made the change but would like to quickly verify it before putting it back.

@JanneKiiskila and @SeppoTakalo I went ahead and put it in the repo, #9dab5190bbd82206c125bd493b91c6b82ff3c6d7.  I did not update the wnc14a2a-driver.lib in easy-connect though, wasn't sure if you wanted to try it first to make sure noting got broke.

Yes there are tests for this.
Basically if this works, you should be able to run Mbed OS socket tests.
See the test plan: https://github.com/ARMmbed/mbed-os/blob/master/TESTS/netsocket/README.md
You need to provide test config (mbed_app.json) similar to this ESP8266 test config https://github.com/ARMmbed/mbed-os/blob/master/tools/test_configs/ESP8266Interface.json#L18
Note the line that sets ESP to be default Wifi.
You should do the same for Avnet driver, and run the mbed OS socket tests as instructed in the README.md I linked.