Proposal to improve process for signing up for kbase account

Notes from Usability Testing session July 22, 2016
REGISTERING AND GETTING TO FIRST NARRATIVE
Recording: 20160722_144350_0000-1630.mp4
Usability Testing Session July 22, 2016
Tester: Allison Perrotta, MIT Grad Student, Alm Lab
Needs: Has microbial isolate genome read sequence.  Needs to assemble genomes, annotate, and do comparative genomics with clade members.
00:30 - 3 clicks to get to signup
00:40 - No idea what Globus is
01:00 - Doesn’t want to make an account for some service she isn’t familiar with.  Was expecting to sign up for KBase.  Especially doesn’t want to get another email address to keep track of.  Adding two email addresses is too many.  Not sure which will be used for communication.  Navigated past explanation of why using Globus.
02:00 - Password strength blocks only after submission.  Should say “Too Weak” instead of just “Weak”.
04:00 - Why am I linking globus to other account?  Then why did I have to make the Globus account at all.  Do you have some kind of sweet deal with Globus?
04:45 - Not clear which is primary identity.
04:45 - Uses back button for navigation
04:45 - Creates too many tabs
05:15 - Confused about Globus.  Starts Googling Globus.
05:45 - Confused about all this Globus usage such as managing Narratives, etc.  Seems like more than just a login.  Shouldn’t have to be aware of this at all.
07:00 - Over 6 minutes and still not using KBase.
08:00 - “Creeps me out that Globus will be seeing my files”.  Thought it would be just KBase.  Concerned about privacy of data.
08:30 - On User Group page.  Feels like reg questions were already provided.
10:20 - Joins “kbase_users”.  Wants to see if she knows any of the members of the ENIGMA subgroup.
10:30 - Not sure what to do.  Visit the group?
10:30 - There is only one subgroup.
11:25 - Scans user list by eye and can’t really find people because sorted by email address.  No search.  No way to build colleague network.  Don’t know people’s usernames.
12:25 - Wants to start using KBase, but is stuck trying to access it by visiting the group.  THINKS SOCIAL IS THE FRAMEWORK.  No path to using KBase!
14:50 - Guided her back to home page.  Immediately goes to “Sign In”.  Doesn’t read anything on home page.  All other information on home page useless because she dives ahead.
15:15 - Is on her Dashboard.  Doesn’t know what a Narrative is.  Can’t find info.  Gives up and clicks the biggest button.
16:00 - Is soothed by the loading page, probably because it’s clutter free.  Also likes the progress animation.
16:10 - “What is a Narrative” should be clear before clicking on the big button.  Maybe a summary next to the button in the empty dashboard?

Making this easy and fast should be TOP TOP TOP priority.  It is literally the first exposure of a user to KBase and a bad first impression and barrier to entry will lead to immediate and likely permanent user loss.

For what it’s worth, I can tell you that today in training the UGA researchers about KBase and SDK, the signup is a huge stumbling block. One of the three trainees was able to successfully signup and log into their account. Another was was able to create an account, but had a lot of difficulty in using Globus to reset their password (which I guess they forgot after a day…) - he eventually got it up and running after some nudging. The third person seemed to have the most difficulty. He had created an account, but couldn’t get his username to work in login. Ranjan and I trying searching for his username in the system but to no avail. He thought that he might have not fully created an account, so then he tried to create a new account, but Globus said that he did actually have an account with his email address, but he couldn’t recover the username easily. I think he ended up emailing Globus, but I’m not sure if they got back to him.


Simplify to_byte overloads

:notebook_with_decorative_cover: Description

Problem:

to_byte with an argument delegates to an impl function template to
provide a nice error when the type isn't an unsigned char. This level
of indirection is unnecessary.

Solution:

Use if constexpr and always_false_v to achieve the same goal.

:memo: Checklist

[X] All user-facing changes have changelog entries.
[X] The changes are reflected on docs.tenzir.com/vast, if necessary.
[X] The PR description contains instructions for the reviewer, if necessary.

:dart: Review Instructions

File-by-file.

This made me wonder: std::byte C++17, so we could switch to that if the standard libraries from our compilers support it.

This made me wonder: std::byte C++17, so we could switch to that if the standard libraries from our compilers support it.

Good observation. I think the min compilers supported by Vast should support std::byte right now.  I'll remove byte.hpp in favor of std::byte soon 👍

Awesome, looking forward to i!

Awesome, looking forward to it!

https://github.com/tenzir/vast/pull/1392 replaces vast::byte with std::byte as discussed. I think we can also replace vast::span with std::span for what its worth.

I think we can also replace vast::span with std::span for what its worth.

As I was writing my comment, I thought about that as well. However, std::span is C++20. Compiler support for std::span is available as of GCC 10, Clang 7, and Apple Clang 10. Once we're at GCC 10, we can do this. (I just tried to include <span> while using -std=c++17 and it works.)

I think we can also replace vast::span with std::span for what its worth.

As I was writing my comment, I thought about that as well. However, std::span is C++20. Compiler support for std::span is available as of GCC 10, Clang 7, and Apple Clang 10. Once we're at GCC 10, we can do this. (I just tried to include <span> while using -std=c++17 and it works.)

Ah right, std::span is C++20.  What's the process/timeline like for updating supported min compiler versions for Vast?  Do you have plans to migrate to using -std=c++2a?

Ah right, std::span is C++20. What's the process/timeline like for updating supported min compiler versions for Vast? Do you have plans to migrate to using -std=c++2a?

We're currently aiming to support Debian Buster (with Backports), and that only has GCC-8 at the moment.

[..] and that only has GCC-8 at the moment

That said, this is the only requirement at the moment that prevents us from bumping the minimum compiler requirement. In general, we want to support the latest compiler when possible.
But I don't know the timeline. Maybe @satta? What's the Debian roadmap for adopting GCC 10, if any, for buster backports? I could only find gcc-8 there.

But I don't know the timeline.

Wait until the next Debian version is considered stable, and then a while until it was adopted so we can actually make the switch. Maybe sometime later this year.

But I don't know the timeline. Maybe @satta? What's the Debian roadmap for adopting GCC 10, if any, for buster backports? I could only find gcc-8 there.

There is no "roadmap" for GCC10 in buster-backports as backports are not developed in an organized, directed fashion -- someone would need to take care of the backport and adopt it. There might be all sorts of gcc-glibc-etc. interdependencies going on that might complicate things, so don't expect anything with buster on it to get GCC10 anytime soon.


chore: dynamically pick up config change for cache setting

Fixes #191
Modifications
This PR updates it to dynamically pick up config change for cache setting.
Verification
Verified in a local cluster by update the includedResources field in config map and see the cache setting is reloaded.

can you look at the unit test error?

can you look at the unit test error?

fixed, thanks!


make tachyon version configurable

make tachyon configurable

@shivaram thanks for creating new branch.

@uronce-cc - This has a merge conflict. Can you bring it up to date with branch-1.4 ?

@shivaram done.

The merge still has some issues. For example there is still code which looks like
+<<<<<<< HEAD
+=======
+TACHYON_VERSION=0.5.0

+>>>>>>> 13efaac... update tachyon version

which should be cleared during merge

@shivaram wrongly git rebase...

@uronce-cc  - It still says 'We can’t automatically merge this pull request' because your branch is not rebased on top of (or merged with) the first commit in https://github.com/mesos/spark-ec2/commits/branch-1.4


Nodes do not snap to grid like node-red and as such, layout differs

Current Behavior
Nodes do not snap to grid like node-red and as such, layout differs

Expected Behavior
Nodes should snap to grid to improve fidelity
Steps To Reproduce
try out the built in demo npm run demo
Environment

FlowFuse version:
Node.js version:
npm version:
Platform/OS:
Browser:

improved in #21 - closing for now


Emacs complains about missing arch-dependent data dir

$ emacs
Warning: arch-dependent data dir '/opt/gnu.org/emacs/v29.1.0/libexec/emacs/29.1/aarch64-apple-darwin22.5.0/': No such file or directory

Same issue as https://github.com/NixOS/nixpkgs/issues/24445

thanks for the report. We're getting pretty good at fixing these sorts of relocatability issues now and the link will certainly help.


Subfolders in gH2AX_dir

If gH2AX_dir contains subfolders, the script crashes (in StackMaskin).

Works (I put folder and other files into gH2AX directory, no troubles occurred)


fix NCCL error with non-consecutive trainer gpus

What does this PR do?
Fixes #8139
Can be reproduced on master with the following command:
NCCL_DEBUG=INFO python pl_examples/basic_examples/simple_image_classifier.py --trainer.gpus "1,3"  --trainer.accelerator  ddp

The issue appeared after #7202, where a barrier was introduced right after init_process group but before the ddp model gets configured with device_ids. When calling torch.distributed.barrier on a number of devices, it is assumed that it is affecting devices 0, 1, 2, etc. However, when we want to trainer on gpus=[1,3] for example that's not the case.
The fix is to use the device_ids argument in the barrier call. I do not know why and how this works, don't ask me. But if you know why, let me know.
Partially related, which need to be informed about this fix:

#4705
#7817

Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
Note: this cannot really be tested, needs a > 2 GPUs machine!
[x] Did you verify new and existing tests pass locally with your changes?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

@awaelchli Should we add a test for this?

@kaushikb11 I can add a test but it will require >3 gpus and will never run on our CI (which only has 2). We could then ask Adrian nicely to manually run this test on every ddp-related PR, what do you think? xD

@awaelchli could we maybe mock the barrier call just to see if it is actually called with the devices instead of just the number?

Hello @awaelchli! Thanks for updating this PR.

In the file tests/plugins/test_ddp_plugin.py:

Line 65:1: W293 blank line contains whitespace

@kaushikb11 @justusschock thanks for your suggetions. I added a test, it's not particularly smart but it fails on master with NCCL error and passes here. Again, it will not run in our ci due to the gpu requirements, but I tested on another server by running the special test directly (master and this PR).

@ananthsub This was also my suspicion, and I first tried to set torch.cuda.set_device(local_rank) before the first barrier call  without success.
In the torch.distributed docs it is clearly written that it is the local rank by default for the DistributedDataParallel wrapper.

Please ensure that device_ids argument is set to be the only GPU device id that your code will be operating on. This is generally the local rank of the process. In other words, the device_ids needs to be [args.local_rank], and output_device needs to be args.local_rank in order to use this utility.

And it seems the barrier(device_ids=) need to match this. However, it is a bit a puzzle for me why the barrier needs this when other collective calls don't. I tried to set the device via


Support for processing the PDF diagnostic report

Good Morning,
I am testing a series of studies that contain files with the diagnostic report in pdf, although the file extension of the report is not pdf since this is in dicom.
Try another viewer and as you can see this render it normally:

And when I test it in the OHIF viewer the file is not rendered, as I suppose there is no support for this type of files:

Likewise try to load with cornerstoneWADOImageLoader and throws the following error:
"Uncaught TypeError: Can not read property 'encapsulatedPixelData' of undefined"

I send the study so that they can try:
pepita -<PHONE_NUMBER>0.zip
I would like to know if it is possible to add this functionality ?,
Thank you very much and I will be attentive to your response.

Hi Christian,
The file you have attached is not actually an encapsulated PDF. According to the DICOM dump (https://rawgit.com/chafey/dicomParser/master/examples/dumpWithDataDictionary/index.html) it is a DICOM Structured Report (SOPClassUID : "1.2.840.10<IP_ADDRESS>.<IP_ADDRESS>" [ Basic Text SR Storage ]). The viewer you are using is converting it to an image or PDF for display, I guess.
DICOM SR support is on the roadmap, but I don't have a timeline for it. I'll leave this issue open for now so we can track the feature.
Thanks for the sample data.

This stuff renders now, though the display UI could be made to look a bit prettier. Happy to take a PR to update the dicom-html extension.


[Question] Is there a way to extend or customize the ui from config?

Hi, let's say I want to make the btn class not too round.
I know I can do it by adding rounded or anything to override the default value, but I have to repeat this step every time I add a button.
I can't find any guide on the docs, so maybe this feature is not available?

I'm improving that part of docs.
There are 3 ways:

Using utility classes on each button (as you mentioned)
Apply CSS to all btns: https://play.tailwindcss.com/f6OJRDCWM7?file=css
Extending a theme and change --rounded-btn CSS variable: https://play.tailwindcss.com/SHUzujnDWq?file=config


feat: Missing types from type hints are now replaced by docstring types

Summary of Changes

If the analyzed code misses type hints for parameters or results, we now take over the type defined in docstrings, if present.
If defined types in type hints and docstrings are contrary to one another, we throw a warning for the user and, by default, prefer the type information in the code.
The warning can be enabled or disabled by setting the preference with -tsw. By default warnings are enabled.
The preference whether code or docstring type information should be preferred can be set the -tsp. By default code is the preferred source.

The description of the PR should be updated then this should be good to go.

@lars-reimann I updated the description.


Datamodel refinement

This code works towards #93 and #116, and also makes some progress towards #108.

The OWL converter needs some more work. The code freely swaps polarities of atoms, which is incompatible with the new strict separation of existential and universal variables.

This branch now merely needs some more tests in the core module to cover all of the new functions. But basic functionality is there.

Ah, Set.of is Java 9 and we are still based on Java 8. I will fix this in due course ...

Coverage decreased (-1.8%) to 79.574% when pulling 0a0ccfe44fd899d7d3a710f0764e6afe8565a640 on datamodel-refinement into 37846b01384d07c0f2ce9b03a67867edb2bbcd80 on master.

Using more Java 8 features is probably a reason for decreasing cobertura coverage here ... We would need to move to Jacoco to get this fixed.


Generating page images does not support Chinese titles？

Today I tried to publish an article with a Chinese title. When I forwarded it to Twitter, I found that the Chinese characters in the generated pictures had changed to "口口口口". How can I solve this problem to support these words?

Hi Bro, did you soved this? cloud you share me some message about this?

i found that, although i throw errors, but the content can still generated. but it still a issue, right?


Quests editor freezing up during attempted delivery quest.

(Required) What is your Quests version / build number (do not say "latest")?
3.3.4-b316
(Required) Please describe the problem and how to replicate it. Any console errors?
When attempting to make a delivery quest using a Citizens 2 NPC, after defining the item and amount of it to deliver and selecting 8. done, the quests CUI freezes up, and stops responding. The only way out is to Re-Log.
(Optional) Can you provide an example quest from quests.yml using Pastebin.com?
No, because the problem prevents the quests from being created.

Got the same issue please help :/

@Archaie What version of Spigot are you on?

1.12.2
but i fixed it its ok
just vault outdated version i had


[2011-03-21] Filesystem::sync() iterates through ALL SiteTree objects in memory

created by: @chillu (ischommer)
created at: 2011-03-21
original ticket: http://open.silverstripe.org/ticket/6534

Given that you can trigger a sync() from the UI (button at bottom left of AssetAdmin), this means a normal user action can cause PHP to exceed its memory, or in the worst case bring down a server running out of memory when having couple of thousand pages.
Rewrite to either find affected pages for link/image tracking, or iterate through sitetree in batches for ~100. This will most likely be eased by the ORM rewrite in 3.0, but would still create/cache large in memory footprints - so really we need to avoid a full DataObject::get().

Filesystem::sync is gone in 4.0, so, fixed. ;d


a11y - Large Loading spinner is very light in color

Could you use a darker color value for the large loading spinner so it has more contrast against a light background?
While not yet a requirement in WCAG there are suggestions of at least a 3:1 ratio for graphic elements.
https://www.w3.org/WAI/WCAG21/Understanding/non-text-contrast.html

What version of the Carbon Design System are you using?
8.10.1

Add labels
a11y

We will be addressing this in our v9 release coming June 1st. Thanks for making an issue.


Question - Inconsistent relationship query returns

Regarding the screenshot below:
So you can see I call patch_meeting.rels and get an empty array
Then I call evaluation.from_node which returns the patch_meeting object. So the relationship does exist and it is from the patch meeting thus patch_meeting.rels should return that relationship. Also worth mentioning this call does not alter data in any way.
On the next line I call patch_meeting.rels and now it returns the expected outcome. So it was like when I called evaluation.from_node it remembered that there is a rel there. Really sketchy behaviour.
Code example (inline, gist, or repo)
class Evaluation
  include Neo4j::ActiveRel
  include Neo4j::Timestamps

  enum eval_type: ['assignment', 'parked', 'resolved']

  from_class :PatchMeeting
  to_class :Shipment

  type 'EVALUATION_FOR'
end

Runtime information:
Neo4j database version: 2.3.5
neo4j gem version: 7.0
neo4j-core gem version: 6.1.4

I see that you closed the issue, but let us know if there's anything that's still needed

This was an issue with rspec lazy loading variables in let statements.
I had let(:evaluation) { Evaluation.create blehblehbleh}
With evaluation being a relationship between two objects it wasn't actually being created before I called the evaluation variable in my "it" blocks. Thanks anyways loving graph databases!

Awesome!
In case you don't know, RSpec also lets you do let! instead of let which basically executes the let every time (kind of like a before block)


Separates creating zip and uploading into two separate actions

Fixes #81

@davidcok, let us know if this has been completed and we can review it and  close the issue.
PS: Please see John's note to ensure that this is properly merged with the new updates on the main branch

Testing done with this package
Save ingestion package as zip works fine, however, upload as ingestion package throws this error (Manifest import ingests successfully though, so it is not an issue with the package):

The above bug moved to Issue #110


Print out issues without matching issuetype

Closes #29.
Prints out list of issues that do not have an issue-type that are tested in issue-strategies/index

@mtscout6 Since, at the time of writing this, there is a presentation going on, Matthew and I are going to head out. If there are any issues that you see with the PR, I'll edit it from home, since I get notifications on my student email.

Thanks @kthompson55 I'll get to this first thing Monday morning as that meeting went through most of the afternoon...

:+1: @mtscout6

LGTM


🐞 [BUG] : Missing Login Button on Website

Description
The login button is missing from the website, making it impossible for users to log in.
Screenshots

Any additional information?
Steps to Reproduce

Navigate to the website's homepage.
Look for the login button.
Observe that the login button is not present.

Expected Behavior
A login button should be visible on the homepage (or relevant pages) to allow users to log in.
Actual Behavior
There is no login button visible on the website.
@YadavAkhileshh
Labels : Contributor , GSSOC, Hacktober Fest, Level 2
🔴 Priority: High
What browser are you seeing the problem on?
Firefox, Chrome, Safari, Microsoft Edge
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

@YadavAkhileshh could you please assign it to me i will start working on it.

I would like to work on this issue. Could you please assign me this task


Add broadcast function to the API

This is a renaming and update of the existing xray.broadcast_arrays function,
which now works properly in the light of #648.
xref #649
cc @rabernat
Examples
Broadcast two data arrays against one another to fill out their dimensions:
>>> a = xray.DataArray([1, 2, 3], dims='x')
>>> b = xray.DataArray([5, 6], dims='y')
>>> a
<xray.DataArray (x: 3)>
array([1, 2, 3])
Coordinates:
  * x        (x) int64 0 1 2
>>> b
<xray.DataArray (y: 2)>
array([5, 6])
Coordinates:
  * y        (y) int64 0 1
>>> a2, b2 = xray.broadcast(a, b)
>>> a2
<xray.DataArray (x: 3, y: 2)>
array([[1, 1],
       [2, 2],
       [3, 3]])
Coordinates:
  * x        (x) int64 0 1 2
  * y        (y) int64 0 1
>>> b2
<xray.DataArray (x: 3, y: 2)>
array([[5, 6],
       [5, 6],
       [5, 6]])
Coordinates:
  * y        (y) int64 0 1
  * x        (x) int64 0 1 2

Fill out the dimensions of all data variables in a dataset:
>>> ds = xray.Dataset({'a': a, 'b': b})
>>> ds2, = xray.broadcast(ds)  # use tuple unpacking to extract one dataset
>>> ds2
<xray.Dataset>
Dimensions:  (x: 3, y: 2)
Coordinates:
  * x        (x) int64 0 1 2
  * y        (y) int64 0 1
Data variables:
    a        (x, y) int64 1 1 2 2 3 3
    b        (x, y) int64 5 6 5 6 5 6

Any comments? This needs a rebase, but is otherwise ready for review

LGTM.

thanks for the feedback, guys!

thanks for the feedback, guys!


Add Max Transfer Rate setting to Foundry

By adding this setting allows you to speedup save transfer speed when logging in to server

Can you link patch notes or whatever that have info on this setting? Its not on the foundry website for server configuration


3DCityDB Importer Exporter tool to import my own CityGML data failed, the data is all null

Hi,
I am writing my graduation thesis, "Based on CityGML 3DGIS Application Studies", I use the 3DCityDB Importer Exporter tool to import my own CityGML data into the 3DCityDB database, but it is wrong to show Failed to resolve XLink reference '# fme-gen-3010b1e6 -9537-4a87-8e67-f72ebcf0b4f3 ', I feel very depressed, how can I do?
Best Regards,
Jingwen

Your dataset contains an xlink:href reference to an object with a gml:id of value fme-gen-3010b1e6-9537-4a87-8e67-f72ebcf0b4f3, but there is no such object in the dataset. Thus, the reference cannot be resolved and consequently no data can be imported.
This seems to be a data issue in the first place. Please fix your dataset.

Ha, Claus was faster :)

Are there whitespaces in your XLink target ID. I see at least three? Maybe this causes the problem.
Which version of the 3DCityDB and FME (?) you are using?

Thank you for the enthusiastic help,
my citygml data is converted by the BIM data through the FME tool, when I use the FME tool to view the CityGML data, the display of gml_id can not be found in the file, import to 3DcityDB gml_id is also different.
When I see my original data, did not find XLink target ID have the space,
3DCityDB version is 3.3.0, FME version is 2<IP_ADDRESS>

This is part of my data and FME display:

@zhangjw5 can you share your CityGML file for testing?

Thanks for your reply,
Of course, this is my experimental data.
http://pan.baidu.com/s/1o7D3GU2

Thanks for sharing your dataset.
The dataset contains a lot of gml:MultiGeometry elements. CityGML supports the use of gml:MultiGeometry at few places, for instance, as geometry for gen:GenericCityObject or grp:CityObjectGroup features. So the dataset validates against the CityGML schema.
However, the 3DCityDB lacks support for gml:MultiGeometry. That's why the geometry is not imported and why XLink references cannot be resolved. The latest version 3.3 of the Importer/Exporter logs a corresponding error message when importing your dataset:

[16:20:35 ERROR] GENERIC_CITY_OBJECT 'UUID_cbed4c39-917f-4e5b-b4b1-c39e7a138862': Unsupported geometry type MULTI_GEOMETRY.

To solve this issue, you have to change your FME process to write another geometry type such as gml:MultiSurface instead of gml:MultiGeometry.

Yes, thank you very much!
I tried to import another model and successful, and export the gml file is also open.
However, there have a new question: when I export the date for KML / glTF, it exports nothing.
Then I checked the column ENVELOPE of the table CITYOBJECT,
It is null, I tryed and recalculateed all values using the Importer / Exporter tool,but it did not work.
What should I do?
I have a little bit of puzzles that I import into the database that many properties are null, but exporting the CityGML model in the FME view is correct.
This is my data:http://pan.baidu.com/s/1slxyXCL

I have tested the KML/glTF export of your shared data using both Oracle and PostGIS databases. Everything worked fine on my machine.
A successful KML/glTF export requires a number of parameter settings which are explained in detail in the 3DCityDB documentation. For a quick start, a step-by-step hands-on tutorial is very helpful.

Thanks, listen to your suggestion, I once again read the document carefully and try again,
I think maybe because of my SRID set up a problem, so it's shown like this:

This is my SRID and URN settings:

GML file is set as follows:

Hey,
so your file has the SRS 25833 and your 3DCityDB has 31467? Well, this does not fit together. You can change the database's SRID with the stored procedure change_schema_srid (see documentation - only update metdata - do not transform the coordinates as they are 25833). Then buildings should be placed correctly (only if the source data is correctly referenced, of course).
Cheers

I am very sorry, this is my mistake, I figure the wrong data, in fact, the data screenshots are like this:

Looking forward for your reply, thank you！

Hi,
I found the problem is not SRID, which is due to Google Earth's 3D building display simulation shows the problem.
Data export KML format still can not show, although I have all the models involved in the building properties are set, but still useless. I hope you can get your guidance, what should I do?

Make sure that the parameters on the Altitude/Terrain panel are set as follows:

Run the KML export again and load the exported KML files into Google Earth Pro. You also need to disable the Google Earth's "Terrain" and "3D buildings" layers

Thank you very much,
I tried again and again, the result is:

And I tried another way and the result was wrong:

What method can I use to correct the model？

I also encountered the same problem,would you solve it?

Sorry, I can not give you the right advice, although the exported KML file is like this, but I tried to open the file in 3dcitydb-web-map, the result is like this：

Hi everybody
I have problem to export to citygml.
I use FME and my data is shp.
When I import my gml data to 3DCityDB it says valid but when I port data to kml it was rong in shape and location. My coordinate system is WGS 84 , UTM  SIRD 4326.
When I import data to 3DCityDB it imports data but show this warning:
[10:41:39 WARN] gml:LINEAR_RING (unknown gml:id): Linear ring is not closed. Appending first coordinate to fix it.
this is my data:

But in the 3DCityDB it shows my boundaries coordinate like this:

And the exported data to kml is like this:

Any body help me?

I solved it gays.

Thank U;-)

Hi gays,
I have problem by importing building gml format to 3DCityDB.
This is my data:
CevreSelected1.txt
It's shows this error:
[14:17:47 ERROR] BRIDGE_WALL_SURFACE 'GML_': Skipping import since this is not a top-level feature type. 3dcitydb
Thanks.

Forgot about this one. As the log message said, you got features in your data which are encoded as top-level feature, but by the CityGML schema they are not. In your case, you are missing the bridge.
Incorrect:
<core:cityObjectMember>   <bridge:WallSurface ...   ... wall attributes   </bridge:WallSurface> </core:cityObjectMember>
Correct:
<core:cityObjectMember>   <bridge:Bridge ...   ... bridge attributes     <bridge:boundedBy>       <bride:WallSurface ...       ... wall attributes       </bride:WallSurface       ... more walls, roofs etc. to come here     </bridge:boundedBy>   </bridge:Bridge> </core:cityObjectMember>
Hope this can help you. How did you create the CityGML file, btw?

Hi sir
Thank you for yor answering.
I used FME Desktop.
If it possible say me how can I make true CityGML 3D format.
By which software.
Best regards.

You can use FME for that. I'm not familiar with it, but it seems you just forgot to add the bridge feature. Should be easy to fix.

Any body tell me how to create citygml by fme,my 3d model data format is obj.when i use fme to export gml,it cannot import to 3dcitydb,the error is:GENERIC_CITY_OBJECT 'UUID_cbed4c39-917f-4e5b-b4b1-c39e7a138862': Unsupported geometry type MULTI_GEOMETRY.and i don't know how to use fme fix it.

This is a well known issue :)
Please, take a look at issue #28

Thanks,now i can import data to citydb,but when i export it as gltf and load it by google earth,it displays discorrect.see the picture below.

Could you open your CityGML dataset using FME Data Inspector and check, if the same rendering problem also occur?

Yes,i create models from cityengine,then use fme and convert it to citygml,it may aslo discorrect in fme.how to solve it.thx!

To me, it looks like the order of ordinates is somehow flipped ... so not XYZ but ZYX or ZXY. I think, there is a transformer in FME to change the order of ordinates.

thank you ,i have solved it.it works good .but my citygml is different from the sample data,may lack some settings in fme ,can you give a example about how to use fme to create citygml ？

Maybe the FME 3D Solution Templates are a good starting point: http://www.virtualcitysystems.de/en/news/480-3d-solution-templates-now-available-in-the-fme-hub

Thanks,I have another problem.i created models by city engine,and then use fme to convert it to citygml,then import the citygml data to citydb,and next i export it to kml/gltf,everything looks so good.but when i loaded my data in the browser,the models are a little bigger than the orignal one.my coordinate system is 3847,the src info settings in citydb like this:
first:3857
second:urn:ogc:def:crs,crs:EPSG::3857,crs:EPSG::115700
what's wrong?

When Set Geo-location in SketchUp and then I convert SketchUp file using FME desktop. it cannot add a coordinate system in XML or GML file. Because of the coordinate system, I am unable to import or export data using 3DCity DB importer-exporter
this is the FME for SketchUp to XML converter file for lod3.
@clausnagel
skp2citygml_v1222.2 (backup).zip

Hi,
I am working on my graduation  final year project , "Based on CityGML 3DGIS Application Studies", I use the 3DCityDB Importer Exporter tool to import my own CityGML data into the 3DCityDB database, but it is wrong to show Failed to resolve XLink reference '# fme-gen-72e95666-c183-4dbf-a9cb-2fb8ebfe17ad', I feel very depressed, how can I debug such type of errors ?
Best Regards,
Wajid Ali

Hi Wajid,
Please, check our comments in this thread as your error report sounds "exactly" like the first post. If this does not help, you can still share a sample dataset so that we can reproduce the error and provide help. But, I'm sure you can already solve on your own if you reed the comments such as:

To solve this issue, you have to change your FME process to write another geometry type such as gml:MultiSurface instead of gml:MultiGeometry.

Good luck.

Sir,
DO you have any documentation where all the process of  creating sketch up model and make a citygml  through (citygml2citygml_coordinateSystemChange.fmw) file and import it on the db importer exporter.

Not really, sorry. See issue #34 where we gave some hints on converting SketchUp models.

If the XLink reference cannot be resolved, then either the referenced object does not exist in your dataset or the reference itself is bad (at least there shouldn't be a blank after the hash). As @FxKu wrote we can check your dataset if you share it.
For questions and bugs related to Sketchup or FME processes, please contact the corresponding support teams.

Dear @anwar1010 . This issue has been closed already. Please open a new one referring to the bounding box "error" and I will provide an answer :)


HTTP download error + MKV merge error

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

Hello,
What does it mean [download] Got server HTTP error: HTTP Error 404: Not Found. Retrying fragment 104 (attempt 8 of 10)... ?
And why :
youtube-dl -v -f bestvideo+bestaudio/best --download-archive archive.txt --merge-output-format MKV -i --all-subs --embed-subs --add-metadata --write-info-json --write-description -o "%(uploader)s/%(upload_date)s - %(title)s %(id)s.%(ext)s" --yes-playlist
have some (for example) webm.part + m4a --> --merge-output-format MKV  then failed?


Update epoll.c

style : init some paramas

@Liketree
I have merge to https://github.com/Junbo-Zheng/cSocket, refer to https://github.com/Junbo-Zheng/cSocket/pull/2


Feature Request: Spotlight

Port spotlight feature from AVS. The button is B.
More info and documentations about AVS available at official forum topic (Maybe will be useful for ModelExtras Wiki)

I don't know if it works or not but the code is actually there from AVS port. It's set to B too.
(If broken, drop me a vehicle model that supports spotlights)

@user-grinch Just tested, it's not even working on AVS too or i can't find cars that have spotlight feature. So i don't know.


Function Calling Support for Gemini - Part 2

Why are these changes needed?
Finalize the Gemini function calling PR #2793 from @arjun-g, plus include support for VertexAI tool config for function calling.
Related issue number
Relates to #2387
Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[x] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[x] I've made sure all auto checks have passed.

@luxzoli Thanks! A couple of tests are failing in CI.

Codecov Report
Attention: Patch coverage is 1.97368% with 149 lines in your changes missing coverage. Please review.

Please upload report for BASE (0.2@3ebd7ae). Learn more about missing BASE report.

Files with missing lines
Patch %
Lines

autogen/oai/gemini.py
1.97%
149 Missing :warning:

Additional details and impacted files
@@          Coverage Diff           @@
##             0.2    #3726   +/-   ##
======================================
  Coverage       ?   29.15%
======================================
  Files          ?      115
  Lines          ?    12813
  Branches       ?     2431
======================================
  Hits           ?     3735
  Misses         ?     8733
  Partials       ?      345

Flag
Coverage Δ

unittests
29.15% <1.97%> (?)

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

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

@ekzhu should I prepare a new PR with the updated costs? They are indeed slightly different now for Gemini 1.5 Pro and Gemini 1.5 Flash


Cydia Support?

Cydia support please, I really don't want to import my tweaks. It will be a pain to do so

Cydia works on tweaks branch, but probably never rootless. Tweaks on Sileo are the same as Cydia


Unable to get installed...

So I have installed IIS and attempted to install PHP but web installer fails with a "Unable to download the product list from ." error.
After that I went ahead and looked at the web commander script that looks like it attempts to do the same for you but it seems to fail too.  I have already installed the latest PowerCLI build and the below is the output of the webcommander setup script.
Downloading PowerCLI, WPI and Advanced Logging...
PowerCLI, WPI and Advanced Logging downloaded successfully.
Downloading WebCommander package...
WebCommander package downloaded successfully
IIS is already installed.
Web Platform Installer is already installed.
WARNING: PowerCLI installer is not available.
WARNING: Please download and install PowerCLI manually.
Installing PHP...
WARNING:   Failed to install PHP.
WARNING:   Please install it manually, and run this script again.

If setup.ps1 doesn't work well on your server, please follow the instructions below to deploy webcommander manually.
https://github.com/vmware/webcommander/wiki/Installation-and-configuration-guide


image is display on app

image is not display on app and not showing in firebase storage and in firestore database


Two  are rendered one is materialize the other is normal

i have the following html in my template
<div class="input-field">
   <select materialize="material_select" [materializeSelectOptions]="stationResults" (change) = "filterBy('result', $event.target.value)">
                     <option *ngFor="let result of stationResults" [value]="result">{{result}}</option>
              </select>
</div>

and for some reason two select elements are rendered, attaching screenshot

@unaor can you share a repository that reproduces the problem? When trying the HTML you've pasted in the demo app it worked well.

sure @rubyboy
https://github.com/unaor/angular2-materialize
just clone and ng serve and you would see it right away

@unaor the problem is with the following CSS you've added:
select {
    display: block;
    height: inherit;
    border: none;
}

If you remove the display: block there will only be one.
This will also happen when using vanilla MaterializeCSS. The way MaterializeCSS works is that it creates a different HTML for the selection box. The original "select" is actually being hidden and is only used as a "shadow" to the one you see.
I suggest either removing your styling or checking with the MaterializeCSS library how to style the newly created components.

thank you you were right ,and indeed can't figure how to override the library created components


Use the same secrets for e2e than the org

This eases the secret rotation by using the same secrets we use to test the OHAIs.

Includes the PR #35 so all tests are able to pass.


--timeout times out during LLVMFuzzerInitialize

LLVMFuzzerInitialize can be used to initialize deterministic, but expensive global state, which is later used by the target read-only.
However, --timeout starts counting inside the init function, thus killing the process before any actual fuzzing can be made. I think the solution would be to only start counting in LLVMFuzzerTestOneInput.
Steps to reproduce:
$ cat test.cpp
#include <cassert>
#include <chrono>
#include <iostream>
#include <thread>

bool g_init{};

extern "C" int LLVMFuzzerTestOneInput(const uint8_t *buf, size_t len) {
  assert(g_init);
  assert(len < 1); // A crash
  return 0;
}

extern "C" int LLVMFuzzerInitialize(int *argc, char ***argv) {
  using namespace std::chrono_literals;
  std::cout << "Init expensive global state...\n";
  std::this_thread::sleep_for(1.5s);
  g_init = true;
  std::cout << "Done\n";
  return 0;
}

$ hfuzz-clang test.cpp -lstdc++ -o test.exe && honggfuzz --verbose -iterations=10 --run_time=2 --threads=1 --timeout=1 -i /tmp/a_init/ -- test.exe
Persistent signature found in 'test.exe'. Enabling persistent fuzzing mode
Start time:'2021-02-<IP_ADDRESS>' bin:'test.exe', input:'/tmp/a_init/', output:'/tmp/a_init/', persistent:true, stdin:false, mutation_rate:5, timeout:1, max_runs:0, threads:1, minimize:false, git_commit:ebc191e0525d5944f16d860aad49288b1bf6e448
Entering phase 1/3: Dry Run
Launched new fuzzing thread, no. #0
Persistent mode: Launched new persistent pid=1106642
[2021-02-15T17:53:27+0100][W][1106640] subproc_checkTimeLimit():529 pid=1106642 took too much time (limit 1 s). Killing it with SIGKILL
[2021-02-15T17:53:27+0100][W][1106640] arch_checkWait():232 Persistent mode: pid=1106642 exited with status: SIGNALED, signal: 9 (Killed)
Sz:4 Tm:1,001,219us (i/b/h/e/p/c) New:0/0/0/24/0/8, Cur:0/0/0/24/0/10
Persistent mode: Launched new persistent pid=1106643
[2021-02-15T17:53:28+0100][W][1106640] subproc_checkTimeLimit():529 pid=1106643 took too much time (limit 1 s). Killing it with SIGKILL
[2021-02-15T17:53:28+0100][W][1106640] arch_checkWait():232 Persistent mode: pid=1106643 exited with status: SIGNALED, signal: 9 (Killed)
Sz:8 Tm:1,099,895us (i/b/h/e/p/c) New:0/0/0/0/0/25, Cur:0/0/0/0/0/34
Persistent mode: Launched new persistent pid=1106644
Maximum run time reached, terminating
Sz:16 Tm:299,683us (i/b/h/e/p/c) New:0/0/0/0/0/23, Cur:0/0/0/0/0/34
Terminating thread no. #0, left: 0
Summary iterations:3 time:3 speed:1 crashes_count:0 timeout_count:2 new_units_added:0 slowest_unit_ms:1099 guard_nb:36 branch_coverage_percent:66 peak_rss_mb:4

Yes, this is known, but as you pointed out, the user code can hang in initializers, in LLVMFuzzerInitialize() and, in case it's not a persistent binary, but the one using HF_ITER in its main().
I'll think about it, because it's doable, but in case I add it, I'll probably have to have 2 timeouts, one for initialization, and one for LLVMFuzzerInitialize()


[testing] Debug doc build error on activation offloading PR

Copied a bunch of Jane's changes to see if doc build fails on this PR

closing for now since we pinpointed the error to exposing the APIs in training/init.py. Will reopen if we need to debug more


Mark render* methods as deprecated (and replace usage)

First step toward: #711

Thanks! 🎉

You're welcome!
I've also updated #711 so hopefully the diff is simpler 😉


worker fails if database is not ready

Worker would fail if PostgreSQL is not bootstrapped first

Thanks!
A better solution might be to make the worker not fail if the database isn't available. It should retry until it's available.

Closing for lack of activity


How do I add pins on ImageView

Maybe this isn't the right place to ask this question. I want to add some pins, as an ImageView(preferably, because I have to open an info window on click) or as a Drawable. I tried to subclass your view and draw a rect on onDraw but the rect is not scrolling. Do you have any idea how to do that? Could you show me a road to do so?

Please see the example class PinView

Wow, so sorry. I didn't see it.


Help! Choppy Playback Audio After Wake Word

I don't know what I did wrong but I need some help.
Firstly, I installed the Magic Mirror2, which worked fine.
Then, I installed AlexaPi from here: https://github.com/alexa-pi/AlexaPi/wiki/Installation
When I went through the install process and it asked which one to install, I chose magicmirror instead of the standard, default one. I added the module line from https://github.com/dgonano/MMM-AlexaPi to the config.json file. Now when I load MMM, it's just a blank screen.
Also, the Alexa voice integration is not working properly. I got my token from the Amazon Dev site.
Questions:

should I have not installed it that way?
if not, how do I uninstall the package and re-install the way from the MMM-AlexaPi?
Or should I just install the way of the MMM-AlexaPi? Will it overwrite the already installed version?

Thanks.
Any thoughts will help.

Ok. I fixed the blank screen by adding the MMM-AlexaPi module to the modules folder. Now I can't get alexa wake word to work.

Hmm try restarting alexapi if you haven't already. The wake word is independent of MM.

How do you restart AlexaPi independent of MM?

Or restart the Pi.
If you followed the install procedure for both you should be fine. If you still have an issue can you just list them and I might get you to post some log files

It seems I can get AlexaPi to work on the initial boot of the mirror (set to auto load at startup) but it seems to freeze after a bit. When I CTRL-Q to go back to the main screen and "pm2 stop mm"j or just let it load back up on its own, it doesn't work after that.
Also, the response is a bit slow and when she comes back with an answer, the audio seems to cut in and out? Power issue? I have a Pi3 model B with a Canakit case and supplied power cord. I'm not getting any power warnings in the OS.
Could the power be cutting in and out because the microphone is too sensitive? I don't feel like that is the answer...
]Thoughts?

Perhaps the playback_handler of vlc is what's causing it to be choppy since VLC is a very high demand app?

@dgonano would downgrading my Raspbian to Jessie Lite improve this perhaps? Will the mirror with AlexaPi still work?

To restart alexapi run sudo service AlexaPi restart
Just note magic mirror and alexapi are seperate applications, pm2 won't affect alexapi.
Is your internet connection slow? that currently causes issues with alexapi.
lets try,

restarting alexapi
maybe something to do with the HDMI audio fix mentioned in the wiki

I'm not sure about downgrading, I think I'm running the latest but i'm away at the moment so can't check.

Thanks for getting back to me.

My internet connection is pretty fast actually. I'm getting 30-60mbps depending on the time of day.
restarting AlexaPi only had the same problems

when I restart it comes up with [Warning: Unit file of AlexaPi.service changed on disk, 'systemctl daemon-reload' recommended]

I'm not currently using HDMI audio. I'm using the 3.5mm audio jack.

Also, I just overclocked my CPU to 1300MHz to see if that would work... and no luck. Still the same results.
Perhaps I'll make a video showing what's going on. Do you think that will help?

Video #1 - https://goo.gl/photos/KVb2RBvCK5YLk8GX7
Video #2 - https://goo.gl/photos/cgvE71U3QHAMEMyE8
I hope this helps.

I'm not really an audio expert because I never had any issues.
@renekliment are you familiar with this issue or another person who may know?

@lucallmon have you tried running AlexaPi in debug mode?
Kill MM and AlexaPi, then run sudo /opt/AlexaPi/src/main.py -d and events will be displayed in the terminal as they happen.  Perhaps you can learn something from what they say.

@maso27 how do you kill AlexaPi? I can stop MM, but not sure how to actually stop AlexaPi. Is it sudo service AlexaPi stop?

Yeah that should work.  I use sudo systemctl stop AlexaPi but I think it all comes out the same.
Or, if you'd rather use a big hammer, find the process with ps ax grep AlexaPi, then sudo kill xxx where xxx is the process number.

I keep getting this error:
Sending MM Heatbeat URLError:  [Errno 111] Connection refused INFO: cmn_live.c(88): Update from < 32.83  6.36 -22.48 -7.00 -8.68 -4.72 -12.31  8.11  6.79 -3.15 -0.96  3.09 -0.61 > INFO: cmn_live.c(105): Update to   < 32.63  6.95 -22.60 -6.89 -7.01 -4.07 -10.36  7.78  5.64 -2.70 -1.80  0.68 -1.31 >
thoughts?

Sorry.  I don't know MM at all.

Here's a different set:
Tue Jan 24 11:45:40 2017 Magic Mirror HTTP Server - :5055 Checking Internet Connection...  Why would the connection be refused?  @maso27 regardless of MM, is there anything that looks strange or off? I can post the entirety of the debug log if it will help. Thoughts @dgonano ?

 It looks like it's not running / running on another address or port / firewalled / ...

Yeah that's an issue between the two, I'll have a look when I get home later.
That's seperate to the audio though, Alexa should run fine even with those issues

Check your magic mirror is running on the same device at port 8080

Hhmmm. I'll go back and troubleshoot that.

So, check out the video: https://goo.gl/photos/eLsDTau2UgmPcrkh7
In debug mode, Alexa works great when she's on port 5050. it looks like MMM is running on port 8080. I tried to switch her to 8080 on the dev site, but it just shut down and didn't do anything. I don't know how to change the port in AlexaPi to 8080 or change the port in  MMM to 5050. Thoughts @dgonano @maso27  @renekliment ?

Check /etc/opt/AlexaPi/config.yaml, towards the bottom

I have to edit that in the terminal right?

You can use whatever works for you.  The only thing that could cause problems is the whole windows/linux/mac carriage_return/line_feed incompatibilities that crop up.
But if you're in the terminal, the easiest editor is probably nano.  So, type sudo nano /etc/opt/AlexaPi/config.yaml to get started.

Changing the config files did not change anything. Actually it made AlexaPi crash after the first use in MMM.

See video of everything working: https://goo.gl/photos/koXtZYN6jv4wLfcQA
So, Alexa Pi works great when incorporated in debug mode with an open terminal while MMM is running. Otherwise, if I just let it open at boot as originally configured, she continues to work the same... poorly.
Any additional thoughts? Perhaps setting a script to open terminal to run Alexa in debug mode at the start/boot? I don't know how to do this, but would like to know if someone does.

I'm expecting there's a significance in which user is starting AlexaPi.
The service is started by user alexapi, but when you start it from the terminal, with "sudo", you're starting it as root.
My bet is that there's some sort of incompatibility between users and groups for AlexaPi and Magic Mirror.
@dgonano, any thoughts?
@lucallmon this is a little heavy-handed, but you could try making the adjustments for "running as root" in the wiki and see if that helps.

 It works brilliantly right from startup!
I followed the SystemD run as root wiki and rebooted. And BLAM! Perfect!
Thanks for all your help! From a total NOOB, I appreciate it.

Yeah the communication between the two is done over tcp so users won't be an issue for integrating the two. Only for AlexaPi as you found.
Glad you got it working!
Is it showing the correct stuff on the mirror now?

Okay, this is all weird. I see no reason why running it as root would help, but if it does ...
@lucallmon If your issues are resolved, please close this.

Hey guys,
A bit late for the answer but concerning the error "URLError: [Errno 111] Connection refused" the solution for me was toto open my port 5055 in the router (portforwarding).
Cheers

Closing due to inactivity.


Appstudio update test-component-pac-umkb

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-umkb-on-pull-request has successfully validated your commit.


I have an issue with images working locally but not in netlify builds

Hey, great blog site however i get that my images even the default images are not being display when using netlify.
Is this something you have encountered?
https://640d0fe04eb66f00086728c7--eleijonmarck.netlify.app/blog/rustybucket-1/

I basically get 404 on all of the images

nvm i had a previous blog that had a gitignore on the /public folder. was easy to understand by going to the deploy on netlify and download the full site and seeing that assets was not part of the build


fix(batcher): patch race condition fix

This change is

#1533  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @alonh5 and the rest of your teammates on  Graphite


setup.py is missing tblib dependency

As the title says: tblib is used at https://github.com/tensorflow/tensorflow/blob/5e48683fe0983d5f35f1305f426d1b06904329a4/tensorflow/python/distribute/multi_process_runner.py but not declared as a dependency in setup.py
This is for current master and 2.4rc2

Cause I see that it is only related to CI

I see it is seemingly optional.
Maybe @crccw can clarify why/how that dependency is required?

It's only needed in multi worker tests.

Shouldn't this then go to the TEST_PACKAGES in setup.py?

Shouldn't this then go to the TEST_PACKAGES in setup.py?


feat: update to eye v3.5.1

feat: update to eye v3.5.1

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

npm package (@latest dist-tag)
GitHub release
v4.5.0

Your semantic-release bot :package::rocket:


Handle derivatives when not implemented

At the moment and maybe/likely forever, not all derivatives are available for all regression/correlation models in GaussianProcess.
We should implement a not yet implemented error which can be handled by client code.

Finally derivatives fir all kernels were implemented (#58, #60) meaning this issue is fixed.


[11.x] Notification middleware

This PR implements middleware support on queued notifications.
Why?
Very commonly, I run into rate limit issues with notifications, and ideally, I'd have the rate limiting logic bound with my notification. Notifications already support queued delivery through the use of the ShouldQueue interface and the Queable trait, so we are already leaning into jobs.
Use case: rate limiting
Rate limiting a notification requires the InteractsWithQueue trait to be imported, which allows the job to release itself onto the queue. In addition to that, a middleware() method should be added to the notification.
class MyNotification extends Notification implements ShouldQueue
{
    use InteractsWithQueue;
    use Queuable;

    public function middleware(): array
    {
        return [
            new Ratelimited('my-ratelimiter'),
        ];
    }
}

Other use cases
Personally, I have not had other use cases for notification middleware other than rate limiting, which is why I did consider building a notification-specific implementation of rate limiting. However, I feel like sticking to the existing syntax is better, especially considering the fact that queued notifications are already leaning on jobs.
Even so, middleware might unlock some new additional capabilities, such as content manipulation or conditional sending.

This seems to be passing the notification instance through the middleware... wouldn't job middleware expect a job instance?


UI Alignment issue for unreported nodes

The second column in Overview and Nodes pages align poorly when there are unreported nodes. See screenshot

This has been improved with the latest work I've done, but I think it's still not pixel perfect.

We perhaps need to reconsider using percentages for these elements.  Let's leave the issue open and re-examine it again before the next release.
Cheers
Fotis

I have made a slight adjustment to this alignment in https://github.com/puppet-community/puppetboard/pull/176/files#diff-13c11a71654ac212f4050d7cdb07b6b1, primarily to fix word-wrapping in the nodes page.

Closing due to lack of activity


Bump required RPC server API version to prevent hang.

While here, improve error messages for when an incorrect public
passphrase is used during wallet opening.
Fixes #8.

Incorrect passphrase does it for me.

is that an ok?

Yes with the grumbling that the word public may not add anything to the dialog message.


python312Packages.django-health-check: 3.18.2 -> 3.18.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.django-health-check is: Pluggable app that runs a full check on the deployment
meta.homepage for python312Packages.django-health-check is: https://github.com/KristianOellegaard/django-health-check
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.18.3 with grep in /nix/store/8yjbh48yddv5qj1h9rnbx7ns7cbv1fcq-python3.12-django-health-check-3.18.3
found 3.18.3 in filename of file in /nix/store/8yjbh48yddv5qj1h9rnbx7ns7cbv1fcq-python3.12-django-health-check-3.18.3

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath

baserow
python311Packages.django-health-check
python312Packages.django-health-check

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8yjbh48yddv5qj1h9rnbx7ns7cbv1fcq-python3.12-django-health-check-3.18.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.django-health-check https://github.com/r-ryantm/nixpkgs/archive/ce8730adbd6c1075e65e2843829591225df5dca0.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/ce8730adbd6c1075e65e2843829591225df5dca0#python312Packages.django-health-check

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8yjbh48yddv5qj1h9rnbx7ns7cbv1fcq-python3.12-django-health-check-3.18.3
ls -la /nix/store/8yjbh48yddv5qj1h9rnbx7ns7cbv1fcq-python3.12-django-health-check-3.18.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

4 packages built:

python311Packages.django-health-check
python311Packages.django-health-check.dist
python312Packages.django-health-check
python312Packages.django-health-check.dist

Maintainer pings
cc @onny for testing.

Result of nixpkgs-review pr 328761 run on x86_64-linux 1

2 packages marked as broken and skipped:

baserow
baserow.dist

4 packages built:

python311Packages.django-health-check
python311Packages.django-health-check.dist
python312Packages.django-health-check
python312Packages.django-health-check.dist


New endpoint for UI: Get all valid authrority attributes

Signed-off-by: Ofer Levi<EMAIL_ADDRESS>

@OferLevi85 please take care of the typo in your next PR.


Web component: Carousel - Mobile view, user not able to see the cards and pagination

Detailed description
Issue: On Mobile view, user not able to see the cards and pagination.
User has to scroll left and right to see the cards and pagination.
Additional information

Fix is on the way (#4767). Adopters can also work this around by explicitly setting the width to <dds-carousel>.


feat(network): block context in ReceiptResponse

Motivation
Ref #1002

Solution
Add block_hash and block_number to ReceiptResponse

PR Checklist

[ ] Added Tests
[x] Added Documentation
[ ] Breaking changes

fyi @popzxc


Update description/metadata

It probably doesn't belong to this module, but it would be great to be also able update the description and possibly other metadata, like screenshots and icons. Do you know of any module that does that?

I looked at the API and it looks like that's not available :(
Related: node-chrome-web-store-item-property can retrieve this data by scraping the site.


Grape re-loading instructions for rails don't work

Since rails 4, ActionDispatch::Callbacks.to_prepare is only triggered when files change that it knows about. The only direct workaround is to set reload_classes_only_on_change = false, although there may be a better fix.
https://github.com/rails/rails/issues/7152

I believe the documentation explains how to deal with this, https://github.com/ruby-grape/grape#reloading-in-rails-applications. Please reopen if that doesn't work.

Oh I see, reopened.


Infra/arithmetic2

Still needed :

Move geommean to math
Move div_180 to math
Move dist to arithmetic
Move deg_inrad etc to math
Eliminate ffs/fls
Remove old .cpp from core


Write blog post about accessibility project

Here's a start:
https://doimspp-my.sharepoint.com/:w:/g/personal/mchargs_mms_gov/EekIQSo96t5KswMFG4hLBJgB5_N5XCwR0uA-UP45YAiuhw?e=B5IlWK

@mcharg I put a few comments

@sghahn I started a post about how we prioritized the documents for compliance.  We can add in your part either as a section of the same post or as separate part as a part 2, depending on your preference.  We don't really have specifications for word count or hard deadlines.
We do have a guide for new blog authors, if that's useful.
https://github.com/ONRR/blog-nrrd/wiki/Blog-writing-for-new-authors

@mcharg I read your current blog write-up, this gives me a good start to content consideration. Let me take some time to contemplate what I'd like to write (maybe I can come up with something over the weekend), keep you posted.

@sghahn plans to have a draft started next week.

@Maroyafaied @mcharg. Just an FYI, I have finished writing the blog article and sent it to a coworker to proof read, so I hope to have uploaded in this issue by Friday.

@sghahn Awesome, thanks for the update and I'm looking forward to reading it!

@Maroyafaied @mcharg   I've attached the blog article for your team.
Please let me know if your team decides to use the article, also when and where should RRM look for publication?
Thank you, Shannon
Blog Article_508 Complinace_HahnShannon.docx

Thanks @sghahn I'll work on getting it formatted and send you a link to review it in dev.
One thing I'll need from you is a photo to include with you byline.  Basically any picture we can crop to a square.  See an example of mine at the bottom of this blog post. https://blog-nrrd.doi.gov/stakeholders/

@sghahn Also, what title should we use in your byline?

@mcharg just in case, the article was written more generic and internal to ONRR. By chance is this going on the nrrd site? If so, it may need to be re-written.
I'll see what I can do for a picture.

@sghahn It will be linked from the NRRD site.

@mcharg I don't really understand, if it's linked from the NRRD site, where is the final link, onrresouce?

@sghahn We have a separate site for the blog that is available to the public.

@sghahn I think we can make it work for an external audience without too many edits.  I added it to what I started about prioritizing docs.  Let me know what you think.
https://doimspp-my.sharepoint.com/:w:/r/personal/mchargs_mms_gov/Documents/NRRD Portal/Outreach/Blog/Accessibility.docx?d=w2a4108e9ea3d4adeb303051b884b0498&csf=1&web=1&e=NAMlCF

@CM805 @meshanw @cthomasONRR Can one or more of you take a look and provide comments?

@mcharg I reviewed and it looks good- I just changed some minor formatting issues.

@mcharg Very nice post. I reviewed and made a few minor notes as well.

@sghahn here is the long to our blog page if you want to get an idea of what it would like once it's published. https://blog-nrrd.doi.gov/

Got this put into Markdown.
https://dev-blog-nrrd.app.cloud.gov/accessibility/
@sghahn I had to rewrite a couple of the tips because Markdown can apparently only handle two levels of bullets. Let me know if you want to change anything.

@sghahn If it looks good to you, feel free to share it with Nichelle Rogers to get on onrresource or share it internally!

@mcharg looks great, and wow that was quick! I guess I better look for a picture of myself, I'll try to have it over by tomorrow. Also, I'm fine with the changes on the bullets.
Can you provide me a text copy of the final version so that I may coordinate with Nichelle to get it on onrresource ?

@sghahn I can get you a text copy of the final version tomorrow.  Not a super rush on the image.  We have two other blog posts coming out with our release this Friday, so this one will wait until we do the next release - probably when we get the next new monthly data.

@sghahn Here's a .txt version of the blog post.
Accessibility Blog Post txt version

@mcharg
Thank you for the text file and I haven't forgotten about the picture, will get that over soon.
@Maroyafaied  Also, I will get the blog article sent over to Nichelle to be posted on ONRResource.

@sghahn sounds good, thank you!


Video: Add documentation

Hello there, I was wondering this about the video extension: Does the extension support .mp4 only, or does it support .mov, or .m4v? And does it support streaming from an RTMP server?
If only there was a documentation for this extension, then I would know.

I think it supports any video file that can be played by your browser (so if you can open the video file in your browser and see a video, it will play with the Video extension).

It will work with direct downloads in any format your browser supports. It's just a <video> element so anything crazy like RTMP would not work. If that's possible to do in JS then there's nothing stopping s new extension from being added for that


why?

swipe to bottom is ok, there is problem swiping up

You're likely crashing because of the force unwrapped URL. I suggest you handle it more safely.
We use our GitHub project for bug reports and feature requests. In the future, you should open questions like this on Stack Overflow and tag alamofireimage.
Cheers. 🍻

From our Contribution Guidelines
Asking Questions
We don't use GitHub as a support forum. For any usage questions that are not specific to the project itself, please ask on Stack Overflow instead. By doing so, you'll be more likely to quickly solve your problem, and you'll allow anyone else with the same question to find the answer. This also allows maintainers to focus on improving the project for others.


Queue

Add a queueing action that groups actions together with shader parent config.
Follow up to #54

Readapt exec action.

Create Actions interface, contains all action signatures
Create ActionInvoker add actions to a queue
Implement Actions by Typewriter and ActionInvoker
exec returns an ActionInvoker by which the user can return custom action queue

actionInvoker.type('something')


[QUESTION] What about building the samples based on the latest SNAPSHOP ?

I'm creating this issue in this repo because issues are not available on the langchain4j-examples repo
Today the samples in langchain4j-examples are built on the latest final released (today it's 0.31.0) but what about building them on the latest SNAPSHOT (eg. 0.32.0-SNAPSHOT) ? That would allow building samples with the latest features and bug fixes without waiting for the next release.
I can create a PR on the langchain4j-examples repo, if you want, but the build system has to be ready for that.
Let me know if that's a good idea and if you want me to create a PR.

Hi @agoncal, I would prefer to keep examples on the stable version.

That would allow building samples with the latest features and bug fixes without waiting for the next release.

Good point, we could have a snapshot branch for this which can be merged into main after release?

@langchain4j that would be a good compromise. This way you can automatically run the snapshop branch each time there is a new PR merged in the main langchain repo, making sure all the samples still work or do not use any deprecated APIs. That would allow maintaining the samples, making sure they are up to date, and then merging the snapshop branch into main once a new version of LangChain4j is released

@langchain4j are the nightly snapshot builds published to any maven repo? I'd like to point to the latest snapshot in my project

@sjivan after each PR is merged, snapshots are published to maven central.
You can include this in your pom.xml:
<repositories>
        <repository>
            <id>snapshot-repo</id>
            <url>https://s01.oss.sonatype.org/content/repositories/snapshots</url>
            <snapshots>
                <enabled>true</enabled>
            </snapshots>
        </repository>
    </repositories>

And use latest version, e.g. 0.32.0-SNAPSHOT

Sweet. For Gradle users reading this, they can use:
repositories {
    mavenCentral()

    maven {
        url "https://s01.oss.sonatype.org/content/repositories/snapshots"
        mavenContent {
            snapshotsOnly()
        }
    }
}


package:analyzer's listing in the in-memory file system has terrible performance for large amount of files

package:analyzer version 2.2.0 with Dart SDK 2.14.1.
We are using the in-memory file system to temporary store the dartdoc-generated files on pub.dev. (The idea was to write a .tar.gz archive directly, instead of individual files). At the end of the process we've iterated over the files using getChildren() calls recursively. It turns out this is really inefficient for 100k+ files: takes more than 30 minutes (aborted process after that).
I've created the file list of the directory, attached as list.txt.gz.
list.txt.gz
Recreating that with empty content and scanning the files with recursive getChildren() call should help to reproduce the issue.

Related change in pub-dev that improved/fixed the performance issue: https://github.com/dart-lang/pub-dev/pull/5106

If I may say something. I am unable to reproduce.  By extending BaseTest, the below gives milliseconds for .getChildren()
What could be wrong with the test scenario?
Writing in 58 seconds
Getting in 360 millis

main() {
  defineReflectiveSuite(() {
    defineReflectiveTests(MemoryResourceProviderPerformanceTest);
  });
}

abstract class BaseTest extends FileSystemTestSupport {
...
}

@reflectiveTest
class MemoryResourceProviderPerformanceTest extends BaseTest {

  var chars = 'AaBbCcDdEeFfGgHhIiJjKkLlMmNnOoPpQqRrSsTtUuVvWwXxYyZz1234567890';
  var rnd = Random();
  String getRandomString(int length) => String.fromCharCodes(Iterable.generate(
      length, (_) => chars.codeUnitAt(rnd.nextInt(chars.length))));

  test_getChildren() {
    var filesCount = 100 * 1000;
    var contentLength = 12 * 1024;
    var stopwatch = Stopwatch()..start();

    var folder = provider.getFolder('C:\\');
    for (int i = 0; i < filesCount; i++) {
      getFile(exists: true, content: getRandomString(contentLength), filePath: 'C:\\$i');
    }
    print('Writing in ${stopwatch.elapsed.inSeconds} seconds');
    stopwatch.reset();

    for (var i in folder.getChildren()) {
      if (i is File) {
        i.path;
        i.readAsStringSync();
      }
    }
    print('Reading in ${stopwatch.elapsed.inMilliseconds} millis');
    stopwatch.stop();
  }
}

@asashour: see the repro code below:
import 'dart:convert';
import 'dart:io';

import 'package:analyzer/file_system/file_system.dart';
import 'package:analyzer/file_system/memory_file_system.dart';
import 'package:http/http.dart' as http;

Future<void> main() async {
  final rs = await http.get(
      Uri.parse('https://github.com/dart-lang/sdk/files/7169842/list.txt.gz'));
  final lines = utf8.decode(gzip.decode(rs.bodyBytes)).split('\n');
  final provider = MemoryResourceProvider();
  for (final line in lines) {
    if (line.isEmpty) continue;
    final path = '/tmp/${line.substring(2)}';
    final f = provider.getFile(path);
    f.parent2.create();
    f.writeAsBytesSync([0, 1, 2]);
  }

  final sw = Stopwatch()..start();
  var count = 0;
  void scanFolder(Folder rs) {
    for (final c in rs.getChildren()) {
      if (c is Folder) {
        scanFolder(c);
      } else if (c is File) {
        count++;
        if (count % 100 == 0) {
          print('[${sw.elapsed}] $count');
        }
      }
    }
  }

  scanFolder(provider.getFolder('/tmp'));
  print(count);
  print(sw.elapsed);
}

Thanks for your response.
The reason seem to come from the below loop, in which the logic iterates over all paths, and for each one if its parent is the current path, then add it as a child.
https://github.com/dart-lang/sdk/blob/4aa6373724302f852639af58fa53b541cef99a94/pkg/analyzer/lib/file_system/memory_file_system.dart#L605-L609
This would make recursive .getChildren() very heavy, as shown by your example.
One way to improve the performance, is by having a map of String path, List<String> children, however there could be better solutions.

Also, as a low-hanging fruit, maybe resource could cache the lazily evaluated dirname() of its path?

@scheglov

Thank you, I see the problem.

https://dart-review.googlesource.com/c/sdk/+/213682

The fix was landed, I see about 200-300 ms to process the list.

awesome, thanks!


_bearings_distribution: apply weight during histogram

Here is how _bearings_distribution currently applies the weight parameter:
https://github.com/gboeing/osmnx/blob/ea397d3aec77c7fb394712d5af6d490e0b159d4c/osmnx/bearing.py#L276-L277
_extract_edge_bearings implements the weight by repeating bearings:
https://github.com/gboeing/osmnx/blob/ea397d3aec77c7fb394712d5af6d490e0b159d4c/osmnx/bearing.py#L214-L220
Instead, the weight could be applied by numpy.histogram. The benefits would be:

avoiding a loss of precision when converting a float weight value to an int
greater computational efficiency and reduced memory usage

The resulting counts could have decimals (non-integers), but _bearings_distribution is already typed to return npt.NDArray[np.float64] for bin_counts.

Thanks. Sounds like a promising efficiency improvement. Would you like to open a PR for me to review?

closed by #1147


Add Italian translation

Hello, I added an it.lproj in order to add support to Italian language

I fixed a typo

@p0deje I only added the translated files you listed. Have no idea why the checks have failed.


Remove OMVRBTreeSet from csv persistence implementation

as title

hi,
This is done already in 2.2.x, closing


⚠ Turnos 24x7 has degraded performance

In 7b2ef30, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 1034 ms

Resolved: Turnos 24x7 performance has improved in 56386fb.


Fix function comments based on best practices from Effective Go

Every exported function in a program should have a doc comment. The first sentence should be a summary that starts with the name ({{funcName}}) being declared.
From effective go.
PR generated by CodeLingo. Install here to drive Continuous Higher Standards.

Hi @ant0ine and go-json-rest contributers,
@BlakeMScurr from CodeLingo here. This PR is testing out a new service. I'd love to hear your feedback or answer your questions. We're keen to learn how we can best help dev teams without getting in your way.
@BlakeMScurr & the CodeLingo Team

@ant0ine how valuable do you find these changes? You should pass golint now - is that worth much to you? How much better would it be if all the changes were like https://github.com/ant0ine/go-json-rest/pull/234/files#diff-541659943e3a6411fa01bdaa15aba143L53, rather than just adding a colon after the function name?


@import _variables.scss to each SASS file in project

What kind of change does this PR introduce?
Feature:
After using SASS in project, it's seem very essential to sharing _variables.scss in all SASS files, instead manually import it each time, which can be over head especially when styling Angular Components thats may locating deep in the project directories so the relative path to _variables.scss can be very long. (from experience)

Other information:
Using sass-resources-loader

@gilhanan I'm using sass resource loader, it's great.
The problem is that it is not play nicely with relative imports.
It requires you to add ALL virtual SCSS files into the resource array in the webpack plugin configuration which is something that adds a lot of noise to the starter.
I can live with that... but most devs won't know about it so they will just add @imports to the _variable file... we'll end up supporting sass resource loader.
About the commit
loader: `css-loader!sass-loader!sass-resources-loader?resources=${helpers.root('src/styles/_variables.scss')}```

This is getting too lengthy, we need to move it to an object based loader configuration.
It adds to my comment above, consider the need for multiple SCSS variable files...

This is a common thing, you usually divide these files to categories.

BTW,  i'm up supporting sass-resource-loader since in practice you can't work without it on a large project.
Maybe we should wait a bit for them to go out of beta...
Here is an example to a configuration using object
loader: ExtractTextPlugin.extract({
              fallbackLoader: 'style-loader',
              loader:  [
                'css-loader',
                'postcss-loader',
                'sass-loader',
                {
                  loader: 'sass-resources-loader',
                  query: {
                    resources: [
                       helpers.root('src/styles/_variables.scss')
                   ]
                  }
                }
              ]
            }),

I'll also point out that you can use imports from a main variable file, they just have to be absolute.
In my setup it:
  resources: [
                       helpers.root('src/styles/variables/_index.scss')
                   ]

But we can't put that on the starter, we'll end up confusing people.

Not everyone is going to want to import the same _variables.scss into all stylesheets. This isn't a convention I feel we should force on users.

@shlomiassaf Thanks for the feedback, I made another commit for changing the loader property to object based configuration.
About the PR, the meaning of _variables.scss is to declaring variables, so users allways can override variables which declared before so it's just advantage. And to prevent confusing I think a good comment will clarify it.


Goto Definition not working within an object constructor expression

Description:
Consider the following source and invoke the goto definition for self.item2, which does not identify the valid symbol.
function helloWorld() {
    var helloVar = client object {
        int item2 = 1;
        private string item1 = "";

        public isolated transactional function testFunction() returns int {
               int value = self.item2;
        }
    };
}

Affected Versions:
SwanLake Beta2 at least

Related to #31833


check for mints

I'm no python pro but this would be one less thing to manually check, one almost slipped through today.
If you can do a graphql query this will return their first minted objkt (or an empty array, hopefully).
Endpoint
https://api.hicdex.com/v1/graphql
Query
query ObjktsMinted($address: String!) {
  hic_et_nunc_token(where: {creator: {address: {_eq: $address}}}, limit: 1) {
    id
  }
}

Bad result:
{
  "data": {
    "hic_et_nunc_token": [
      {
        "id": 60343
      }
    ]
  }
}

Good result:
{
  "data": {
    "hic_et_nunc_token": []
  }
}

We implemented this a few days ago, I just hadn't committed. That info is available in bigmap 522.
def get_genesis(acct_id):
    url = f'https://api.tzkt.io/v1/bigmaps/522/keys?sort.asc=id&value.issuer={acct_id}&limit=1'
    r = requests.get(url)
    if r.status_code != 200:
        time.sleep(1)
        return get_genesis(acct_id)
    data = r.json()
    if len(data) > 0:
        return data[0]['key']
    return ''


Issue with /docs/tutorials/kubernetes-basics/deploy-interactive/

This is a...

[X] Feature Request
[ ] Bug Report

Problem:
The interactive screen should be the width of the browsers viewport because it's hard to read command output.
Proposed Solution:
Make the interactive screen as wide as the browser's viewport or wide enough to properly layout the displayed content.
Page to Update:
http://kubernetes.io/...

Thank you for your feedback on this. As recommended we updated the layout to give the terminal more space.
/close


Server Users

We need to support users that are not in any course.
The main goal here is to support things like users that can do administrative actions remotely, but this could also be used to support course users as well.
No work should be done on this  until #1 is complete.
A DB backend will change a lot of how we currently manage users.
We would even be able to have users that are consistent between courses.

Complete in c9a9940bceb5da5026464124d93f201508456e4b .
Not in main yet, but will be soon.


feat: add audio source

[x] Load mp3, wav and ogg
[x] only play current scene and global
[x] check loop mode works

Codecov Report
Attention: 139 lines in your changes are missing coverage. Please review.

Comparison is base (e35a264) 16.42% compared to head (6d2628e) 16.20%.

:exclamation: Current head 6d2628e differs from pull request most recent head 1b68e50. Consider uploading reports for the commit 1b68e50 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #59      +/-   ##
==========================================
- Coverage   16.42%   16.20%   -0.22%
==========================================
  Files          66       68       +2
  Lines        8353     8489     +136
==========================================
+ Hits         1372     1376       +4
- Misses       6981     7113     +132

Files
Coverage Δ

...aland-godot-lib/src/godot_classes/dcl_camera_3d.rs
66.66% <ø> (ø)

...nd-godot-lib/src/godot_classes/dcl_audio_source.rs
66.66% <66.66%> (ø)

...and-godot-lib/src/scene_runner/deleted_entities.rs
0.00% <0.00%> (ø)

...d-godot-lib/src/scene_runner/components/raycast.rs
0.00% <0.00%> (ø)

...t/decentraland-godot-lib/src/scene_runner/scene.rs
22.34% <15.38%> (-2.17%)
:arrow_down:

...traland-godot-lib/src/scene_runner/update_scene.rs
0.00% <0.00%> (ø)

...raland-godot-lib/src/scene_runner/scene_manager.rs
0.35% <0.00%> (-0.01%)
:arrow_down:

...ot-lib/src/scene_runner/components/audio_source.rs
0.00% <0.00%> (ø)

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


Swap Apex.AI TSC rep

Lyle is replacing Dejan on the TSC.

This pull request has been mentioned on ROS Discourse. There might be relevant details there:
https://discourse.ros.org/t/ros-2-tsc-meeting-minutes-2021-04-15/19972/1


Remove vscode folder

https://github.com/apache/arrow-rs/blob/master/.gitignore
https://github.com/apache/arrow-datafusion/blob/main/.gitignore
.vscode/.idea folder usually will be ignored.
Will create a follow up pr to add .vscode/.idea to .gitignore.

Do you have a use case/problem with this? I explicitly include the vscode settings for helpers like https://github.com/geoarrow/geoarrow-rs/pull/256
I personally don't see a problem with checking in vscode configuration

In my opion, everyone has their personal ide/editor setting, if these settings uploaded to repo, may disrupt other people's settings.

I generally agree, but these are pretty limited settings that are workspace-specific and that I think are valuable to be synced per-directory. I don't want to switch to a different computer and have different rust analyzer settings because it's not including all features by default

Ok, I use IDE, not conflict yet :)


Add tabs component

Add a tabs component
Todo:

[x] Write descriptions
[x] Add styling
[x] Make buttons dropdown on small screens if 3 or more tabs
[x] Add tests
[x] Make sure it's not called index when shown in menu to the left
[x] Make sure it's accessible

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Dedicated sub CHANGELOGs with Sync mode

For now we only have one root CHANGELOG for the monorepo. There is some interest of having sub CHANGELOGs :

more concise and readable
optimized for release bots (eg @releasebutler)
?

Fun fact: Lerna made this feature early, before they included a root CHANGELOG as an enhancement, see this.
It could be a new option for the version builder. We can use the standard-changelog package to generate sub CHANGELOGs.

In fact, the sub changelog is more important but it wasn't the quickest solution with standard-version 😊.
In addition, it helps users know why they update if some commits are generic or if they miss the scope.
It doesn't have to be option. Actually, what should be the option is the root changelog. What do you think?
I guess we should call https://github.com/conventional-changelog/standard-version/blob/master/lib/lifecycles/changelog.js that wraps standard-changelog and updates the file.

In fact, the sub changelog is more important but it wasn't the quickest solution with standard-version 😊.
In addition, it helps users know why they update if some commits are generic or if they miss the scope.
It doesn't have to be option. Actually, what should be the option is the root changelog. What do you think?
I guess we should call https://github.com/conventional-changelog/standard-version/blob/master/lib/lifecycles/changelog.js that wraps standard-changelog and updates the file.

It doesn't have to be option. Actually, what should be the option is the root changelog. What do you think?

Agree with that, but it's a major change. Let's go for it.

It doesn't have to be option. Actually, what should be the option is the root changelog. What do you think?

Agree with that, but it's a major change. Let's go for it.

Looking forward this feature!

Looking forward this feature!

@glebmachine I'm working on this, it will be available soon. :blush:

@glebmachine I'm working on this, it will be available soon. :blush:

@glebmachine We published a release including this feature.

@glebmachine We published a release including this feature.

I'm found that if no previous tag was set, cli command exits silently.
![image](https://user-images.githubusercontent.com/1610882/106267924-6c082e00-623b-11eb-8363-97c34a656c68.png
And one more thing)
Is it possible to calculate Changelog for each library in parallel?
I've got about 50 libraries and it takes for ages
Here concat can be replaced with combineLatest or similar
https://github.com/jscutlery/semver/blob/968356806a22c6676c63cd067d20628ae2ccf3f9/packages/semver/src/builders/version/version.ts#L33

I'm found that if no previous tag was set, cli command exits silently.
![image](https://user-images.githubusercontent.com/1610882/106267924-6c082e00-623b-11eb-8363-97c34a656c68.png
And one more thing)
Is it possible to calculate Changelog for each library in parallel?
I've got about 50 libraries and it takes for ages
Here concat can be replaced with combineLatest or similar
https://github.com/jscutlery/semver/blob/968356806a22c6676c63cd067d20628ae2ccf3f9/packages/semver/src/builders/version/version.ts#L33

@glebmachine I will investigate the bug, thank you. Nice suggestion for the parallelization. :+1:

@glebmachine I will investigate the bug, thank you. Nice suggestion for the parallelization. :+1:

@glebmachine I'm not able to reproduce the bug, can you send us a reproduction?

@glebmachine I'm not able to reproduce the bug, can you send us a reproduction?

I just opened an issue for changelog parallelization.

I just opened an issue for changelog parallelization.


ADDED : [iOS 13 & Swift 5 - The Complete iOS App Development Bootcamp]

@ishandeveloper Hello, can you please check my PR? Thanks


Solved Hugging Face cache issue.

Fixes #2276

I don't understand why would State restore need a transformers cache... We don't save anything that needs that.
I think the error and the restore are likely unrelated.

There are 2 issues in that. He described this in the "Describe the Bug" section.


dev.ts does not work when there is no islands/ directory in the PWD

Sometimes we just want routes/.

This already works in canary, I haven't tested it in stable. What error are you getting?

Can confirm that this is resolved in main but not released yet. The fix for this will be included in the next release.
@roj1512 The trace shows that you're using Fresh 1.2.0 . The fix will be part of the next release sometime this week.

Thanks, I will just use main now on!


Update README.md (trivial)

minor uri typo for the FAQ

Hey @truatpasteurdotfr Thanks for the PR!


Bug: Usage of deprecated numpy types

The usage of deprecated numpy types, such as np.float are causing problems as those are not available anymore in recent numpy versions.

was a bit too fast here, seems like this is resolved in !38


Cannot read properties of undefined (reading 'name')

Tryin to connect to our Dev box and getting the error in subject line.
Initial error window is:
Not connected to (systemname)! No response from server
Source: Code for IBMi
When I look at Runtime Status of Code for IBMi, I see "Cannot read properties of undefined (reading 'name')" errors.
Activation event: onStartupFinished
Port 22 is open.
I can connect to our production box fine.

Context
Version

Code for IBM i version
2.1.0

Visual Studio Code version
1.81.1

Operating System
win32_x64

Active extensions
Code for IBM i Walkthroughs (vscode-ibmi-walkthroughs): 0.3.1
Emmet (emmet): 1.0.0
Git (git): 1.0.0
Git Base (git-base): 1.0.0
GitHub (github): 0.0.1
GitHub Authentication (github-authentication): 0.0.2
JSON Language Features (json-language-features): 1.0.0
Merge Conflict (merge-conflict): 1.0.0
Microsoft Account (microsoft-authentication): 0.0.1
Node Debug Auto-attach (debug-auto-launch): 1.0.0
TypeScript and JavaScript Language Features (typescript-language-features): 1.0.0
WSL: Recommender (remote-wsl-recommender): 0.0.19

Not connected 🔌

Can you connect through a regular SSH client? For example, ssh from most shells, or putty, etc?

I'm not familiar with this. I'll do some research.

@rpxr400 the next step is to find out if the issue is due to SSH or due to Code for IBM i.
This error you're seeing is always shown and isn't unique to you, so I suspect some ssh issues.
Let us know when you manage to connect outside of Code for IBM i, and when you can, ensure your connection settings are correct.

Hi,
I have the same issue after Update to the latest version of VS Code (or the Plugin).

And on a second machine:

ssh connection works perfectly.
When I add a new connection in the Plugin, and name it like Server 2 (because the name is already taken (so it can read it somehow), and connect immediately instead of just save&exit, it connects and I can work like always. It does not show up and on the next try "Server 2" is already taken.
Best regards,
Stefan

@vuxat what language is your vscode instance configured to use? Looks like it could be related to locales?

@vuxat what language is your vscode instance configured to use? Looks like it could be related to locales.

@worksofliam my VSCode editors either run with english or german settings. de_AT.UTF8 maybe in the background?

Look what I found!

@worksofliam thank you!

@vuxat Waiting for the attached PR above to be merged and then we can create a release :)

@vuxat 2.1.1 is released with a fix. Thanks!


Apply patches v1.22.4

@bartvdbraak I noticed you've updated the locales for non-English languages. These will be overwritten during the sync from our translation tool Crowdin. If you'd like to contribute your translations, please visit https://crowdin.com/project/gitea. Please revert the changes done on these files. :tea:

Apologies for the accidental creation of this PR. I was performing operations on our fork and mistakenly created it. My sincere apologies for any confusion caused.


Spam - figure out some ways to prevent people submitting spam feedback

This isn't urgent--we've gotten a very small number of submissions.
Some ideas to explore if this happens in the future:

Show the feedback form a few pages in instead of on the first page
try some spam detection algorithms
filter out links in the submissions

Nothing seemed too actionable with this yet, but will tag this so we can track if it comes up in the future.

Reopening this and moving it to the Github Feedback. We're still getting a lot of spam entries, I'll probably do it as on-page validation, so if there are any false positives (which there shouldn't be many), we don't silently filter any actual feedback from users.


mitmproxy (10.0.0) is incompatible with pydantic

Problem Description
When using poetry to manage your project, you cannot install the latest version of mitmproxy alongside the latest version of pydantic.
Because mitmproxy (10.0.0) depends on typing-extensions (>=4.3,<4.6)
 and pydantic (2.1.1) depends on typing-extensions (>=4.6.1), mitmproxy (10.0.0) is incompatible with pydantic (2.1.1).
And because no versions of pydantic match >2.1.1,<3.0.0, mitmproxy (10.0.0) is incompatible with pydantic (>=2.1.1,<3.0.0).
So, because xxx depends on both mitmproxy (10.0.0) and pydantic (^2.1.1), version solving failed.

Steps to reproduce the behavior:

Create a poetry project
Add latest version of pydantic
Add latest version of poetry

System Information
N/A

If you upgrade to 3.11, things should work.

I am using 3.11, and it does not seem to work:
$ poetry -V
Poetry (version 1.5.1)

$ poetry env info

Virtualenv
Python:         3.11.4
Implementation: CPython
Path:           <repo>/.venv
Executable:     <repo>/.venv/bin/python
Valid:          True

System
Platform:   linux
OS:         posix
Python:     3.11.4
Path:       <home>/.asdf/installs/python/3.11.4
Executable: <home>/.asdf/installs/python/3.11.4/bin/python3.11

$ poetry add<EMAIL_ADDRESS>
Updating dependencies
Resolving dependencies... (0.1s)

Because mitmproxy (10.0.0) depends on typing-extensions (>=4.3,<4.6)
 and pydantic (2.1.1) depends on typing-extensions (>=4.6.1), mitmproxy (10.0.0) is incompatible with pydantic (2.1.1).
And because no versions of pydantic match >2.1.1,<3.0.0, mitmproxy (10.0.0) is incompatible with pydantic (>=2.1.1,<3.0.0).
So, because <project> depends on both pydantic (^2.1.1) and mitmproxy (10.0.0), version solving failed.

This is probably because your own package declares Python 3.10 as supported. If you bump that to ^3.11, things should work. :)
Next patch release should fix things otherwise. :)


Uncaught NetworkError: Failed to execute 'send' on 'XMLHttpRequest': Failed to load 'http://local...

Please help, I just installed better bib text to get this going. Details are as follows:
[Enter steps to reproduce below:]

...
...

Atom Version: 1.0.7
System: Mac OS X 10.10.4
Thrown From: zotero-citations package, v0.3.0
Stack Trace
Uncaught NetworkError: Failed to execute 'send' on 'XMLHttpRequest': Failed to load 'http://localhost:23119/better-bibtex/schomd'.
At /Users/user/.atom/packages/zotero-citations/lib/zotero-citations.coffee:22

Error: Failed to execute 'send' on 'XMLHttpRequest': Failed to load 'http://localhost:23119/better-bibtex/schomd'.
  at Error (native)
  at Object.module.exports.ZoteroScan.bibliography (/Users/user/.atom/packages/zotero-citations/lib/zotero-citations.coffee:22:12)
  at Object.module.exports.ZoteroScan.scan (/Users/user/.atom/packages/zotero-citations/lib/zotero-citations.coffee:95:14)
  at atom-workspace.subscriptions.add.atom.commands.add.zotero-citations:scan (/Users/user/.atom/packages/zotero-citations/lib/zotero-citations.coffee:8:89)
  at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:241:29)
  at /Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:61
  at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:524:16)
  at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:347:22)
  at HTMLDocument.module.exports.WindowEventHandler.onKeydown (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:177:20)

Commands
     -1:09.0 core:copy (div.package-readme.native-key-bindings)
     -1:04.7.0 core:paste (atom-text-editor.editor.is-focused)
     -1:03.6.0 core:save (atom-text-editor.editor.is-focused)
     -0:46.9.0 editor:consolidate-selections (atom-text-editor.editor.is-focused)
     -0:46.9.0 core:cancel (atom-text-editor.editor.is-focused)
     -0:10.9.0 core:select-all (atom-text-editor.editor.is-focused)
     -0:09.0 core:undo (atom-text-editor.editor.is-focused)
     -0:06.4.0 core:redo (atom-text-editor.editor.is-focused)
     -0:04.2.0 zotero-citations:scan (atom-text-editor.editor.is-focused)

Config
{
  "core": {}
}

Installed Packages
# User
autocomplete-bibtex, v0.6.0
markdown-toc, v0.3.0
markdown-writer, v1.9.0
zotero-citations, v0.3.0

# Dev
No dev packages

Have you enabled 'Enable export by HTTP'?

Yes I have.
I have 2 plugins - one in standalone and one in firefox.
I have enabled for firefox.
However, for the standalone the setting does not exist or refuses to show.

On 25 Aug 2015, at 3:16 pm, retorquere<EMAIL_ADDRESS>wrote:
Have you enabled 'Enable export by HTTP'?
—
Reply to this email directly or view it on GitHub https://github.com/ZotPlus/zotero-citations/issues/6#issuecomment-134599999.

That's because in standalone the feature is always on. I can't reproduce
the error currently; do you experience this with bbt in Firefox or in
standalone (not simultaneously, BBT doesn't work in that mode)?

I got this error when Firefox was not open. Possibly as simple as that?

That would do it, yes :)

:)


added another odin binding

You can change the name to whatever you want to because the binding doesn't have a name.

@AKDev21 thanks for the addition!