Increase Microsoft.Bot.Builder.Azure code coverage

Parent issue: # 5167
Description
Increase the code coverage for the Microsoft.Bot.Builder.Azure library.

Note: Originally the code coverage registered was 45%, but multiple tests were skipped due to not having CosmosDBEmulator locally installed and running in the machine, after that, it went up to 85%.

Related Pull Requests

Increase Microsoft.Bot.Builder.Azure code coverage

Current Coverage status

Promoted to MS # 5889


Downloading lists of URLs instead of images

Ripme version: 2.1.6-1-68189f27
Java version: <IP_ADDRESS>
Operating system: 10.0.19045.3570
Exact URL you were trying to rip when the problem occurred: https://www.reddit.com/user/ImOnHereToWasteTime/submitted/ (one URL of several)
Please include any additional information about how to reproduce the problem: Just running rips like normal

Expected Behavior
Download images
Actual Behavior
Create text file of urls
urls.txt

Had "Save URLs Only" checked. Not sure why that's even an option.


Change placeholder icon images to language-agnostic symbols

Currently using single-letter encircled icons for placeholder images, but the icons only make sense in EN-US; need to move away from these in favor of universally-recognized and language-nonspecific icons.

[x] Long Beam longBeam
[x] Charge Beam chargeBeam
[X] Diffusion Beam diffusionBeamA
[x] Spazer Beam wideBeam
[x] Wide Beam wideBeamA
[x] Wave Beam waveBeam
[x] Ice Beam iceBeam
[x] Plasma Beam plasmaBeam
[x] Nova Beam novaBeam
[x] Hyper Beam hyperBeam
[x] Thermal Visor thermalVisor - Full replacement
[x] Dark Ammo Expansion darkAmmoExpansion - Full replacement
[x] Light Ammo Expansion lightAmmoExpansion - Full replacement
[x] Phantom Cloak phantomCloak - Uses invisibility
[x] Pulse Radar pulseRadar - Uses scanPulse
[x] X-Ray Visor xRayVisor - Is there a necessary replacement for X ?
[x] Flash Shift flashShift - WARNING: no model for this pickup otherwise
[x] Energy Tank energyTank
[x] Energy Tank energyTankA - Base already uses E, is this necessary?
[x] Reserve Tank reserveTank - Armor up energyTank and remove the R?
[x] Energy Part energyPart
[x] E-Recovery Tank eRecoveryTank - Remove the E?
[x] Violet Translator violetTranslator - Need to replace color designations with Luminoth glyphs
[x] Amber Translator amberTranslator - Need to replace color designations with Luminoth glyphs
[x] Emerald Translator emeraldTranslator - Need to replace color designations with Luminoth glyphs
[x] Cobalt Translator cobaltTranslator - Need to replace color designations with Luminoth glyphs
[x] Dark Agon Key darkAgonKey
[x] Dark Torvus Key darkTorvusKey
[x] Ing Hive Key ingHiveKey
[x] Sky Temple Key skyTempleKey
[x] Unknown Item unknownItem - Is there a good replacement for ? ?
[x] Unknown Item 1 unknownItem1 - Are there good replacements for ? and numbers? Alpha the original and add ?? Shrink revealed item and encircle?
[x] Unknown Item 2 unknownItem2 - Are there good replacements for ? and numbers? Alpha the original and add ?? Shrink revealed item and encircle?
[x] Unknown Item 3 unknownItem3 - Are there good replacements for ? and numbers? Alpha the original and add ?? Shrink revealed item and encircle?
[x] Hint System hint - Is there a good replacement for ? ?
[x] Infinite Beam Ammo infiniteBeamAmmo - Replace infinity symbol?
[x] Infinite Missiles infiniteMissiles - Replace infinity symbol?

Alternatives drawn


To much time required

First of all, thank you for your amazing work on ControlNet. I have been using it for various projects and have really appreciated its capabilities.
I'm running the 5B model with edge control on 80G-A100GPU, however, during inference, it only takes around 4G memery and 10min to generate a video? Is there anything wrong or I miss somthing?

Hey :)
Have you tried turn off model_cpu_offload and sequential_cpu_offload?


Integrate react suspense in the UI components

We want to have the following behaviour:

Breadcrumb component triggers the data fetch
If the data hasn't arrived in less then 0.1s, show a placeholder

See also https://github.com/Semantic-Org/Semantic-UI-React/issues/2732

With the current state of things, I see React Suspense at least 2-3 years in the future, perhaps even more. Full compatibility is not guaranteed even in major architectural pieces such as Redux. Perhaps limited adoptions is what we really need. Baby steps.


Runtime error with OpenJDK 11

When running an application using MapJFX 2.7.3 and OpenJFX 11.0.2 under OpenJDK 11 the following runtime error occurs:
WARNING: java.lang.UnsatisfiedLinkError: Can't load library: /home/test/app/target/libprism_es2.so
Loaded library /libprism_es2.so from resource
WARNING: java.lang.UnsatisfiedLinkError: Can't load library: /home/test/app/target/libglass.so
Loaded library /libglass.so from resource
WARNING: java.lang.UnsatisfiedLinkError: Can't load library: /home/test/app/target/libglassgtk3.so
Loaded library /libglassgtk3.so from resource
WARNING: java.lang.UnsatisfiedLinkError: Can't load library: /home/test/app/target/libjavafx_font.so
Loaded library /libjavafx_font.so from resource
WARNING: java.lang.UnsatisfiedLinkError: Can't load library: /home/test/app/target/libjavafx_font_freetype.so
Loaded library /libjavafx_font_freetype.so from resource
WARNING: java.lang.UnsatisfiedLinkError: Can't load library: /home/test/app/target/libjfxwebkit.so
Loaded library /libjfxwebkit.so from resource
WARNING: java.lang.UnsatisfiedLinkError: Can't load library: /home/test/app/target/libjavafx_iio.so
Loaded library /libjavafx_iio.so from resource
WARNING: java.lang.UnsatisfiedLinkError: Can't load library: /home/test/app/target/libjavafx_font_pango.so
Loaded library /libjavafx_font_pango.so from resource
...
GraphicsPipeline.createPipeline failed for com.sun.prism.sw.SWPipeline
java.lang.UnsatisfiedLinkError: no prism_sw in java.library.path:
...

The same application runs without errors under Oracle JDK 11.
Is there any requirement to use Oracle JDK 11 for MapJFX? Are there any known issues with MapJFX when using OpenJDK 11?

Not that I know of. I am building and testing with OpenJDK on Mac and Windows. What OS are you using?

We have tested our application (build and testing) on macOS, Window 10 and Ubuntu 18.4/19.4 with OpenJDK 11.0.6.

I just tried on my Mac:
java -version                                                                                1 ↵
openjdk version "11.0.6" 2020-01-14
OpenJDK Runtime Environment AdoptOpenJDK (build 11.0.6+10)
OpenJDK 64-Bit Server VM AdoptOpenJDK (build 11.0.6+10, mixed mode)

I removed my local maven cache and built the demo app in version 2.7.3, so using the mapjfx-library from maven central.

Ok, thank you for the quick reply. We will check if this an issue with our application.


no objects

Hi .. this is a first effort - I gained the api key and the path .. but neither is there a green dot - nor do I see any objects. Help!

I see .. it is only cron driven .. I expected an update when I start the adapter


CSGen: Infer generic arguments in typed placeholders

To be rebased on #33663, only the last 2 commits are relevant.

I'm going to punt reviewing this and ask @hborla or @xedin to take a look instead.

@AnthonyLatsis Looks like you need to manually rebase...

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test Windows


Discovered entities with a name that starts with the device name is no longer allowed on HA

Description of the issue
The naming convention of HA has changed and the name of the device is no longer allowed on the entity name
refer to (https://developers.home-assistant.io/blog/2023-057-21-change-naming-mqtt-entities/) on the issue.
To my underrating, in the code of register_device.ts the device.name should be removed from the name field on both switch and sensor entities so the name should be:
name: "power" and name:  "activity" respectively
in addition, on the base entity on ha_mqtt.ts an 'has_entity_name' as mandatory attribute and it should be set to 'true' when publishing it.
Addon version
1.3.1
Platform
Home Assistant 2023.8.2
Supervisor 2023.08.1
Frontend 20230802.0 - latest
Logs of the issue (if applicable)
This is a the warning I am getting from HA
Some MQTT entities have an entity name that starts with the device name. This is not expected. To avoid a duplicate name the device name prefix is stripped off the entity name as a work-a-round. Please inform the maintainer of the software application that supplies the affected entities to fix this issue.
List of affected entities:
switch.ps5_807_power
sensor.ps5_807_activity

I would like to kindly but firmly ask you to check whether this issue has already been created.
Duplicate of #282.


goatools.associations.read_gaf function exiting early on illegal GAF lines (should just ignore)

Hi there!
The function read_gaf from goatools.associations can't read version 2.1 of 'gaf' files from the Gene Ontology Consortium (current version 2.1). (The functions works fine with version 2.0.)
Trying to read gaf files from the geneontology website give this traceback:
from goatools.associations import read_gaf associations = read_gaf("http://geneontology.org/gene-associations/gene_association.tair.gz")
TypeError                                 Traceback (most recent call last)
<ipython-input-10-1667bc12273f> in <module>()

----> 6 associations = read_gaf("http://geneontology.org/gene-associations/gene_association.tair.gz")

~/anaconda3/lib/python3.6/site-packages/goatools/associations.py in read_gaf(fin_gaf, prt, **kws)
    157     id2gos = defaultdict(set)
    158     # Optional detailed associations split by taxid and having both ID2GOs & GO2IDs
--> 159     gafobj = GafReader(fin_gaf, hdr_only, prt)
    160     # Optionally specify a subset of GOs based on their evidence.
    161     # By default, return id2gos. User can cause go2geneids to be returned by:

~/anaconda3/lib/python3.6/site-packages/goatools/anno/gaf_reader.py in __init__(self, filename, hdr_only, prt)
     64         # Initialize associations and header information
     65         self.hdr = None
---> 66         self.associations = self.read_gaf(filename, hdr_only, prt) if filename is not None else []
     67
     68     def prt_summary_anno2ev(self, prt=sys.stdout):

~/anaconda3/lib/python3.6/site-packages/goatools/anno/gaf_reader.py in read_gaf(self, fin_gaf, hdr_only, prt)
    141         hdrobj = GafHdr()
    142         ifstrm = nopen(fin_gaf)
--> 143         for line in ifstrm:
    144             # Read header
    145             if ntgafobj is None:

~/anaconda3/lib/python3.6/site-packages/goatools/base.py in ungzipper(fh, blocksize)
     98     import zlib
     99     uzip = zlib.decompressobj(16 + zlib.MAX_WBITS)
--> 100     data = uzip.decompress(fh.read(blocksize)).split("\n")
    101
    102     while len(data[0]):

TypeError: a bytes-like object is required, not 'str'

And using an extracted file, I get this traceback:
---------------------------------------------------------------------------
AssertionError                            Traceback (most recent call last)
<ipython-input-12-991e9ff245b5> in <module>()

----> 5 associations = read_gaf('/home/laurent/Downloads/tair_merged.gaf')

~/anaconda3/lib/python3.6/site-packages/goatools/associations.py in read_gaf(fin_gaf, prt, **kws)
    157     id2gos = defaultdict(set)
    158     # Optional detailed associations split by taxid and having both ID2GOs & GO2IDs
--> 159     gafobj = GafReader(fin_gaf, hdr_only, prt)
    160     # Optionally specify a subset of GOs based on their evidence.
    161     # By default, return id2gos. User can cause go2geneids to be returned by:

~/anaconda3/lib/python3.6/site-packages/goatools/anno/gaf_reader.py in __init__(self, filename, hdr_only, prt)
     64         # Initialize associations and header information
     65         self.hdr = None
---> 66         self.associations = self.read_gaf(filename, hdr_only, prt) if filename is not None else []
     67
     68     def prt_summary_anno2ev(self, prt=sys.stdout):

~/anaconda3/lib/python3.6/site-packages/goatools/anno/gaf_reader.py in read_gaf(self, fin_gaf, hdr_only, prt)
    156             # Read data
    157             if ntgafobj is not None:
--> 158                 flds = self._split_line(line, exp_numcol)
    159                 ntgaf = self._get_ntgaf(ntgafobj, flds, ver)
    160                 ga_lst.append(ntgaf)

~/anaconda3/lib/python3.6/site-packages/goatools/anno/gaf_reader.py in _split_line(line, exp_numcol)
    170         line = line.rstrip('\r\n')
    171         flds = re.split('\t', line)
--> 172         assert len(flds) == exp_numcol, "UNEXPECTED NUMBER OF COLUMNS"
    173         return flds
    174

AssertionError: UNEXPECTED NUMBER OF COLUMNS

Best,
Laurent

Laurent,
Thank you for your interest in GOATOOLS and taking the time to contact us regarding reading the GAF files.

If you did a 'git clone' to download GOATOOLS, could you do a 'git pull' to get the latest code and then re-try reading the gaf file? We have noticed that lately some GAF files have had 'inconsistencies' not seen before. We have seen the column error and added code to address it. Another new item we have seen is some GAF files lately have some lines that no longer have the "required" DB_Symbol. We have recently added code to read gaf files despite these new inconsistencies. Now, instead of reporting an illegal line and ending the program, we ignore the illegal line. In the case of DB_Symbol, we added an override so you can read GAF lines that are missing DB_Symbol if you want to.

Can you send us the gaf file that when read, produces the error?

Thank you again for using GOATOOLS. The gaf error may already be addressed. If it is, then I will ask that we release a new GOATOOLS version into PyPI, as the current version does not have our latest GAF updates.

Hi,
I didn't download Goatools with a git clone, but with PyPI indeed.
here is the file producing the error: https://we.tl/WjhdYvSMel

I just loaded your gaf file with our latest code:
  READ 260,258 associations: tair_merged.gaf
  27,226 IDs found in tair_merged.gaf

We do see 24 lines ignored due to a missing DB_ID . These messages look like this:
**ERROR: UNEXPECTED REQUIRED VAL() FOR COL(0):DB: DB() DB_ID()
**WARNING: IGNORED tair_merged.gaf[225854]:
		At1g21570		GO:0003723	PAINT_REF:23102	IBA	PANTHER:PTN000559726	F	Zinc finger CCCH domain-containing protein 7	UniProtKB:Q3ED78|PTN000559779	protein	taxon:3702	20170427	GO_Central

This has been a new thing we are seeing in the GAF files. DB_ID is definitely required, so these 24 lines will be ignored.  If you want to clone, you can get the latest code immediately.
Thank you for taking the time to report this issue. I will close it when PyPI is updated. Thank you for using GOATOOLS.

One more note...
Please download the gaf file and uncompress it before reading it.
I took the code out that automatically downloads it from the web and uncompresses it in the last version.
Please let us know if this an issue for you and you would prefer that the option remain to read the file directly from the web as a compressed file.

Thank you very much for your help.
I don't need the option of downloading gaf files from the web. However, as you removed this functionality, you should change the line in the notebook "semantic similarity" (3rd block).

The latest code is now available in PyPI. I am going to close this issue. If you see anything else, please do open a new issue.
Thanks for the heads up on changing the documentation. Thank you for using GOATOOLS and for taking the time to contact us.

FYI: Newly downloaded GAF files once again have illegal lines, but the format of the illegal line is new (missing middle columns).
I have updated our code to ignore these lines and will ask @tanghaibao to create a new PyPi version of GOATOOLS.
So if you again see GAF errors, please update your GOATOOLS package.

I also had troubles with reading a 2.1 gaf file. Optional last two columns make the trouble. They are missing in most lines, but gaf_reader.py exits as soon as the line does not have 17 values. I first tinkered to skip reading those columns. Reading was successful up to 1800 lines, when 16th column was filled. I cut -f 15 my file and it worked. Quick and dirty solutions, I think the code should allow missing values in 16th and 17th columns for 2.x gaf files, IMHO.

Thank you for your interest in GOATOOLS and thank you for taking the time to contact us.
Can you place your gaf file on a Google drive? Or email it to<EMAIL_ADDRESS>

I'm still having problems loading PomBase GO annotations (ftp://ftp.geneontology.org/pub/go/gene-associations/gene_association.pombase.gz) using GOATOOLS 0.8.2 from PyPI:
AssertionError: EXP 17 COLUMNS, SAW 15

Everything works fine when I install from git master though.

@tanghaibao,  Please make a new PyPI version. We saw some new forms of illegal gaf lines (which should be ignored) recently. The GOATOOLS master has the updates, but PyPI does not.
@harryscholes Thank you for taking the time to contact us and thank you for the specific example where you are seeing the error in code from PyPI.

@dvklopfenstein pushed to PyPi (v0.8.4) - let me know if anything is wrong. Haibao

@tanghaibao PyPI version works now, thanks. Do you think it would be worth including tests of loading some popular gaf files in the Travis builds?

So glad to hear that the PyPI version is working now.
We do have a test that loads ~40 gaf files (tests/test_termcounts_asscs.py). But it is not on our Travis-CI subset of tests because it takes a while to run and causes Travis-CI to timeout. It is a requirement to run by anyone who submits a pull request.
This test was passing. Then some GAF lines started to have an illegal format. We loosened the code to skip those illegal gaf lines. All was well. Then a new way for the gaf lines to be illegal popped up. We adjusted for that too. Hopefully, we are good to go for a while.
Thank you for taking your time to write us and thank you for your interest in GOATOOLS.


Improve GPU power attribution

What would you like to be added?
The current GPU power attribution is based on SM utilization. It doesn't account for memory usage. We should consider both factors.
Why is this needed?
Build a GPU power model that has both SM and memory utilization, so that workloads of compute and memory intensive natures can be attributed with fair power usage.
@marceloamaral @sunya-ch wdyt?

SM/Memory utilization models can also be extended to model MIG devices power consumption, if such devices cannot read power usage directly.

related issue #1198

@rootfs The problem with using memory utilization in the power model is that it only tells us about how memory is allocated, not how much it's actually being used for tasks like reading and writing data. These operations are what really affect how much power the GPU's memory is consuming.
We can get how much memory the GPU is using for reading and writing data, but currently, we can't do this for each process running on the GPU. So, it's probably best to focus only on the Ratio Power model using SM utilization for now. Another advantage of the simple Ratio power model is that it is not workload-specific compared to a trained model.

For the MIG, we can find which process is using the MIG device, so that Kepler will discovery the process/containers.
However, I am still investigating how to associate the GPU utilization to each MIG/Process.
NVIDIA dcgm has some logic for that, I am currently trying to understand that.
https://docs.nvidia.com/datacenter/dcgm/latest/user-guide/feature-overview.html


Command to "Open Changes"

Keyboard shortcut to see the SCM changes for the current file, is missing, even though there is a button in the UI.

You can define your own keybindings: https://code.visualstudio.com/docs/getstarted/keybindings

I was actually referring to the command, for some reason it was not showing up in the command list, but now it is. thanks!


[bug: #19, #22] Fix examples and v27 Jest support / ESM

Fix examples and v27 Jest support / ESM.

Resolves #19
Resolves #22

Changelog

Add new --jsdom flag / option

Enables the user to specify whether they want per-test environment config with @jest-environment data-block, or to default to jsdom within jest.config.json.

Ensure Jest DOM @types are only installed if both ts and jest-dom option are true
Add ESM specific configuration for Jest:

Use --experimental-vm-modules flag when running Jest
Enable useESM configuration property for ts-jest
Force .mjs transformer usage in jest.config.json for svelte-jester. See https://github.com/rossyman/svelte-add-jest/issues/19#issuecomment-916218761
Add .svelte extensions to Jest extensionsToTreatAsEsm; and .ts depending on the TS option

Improve example test files for a mixture of TS, JS and JS DOM configurations
Remove divergence of lib property within tsconfig.spec.json to extend from tsconfig.json

Just FYI, we're planning on bringing back CJS support to svelte-jester in https://github.com/mihar-22/svelte-jester/pull/71. I think that ESM is probably the better default for SvelteKit projects though, so this is still a good change. Possibly some people will want it as an option, but we can see how this goes


[Bug]: Version conflict between scipy 1.14.1 and statsmodels 0.14.2

Is there already an issue for this?

[X] I found no existing issue for this bug.

Describe the bug
See this scipy issue. For the time being, the workaround is to fix scipy to 1.14.0. We should periodically check if the incompatibility is gone with a newer version of scipy and/ or statsmodels and if that's the case, we can delete the corresponding version fixing in setup.py.
Environment
- OS:MacOSX
- Python:3.11
- ampycloud:2.0.0

Code of Conduct

[X] I agree to follow the ampycloud Code of Conduct.

This issue is fixed in statsmodels==0.14.3, see https://github.com/statsmodels/statsmodels/issues/9333

very cool, thanks a lot @nickodell for letting us know! I will adapt our dependencies accordingly.


lxd cluster edit

This PR adds the lxd cluster edit command, to manually edit the cluster configuration.
Executing the command will open up a yaml list of nodes:
- id: 1
  address: <IP_ADDRESS>:8443
  role: voter
- id: 2
  address: <IP_ADDRESS>:8443
  role: stand-by
- id: 3
  address: <IP_ADDRESS>:8443
  role: spare

The role and address values are writable, except in the case of spare.
I also blocked reordering of the nodes to make it a bit more readable from machine to machine. This can probably be removed if we ever decide to include name instead of id, but that would involve adding a column to the raft_nodes database.
Previously....

Interestingly, everything here was working up until this week, and now suddenly I can't get large clusters to elect leaders. Seems there's something that either I'm missing, or something with dqlite is acting up. This is particularly evident in the test script I added. Any cluster larger than 2 members seems to get stuck perpetually looking for leaders.

So as it turns out, I deleted a line setting the database dir to /var/lib/lxd/database/global and so everything was freaking out :)
Should be good to go now!

Build finished.

Build finished.

Build finished.

/lxc-ci/build/tmp.SM7gWAWayX/go/src/github.com/lxc/lxd/lxd/main_cluster.go:127:4: ineffectual assignment to newNode

Testsuite passed


gRPC swallowing other exceptions after context is aborted

What version of gRPC and what language are you using?
Python, 1.56.2
What operating system (Linux, Windows,...) and version?
Linux, MacOS
What runtime / compiler are you using (e.g. python version or version of gcc)
python:3.10.2
What did you do?
This caused some problems in our project because we were unaware some cleanup code in a try/except block was failing after the context had aborted. Any exception that happens in higher-level code after aborted has been set on _Context._state never gets logged.
Something like this in an RPC method will hit the issue:
        try:
            status = status_pb2.Status(
                code=code_pb2.ABORTED, message=""
            )
            context.abort_with_status(rpc_status.to_status(status))
        except Exception:
            pass
        finally:
            # This is swallowed because aborted is already set on _Context._state
            raise ValueError

This except Exception block here is not logging any exceptions that happen if _Context._state.aborted is set.
https://github.com/grpc/grpc/blob/cad2255d8c01625e394fa997fb7dbdf1968b2d91/src/python/grpcio/grpc/_server.py#L445-L449
Should it not log exceptions regardless, so that other code doesn't fail silently?
What did you expect to see?
Expected an exception/stacktrace from the code that executed after the context aborted.
What did you see instead?
Nothing. No logging of errors. It failed silently.
Anything else we should know about your project / environment?

Looks like it's a rare case where another exception happened in the finally block, you're right that we're not logging anything if it was already aborted, I'll create a PR to fix this shortly.


Rendering rule annotation (e.g. changed lines) in minimap and scrollbar is confusing

https://github.com/microsoft/vscode/issues/91250
Mode: "editor.minimap.mode": "cover"
Observer that the changed line annotation is rendered in the scroll bar and the minimap. I found that confusing and first thought it is a rendering bug.

This is intentional. There are other decorations, like errors, warnings, find matches, cursor position which are rendered in both places.
We could add an option to hide the scrollbar and the overview ruler entirely at #28653 , so I suggest we track in #28653.


Meta.get_annotation and extension methods

Pull Request Description
Attempt to fix #6955. Writing a test to check behavior of Meta.get_annotation and extension methods. The idea is to deprecate Meta.get_attribute and rather than that obtain the annotation for function itself.
That however requires to pass function as a handle thru the visualizations. The problem there is that saturated functions like to turn themselves into into results and we shall prevent that. Hence 5d87c9c introduces Meta.Function with its get_annotation parameter_name method.
Important Notes
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[ ] The documentation has been updated, if necessary.
[ ] All code follows the
Scala,
Java,
All code has been tested:

[ ] Unit tests have been written where possible.
[ ] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

So to be clear. The IDE would call: Widgets.get_widget_json value value.my_function ["arg1", "arg3"] ....

Wouldn't it have to be
Widgets.get_widget_json value (value.my_function ...) ["arg1", "arg3"]

instead?
I think we need to use ... to suspend defaults on my_function, not on get_widget_json (the latter I don't think should have any default arguments).

I don't know how the IDE is supposed to accomplish that by using the Visualizations API. Right now it is not possible to refer to multiple existing expressions in a single visualization call. See https://github.com/orgs/enso-org/discussions/6832 for more details.
Right now, the self argument's expression is passed using the expression ID (effectively passing in its evaluation result), but the method name is passed in as a string literal. If we extend the visualizations to support referring to multiple expressions (see discussion linked above), I could pass in the actual method application expression as well. But then, adding ... to it would not be possible anyway.

@JaroslavTulach What's the status here?

Still struggling and I will continue to struggle for the next iteration.

Will be fixed by #7115 - no need for the changes in this PR anymore.


How to install a package via git together with a dependency

Hi,
I need to install a package via
pipx install --spec git+https://github.com/saulpw/visidata.git@develop visidata

I would like to install at the same time lxml, that is dependency.
Thank you

Does injecting work?
pip install ... visidata
pipx inject lxml

Also, even if installing via git, the main package should correctly be specifying its dependencies (and they will be installed by pipx.)


"Dănești" is listed in the drop-down menu

Description:
"Dănești" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Dănești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Dănești, CĂLĂRAȘI (Frăsinet)".


Configuration error in localhost in geoportal-server-catalog (error status:500)

Do you have a tutorial or document on how to install Geoportal on the local computer with all requirements (elastic search) properly? The installation I did was not successful. When adding metadata this error occurred.

hi, you can find documentation here in GitHub on the wiki: https://github.com/Esri/geoportal-server-catalog/wiki
various installation and configuration topics are collected on this page: https://github.com/Esri/geoportal-server-catalog/wiki#installation-and-upgrade

@ShameeraJayawardana Link for elastic configuration
https://github.com/Esri/geoportal-server-catalog/wiki/Elastic-Search-configuration


super and sub script support

hi there
i'm using your library and would like to extend it to render some extended MD things
i thought i'd start fairly simple and was wondering if you'd like these in the main library
i realise i have slightly changed some core code but it passes all the core spec tests.
i'm happy to make any changes you suggest.

Thanks, but this project only parses the CommonMark spec or some extensions under flags.
For this case, I guess Github Flavored Markdown must have support for superscriptsa and subscripts, and then such a specification could be used under the ENABLE_GFM flag. Also note that your current implementation is breaking CommonMark for strikethrough.

thanks for the reply @Martin1887
i could activate these extensions only with ENABLE_SUPER_SUB flag or something

For this case, I guess Github Flavored Markdown must have support for superscriptsa and subscripts, and then such a specification could be used under the ENABLE_GFM flag.

I don't think super and sub script are in the GFM spec

Also note that your current implementation is breaking CommonMark for strikethrough.

I don't think strikethrough is in the CommonMark spec, but it is in the GFM spec
https://github.github.com/gfm/#strikethrough-extension-
and you are correct, my changes would break the GFM spec
i have extensions to links and spoilers that i want to do so i guess i will just put them in my fork

Uhm... it seems you are right about specs. I thought that all GFM features were under a flag but I guess developers then though that it cannot hurt.
Some specs have superscripts and subscripts, so simply the most accepted one should be found before merging the changes. @notriddle?

So it seems this could be merged after creating a new flag ENABLE_SUPER_SUB.
Moving the strike-through under the ENABLE_GFM would be a breaking change though, but it could be done in a 0.13 to align to the project's philosophy.

i have tried to put my changes behind a ENABLE_SUPER_SUB flag @notriddle , @Martin1887
please let me know if i need to do anything further

So strike-through is already under a flag, I understood from the previous messages it was not.
I have to check in detail the code changes but it looks good, thanks!

I would recommend separating Options::ENABLE_SUPER_SUB into Options::ENABLE_SUPERSCRIPT and Options::ENABLE_SUBSCRIPT because the syntax of subscript conflicts with GitHub's strikethrough syntax. To avoid the conflict, users may want to enable superscript only.

@notriddle @Martin1887 @rhysd
i have tried to address the issues mentioned above. this meant tweaking all the test rust files slightly with another optional test parameter
what do you think?

This seems like the right approach, yes.

It seems good, but some errors must be solved before merging. Thanks!

There is an error in cargo fmt  now, please format the code to avoid it. Thanks!


Logo aligment in embeds

@carlostallon commented on Fri Jul 08 2016

Hi guys, can you move down 8px the logo?

@xavijam commented on Fri Jul 08 2016
This is for CartoDB.js

Fixed in https://github.com/CartoDB/cartodb.js/commit/342857d7c0c27d63aa3b162e1b7c7c33e14acb3b!

thanks guys!


Cannot run go programs built with 1.21.5 on Windows Server anymore

Go version
1.21.5
What operating system and processor architecture are you using (go env)?
Windows Server 2008 R2

What did you do?
I built the simplest program possible (just a Println in the main func) with the new 1.21.5 release of go and tried to run the binary on different Windows Server 2008 machines.
I then installed go 1.21.4 and built the program with that version and see no errors.
What did you expect to see?
I expected to see the print on the terminal
What did you see instead?
for x64 build:
Exception 0xc0000005 0x8 0x0 0x0
PC=0x0

runtime.asmstdcall()
        C:/Program Files/Go/src/runtime/sys_windows_amd64.s:65 +0x75 fp=0x22fca0 sp=0x22fc80 pc=0x45e495
rax     0x0
rbx     0x530da0
rcx     0x583860
rdi     0x7fffffde000
rsi     0x22fea0
rbp     0x22fde0
rsp     0x22fc78
r8      0x0
r9      0x22fee0
r10     0x5556d8
r11     0x21
r12     0x22fec0
r13     0x1
r14     0x530740
r15     0x0
rip     0x0
rflags  0x10293
cs      0x33
fs      0x53
gs      0x2b

for x86 build:
Exception 0xc0000005 0x8 0x0 0x0
PC=0x0

runtime.asmstdcall()
        C:/Program Files/Go/src/runtime/sys_windows_386.s:35 +0x25 fp=0x18fe68 s
p=0x18fe64 pc=0x464065
eax     0x8
ebx     0x596e0c
ecx     0x0
edx     0x18fe90
edi     0x18fe6c
esi     0x18fec4
ebp     0x18fe6c
esp     0x18fe60
eip     0x0
eflags  0x10202
cs      0x23
fs      0x53
gs      0x2b

Go 1.20 was the last release supporting Windows 2008: https://go.dev/doc/go1.20#windows
The fact that your program built with 1.21.4 was running without problems may have been luck.


I got error message:

Invalid Parameter – yii\base\InvalidParamException
The file or directory to be published does not exist: Y:\xampp\htdocs\myweb/vendor/bower-asset/sweetalert2/dist

Hi!
Check composer.jason
https://github.com/Dominus77/yii2-advanced-start/blob/9b32ccbb57c3624e00c038609f845c10482fce33/composer.json#L39
and
https://github.com/Dominus77/yii2-advanced-start/blob/9b32ccbb57c3624e00c038609f845c10482fce33/common/config/main.php#L6
Read
https://github.com/Dominus77/yii2-sweetalert2-widget/issues/2

In this assembly, everything is set up and should work out of the box


No citus 6.1 version available in ubuntu 16.04

Hi I tried to install on my machine with Ubuntu 16.04 citus the way it is in your docs
curl https://install.citusdata.com/community/deb.sh | sudo bash && sudo apt-get -y install postgresql-9.6-citus-6.1
But there is only package postgresql-9.6-citus, which installs version 6.0.
Is it something wrong with the script deb.sh? Can I install it some other way?

Hi there. I am not able to reproduce the issue. Running the similar commands on Ubuntu 16.04 VM on EC2 installs Citus 6.1 just fine. Is there an error or any message you get when running either in the repo install or the package?

Yes, when I try to run
sudo apt-get -y install postgresql-9.6-citus-6.1
I get
E: Unable to locate package postgresql-9.6-citus-6.1
E: Couldn't find any package by glob 'postgresql-9.6-citus-6.1'
E: Couldn't find any package by regex 'postgresql-9.6-citus-6.1'
There is only package postgresql-9.6-citus available.
But what's more important, there is an error when trying to install repo (which is probably the cause of the problem):
Installing /etc/apt/sources.list.d/citusdata_community.list... bash: line 186: curl: command not found
Unable to run:
curl https://repos.citusdata.com/community/config_file.list?os=Ubuntu&dist=xenial&source=script
This is weird, because curl is installed on my machine.

Well, I had some unmet dependencies, running apt-get -f install fixed it and after that I could install repo successfully.


Some refactoring and twitch notifications API

Coverage decreased (-12.4%) to 43.931% when pulling 52f7cc321b055c0fe63e4289cfd8b9418dcd576d on development into 21a9d4343c572657cd898b299a876c73149a15a3 on master.


test-branch/execute-job/wrong-commit-sha-on-pr/adbe798c-0402-4147-b22f-8e89c7d6518d

PR for testing 1e7728fe2c3aa82b252fb837c48979f547a68c5d

/canonical/self-hosted-runners/run-workflows 9ce0b744df15004781dcf1966b1c85dd6407cbc6


BibEdit: Multiedit should check for emply files

Originally on 2013-02-15
Apparently in some cases multiedit generates empty files. There have been two situations so far on CDS, where multiedit sent bibuploads in correct mode for empty files.
Multiedit should make sure that no bibupload tasks are being generate in the case where no updates need to be done.

@kaplun @jmartinm Was this addressed in the INSPIRE OPS maybe?

Nope. I checked the code base and there is no such functionality to protect against empty bibuploads. On the other hand, does bibupload nowadays fail on empty input at all? (i.e. valid empty input with <collection></collection> tags).

Just checked on an Invenio 1.2.1+ installation that for the following input file:
<collection></collection>

the bibupload fails with:
2016-03-30 15:03:56 --> Task #65 started.
2016-03-30 15:03:56 --> Input file '/tmp/z.xml', input mode 'insert'.
2016-03-30 15:03:56 --> ERROR: Cannot parse MARCXML file.
2016-03-30 15:03:56 --> ERROR: Cannot parse MARCXML file.
2016-03-30 15:03:56 --> Task #65 finished. [RUNNING]
2016-03-30 15:03:56 --> Unexpected error occurred: ERROR: Cannot parse MARCXML file..
2016-03-30 15:03:56 --> Unexpected error occurred: ERROR: Cannot parse MARCXML file..

Nope, we are also having the same issue here.

I would tend to close it. Since we don't plan any further work on Legacy (at least on INSPIRE side)


Update GitHub Action deploy steps

What kind of changes does this PR include?

New or updated content

Description

Update GitHub Pages deployment to use the official withastro/action GitHub Action

Without testing the instructions, everything reads good to me, Nate!

I think your suggestion addressed that concern! I'm walking through the deployment steps right now to make sure everything works as intended!

When you're happy, @natemoo-re, I'm happy! It's yours to merge.  :)


Methods from NewMac trait in hmac not resolved

MacOS 10.15.5, VSCode 1.46.1, rust-analyzer .207 (.215 simply doesn't work at all here).
When rust-analyzer is enabled (and rls is disabled):

When rls is enabled instead:

Repro, with the sha2 and hmac crates:
use hmac::{Hmac, NewMac};
use sha2::Sha256;

fn _foo(secret: &[u8], _data: &[u8]) {
    let _mac = Hmac::<Sha256>::new_varkey(secret).expect("boop");
}

fn main() {}

It seems that new_varkey (from the NewMac trait) doesn't get resolved.

impl<D> NewMac for Hmac<D>
where
    D: Update + BlockInput + FixedOutput + Reset + Default + Clone,
    D::BlockSize: ArrayLength<u8>,
    D::OutputSize: ArrayLength<u8>,

That's a lot of where clauses that could potentially be the problem. Also, NewMac: Sized, so maybe we're just still missing the Sized impl (that can be checked by just writing impl<T> Sized for Hmac<T> {})

Triage: still doesn't resolve.

Ping...?

Triage: still doesn't resolve.

Works for me on master.


Optimize run on startup, even with default settings.

When building with the latest Kibana snapshots, the image runs the Kibana optimize step when starting.
Repro:
VERSION=7.0.0-alpha1-SNAPSHOT

mkdir -p artifacts/kibana/target/
(cd artifacts/kibana/target && \
  wget https://snapshots.elastic.co/downloads/kibana/kibana-${VERSION}-linux-x86_64.tar.gz)

mkdir -p artifacts/x-pack-kibana/build/distributions
(cd artifacts/x-pack-kibana/build/distributions && \
  wget https://snapshots.elastic.co/downloads/kibana-plugins/x-pack/x-pack-${VERSION}.zip)

ARTIFACTS_DIR=$PWD/artifacts ELASTIC_VERSION=${VERSION} make build-from-local-artifacts

docker run --rm -it docker.elastic.co/kibana/kibana-x-pack:${VERSION}

log   [06:28:46.549] [info][optimize] Optimizing and caching bundles for ml, stateSessionStorageRedirect, status_page, timelion, graph, monitoring, login, logout, dashboardViewer, apm and kibana. This may take a few minutes

cc @spalger

Welp, I was able to reproduce this one, but have not built both 6.x and master several times now and it always starts without optimizing... Perhaps there was something on the Kibana side that caused this and has been fixed, but I've looked through the history and can't find anything.
If you're still seeing this I image it's one of two things:

the plugin-install-time optimization is not producing bundles for all of the plugins, which is triggering the optimization at startup time. This is how the optimizer is intended to work, but we can prevent it by fixing whatever is changing between plugin install time and startup. To do this we need the optimize/bundles directory from before Kibana starts, and after it has completed optimization.

there is some condition that causes the Kibana server to not see the optimize/bundles/*.bundle.js file for each expected bundle. I imagine it's possible file permissions are to blame, but the Dockerfile template hasn't changed since early November so that seems unlikely...

Not usually one to necropost, but I'm seeing this still/again in 6.2.4


fix(excelutils): 取消负数标红

What this PR does / why we need it:

[ ] Smoke testing completed
[ ] Unit test written

Does this PR need to be backport to the previous release branch?:
If no, just write "NONE".
If don't know, write "UNKNOWN", and let the reviewer decide.
If yes, write the release branches name in the below format and submit the related cherry-pick PR:

release/3.10

/cc @ioito @zexi

/lgtm


[rust] Fix regex

The default regex should be good enough for rust. The previous one had some parenthesis issues, which this fixes.

:pray: :zap:


armel build: use FROM --platform

looks like Debian has set better manifest platform values, which
breaks our previous FROM usage (which was always wrong)

to clarify: this PR unbreaks armel building on forks without ubicloud runners.

replaced by #142


StackOverflow SVG icon request

Hi there. I couldn't find a StackOverflow SVG icon, so was wondering if it is straight-forward to add one?
I read previous issues so I'll take a crack myself as a local override when time allows.

Hi @steelecliftonberry,
the icon has been added in the last commit. Hope it helps.
Cheers


New buttons 🎨

Matching commit for https://github.com/TryGhost/Ghost-Admin/pull/529

@JohnONolan should this have a corresponding PR in Casper for the new styles?

Ignore the above, I didn't realise the private blogging and subscribe page load all of the admin styles.


moved validation message generation into a Helper method

This PR obsoletes my other pull request, making validation message generation more standardized by moving its logic to a Helper method. Attribute translations and custom validation messages (validation.custom.[attribute].[rule]) are honored.

Hi, i guess this request is extended version of this request https://github.com/bllim/laravalid/pull/12 which is accepted ? That's why, I am closing, if it is not please inform me.

@bllim this is the same as #12 but for master branch. #12 was for laravel4 branch

Really o.O I couldn't notice that sorry :\ I am accepting.


[Auto-generated] Check & update dependencies (pyproject.toml)

Update dependencies (pyproject.toml)
Automatically created PR based on the 'CI - Check pyproject.toml dependencies' workflow in SINTEF/ci-cd.

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

:exclamation: No coverage uploaded for pull request base (ci/dependabot-updates@b201103). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@                   Coverage Diff                    @@
##             ci/dependabot-updates     #384   +/-   ##
========================================================
  Coverage                         ?   82.17%
========================================================
  Files                            ?       44
  Lines                            ?     1257
  Branches                         ?        0
========================================================
  Hits                             ?     1033
  Misses                           ?      224
  Partials                         ?        0

Flag
Coverage Δ

linux
82.17% <0.00%> (?)

linux-strategies
84.50% <0.00%> (?)

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.


wip: Fix e2e yaml false success report

Changes
Even if a TaskRun, or a PipelineRun fails, the script will return
an exit code 0, which will show in CI as success. This fixes that.
Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ :no_good_man: ] Includes tests (if functionality changed/added)
[ :no_good_man: ] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

See the contribution guide
for more details.

(this should fail right now.. it does not :sweat_smile: )

Yay! Failure! =)

This should be green once https://github.com/tektoncd/pipeline/pull/804 is in :angel:

Well, let's do something about that. =)

/test integration

/test pull-tekton-pipeline-integration-tests
whoops wrong message

@abayer @bobcatfish ready to be merged :angel:

/lgtm


Fix Indexing Gemspec Awareness Bug

Motivation
Solves #1059
Fixes an issue in which the Indexer would hang on startup of the Ruby LSP.
Implementation
Rescued an error that occurred here, such that it no longer halts execution of the initial indexing request.
Automated Tests
I was looking for some tests around gemspec awareness in the configuration_spec file, but I couldn't find anything. I'd like to add a test if there's an appropriate place to do so.
Manual Tests

Boot up a rails app that does not download tzinfo-data as a dependency, despite being included in the bundle
Confirm the indexing request works correctly and completes with features that depend on the index working correctly.

Thank you for the fix!


Request: Vehicles choose random lane on roads if possible

Describe your idea
Vehicles would use random lane on a road, switching only when necessary
Screenshots?
None
Notes or questions?
Example:
Original toll booth vehicle counts:
Original                       Improved
16/00/00/00                04/04/04/04

How is this different from Dynamic Lane Selection?
https://tmpe.viathinksoft.com/wiki/index.php?title=Dynamic_Lane_Selection

Isn't it better if we have individual driving styles for drivers?

Dynamic Lane Selection already has individual driver styles, specifically:

Each driver gets their own 'pseudo-random seed'
That seed determines, for that driver:

A speed nerf/buff of their vehicle speed
When they will make lane change decisions

The DLS slider in mod options determines how many drivers will be affected (higher DLS means more drivers but at the expense of frame rate due to increased processing load).

It would keep single lane for as long as possible.

The lane lookahead gets computationally expensive and it would likely be non-viable to try and route it through junctions in realtime: https://tmpe.viathinksoft.com/wiki/index.php?title=Dynamic_Lane_Selection


Add all Bootstrap 4 event properties to custom events (in addition to relatedTarget)

I've got a pull request ready that I'll send after submitting this issue.
I'm using the Carousel component and was using the "direction", "from" and "to" properties on the slide event. These are described here in the Bootstrap 4 docs. I didn't see these in bootstrap.native so I decided to add it.
Here's what I've done so far in the PR:

Modified util/bootstrapCustomEvent so that instead of taking relatedTarget as the third function argument, it instead accepts a list of properties that are then copied onto the event. This is similar to how Bootstrap's custom event handler works.

Updated carousel to use this new syntax for the slide and slid events and to pass direction, from, to as well as relatedTarget. I've built and tested everything so far in Firefox and Chromium. Here you can see all the properties in browser console:

I updated all the other components (dropdown, modal, scrollspy, tab) that were passing 3 arguments to bootstrapCustomEvent() so that they work with the new modified bootstrapCustomEvent() syntax. However since I don't use these components I have not regression tested anything. I also haven't looked at the Bootstrap 4 docs to see if any other components besides Carousel add useful properties to the custom events that we would also want to add in bootstrap.native. (I could maybe work on that, but wanted to submit this first to see what you think)

Thanks for considering the PR. Of course if you prefer to implement this in a different way that would be great too.

Thanks for the PR. Much appreciated.
Please consider:

updating all components accordingly into your fork
updating the index.html in the Events of each component, especially Carousel, a quick draft would suffice for now; users might want to know how make use of the new functionality
any other suggestion on the functionality upgrade, since our implementation dates back to initial original jQuery plugins, find any opportunities for improvement

Thanks again.

Sure, sounds like a good plan. I'll make the index.html updates for Carousel and then have a look at updating all the other components with any other event properties in my fork.

OK, so that did not take very long at all, because there was not much else to do. For each of the 11 components I looked at the Bootstrap 4 versions and it turns out Carousel is the only one that adds any custom properties to events (other than relatedTarget). So no additional code changes were necessary.
Then for the 5 bootstrap.native components that I modified in my PR, I clicked through all the examples in index.html and verified in the Firefox console that when the events are triggered event.relatedTarget continues to be set to the correct DOM element. So that's one successful regression test.
I also click-tested the other 6 components (that I didn't modify) to verify functionality and see that they weren't throwing any errors into the console. (alert, button, collapse, popover, toast, tooltip)
I updated index.html with documentation for the new Carousel event properties and with a code example. That should show up in the PR.
I'll attach my build here just in case you want to verify your build is the same as what I tested.
bootstrap-native.js.gz

Alright, we're merge this commit and update the code the properties object would likely get renamed to something like eventProps to avoid potential conflicts.

Sure, that rename makes sense. Thanks!


[Placeholder] New onboarding experience and walkthrough

Description
A new contributor to OpenStreetMap can be overwhelmed with what to do. This can lead to people stepping away from OSM or potentially adding incorrect data to OSM. Rapid can strive to be the best experience for editors mapping for the first time.
Right now a new mapper might experience Rapid in the following way:

Sign up for an OSM account on OSM.org.
Come back to Rapid Editor.
Log in with newly created OSM account.
Start the tutorial if instructed to do so.
Start mapping whatever the proposed task is. The walkthrough helps provide some context, but they will still be quite overwhelmed.

I think there is huge value if Rapid can integrate 1-4 more closely. A new user might visit Rapid and be prompted with the log in/create account option. They would opt to create an OSM account, ideally without leaving Rapid (if that's possible). Following this step they would be taken to the tutorial.

Yea, I agree there is lots of room for improvement, and other people have been interested in improving the signup flow too..
Check out the comments on https://github.com/openstreetmap/openstreetmap-website/pull/4947#issuecomment-2210588415  ,
it looks like MapBuilder ran some numbers and found that they lose a lot of users by making them go to osm.org and complete the signup flow, even if they are already using an auth provider.
If I'm reading it right, the proposed change would streamline steps 1-2-3.

Thanks for linking that @bhousel. That's definitely an improvement.
I am hoping for more of an embedded sign up experience, but I can imagine OSM being reluctant to make it too easy to sign up on other websites.


Changes for archival

Changes for archival of edgehtml content.

Docs Build status updates of commit 15b5dd7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/edgehtml/hosting/toc.md
:white_check_mark:Succeeded

microsoft-edge/edgehtml/hosting/toc.yml
:white_check_mark:Succeeded

microsoft-edge/edgehtml/toc.yml
:white_check_mark:Succeeded
View

microsoft-edge/toc.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

#sign-off and merged.


Add color for the GDScript language (#355570)

Description
I've added coloring (#355570) to the GDScript programming language, since I got tired of seeing gray. #478cbf is the hex code for the color used in the Godot logo and icon, which would be the best to use, but conflicts. However this color (#355570) also works well, it's the background color for the icon:

I would make it brighter, but if I did, then it would conflict with other colors.
Checklist:
I removed the checklist as none of the items applied.

GDScript (#478cbf) is too close to ["6594b9", "3F85AF", "478cbf", "0096D8", "198CE7"]

These colors seem fairly distinct to me, but if GDScript is too similar to them, why aren't they listed as too similar to each other?

These colors seem fairly distinct to me, but if GDScript is too similar to them, why aren't they listed as too similar to each other?

I don’t fully understand the colour mathematics used to determine the proximity of colours but my vague understanding is the proximity calcs are detecting that the current colours are sufficiently different, but as soon as you add your colour, it ends up at a point within the others that has a knock on effect on the proximity calcs for the others once your colour is added to the mix.
In simple terms take: A xxxxxxxxxx B vs A xxxx C xxxx B and a “threshold” of 5.
A is 8 x’s away from B, but as soon as we put C in, there’s only a “distance” of 4 between C and another letter, thus it no longer meets our threshold requirement.
This is something the maintainer of the color-proximity gem may have more knowledge about. We use that gem for the proximity test.

In any case, none of those languages would ever be used in combination with GDScript. Godot only supports GDScript, C#, C++, visual scripting, and in the future perhaps F#.

You might not, but others may 😉

I assume there's a way to tell it to skip comparing GDScript to those languages, since it obviously doesn't mind that Clean is too similar to the others.

Nope as you can not predict how peeps will mix and match the content of their repos. Sadly, you’ve gone for a shade of blue and this is by far the most popular colour. You’re going to need to keep trying until you find another colour.
And no, we can’t decrease the threshold used in the test as this brings differences down too low for most peeps to distinguish.

@lildude I think there's something here that's causing the proximity checker to ignore the existing colors. I just pushed a test commit to make it equal to Clean's color, and it does not give any warnings about proximity to Clean's color despite the colors being equal - the test passes.
It would be preferable to use Godot's official color of #478cbf, but if the requirement is that the Travis CI tests need to pass, then this color is close enough and I can squash the commits so we can merge.

Hmmm, I thought we had a test for duplicate colours. Guess not (I’ll add one soon). We can’t really have duplicates for anything other than the default fallback grey. Everything else should be unique.
Don’t worry about squashing commits; we squash on merge anyway.

Looks like my patch is live, and it looks great. Thanks everyone!


[Question] How to use result of server method within client side collection hook?

I'm trying to set up some server and client side before.insert and before.update hooks and I need the document to be enhanced with the result of a server side method.
I cannot do it purely on the client due to lack of certain libraries on the client.
I cannot do it purely on the server because I need the result immediately without which the UI flickers and jumps around too much.
The problem is the server side method being called asynchronously on the client leaving me with undefined.
Is there a clean way to run a blocking function that I can use within a client side before.insert / before.update hook to get some result from the server?

There's no support for async hooks right now, it's being tracked in #71.


fix(DEVCOM.LuaJIT): remove version 2.1.19668

Package is no longer available.

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/DevelopersCommunity/cmake-luajit/releases/download/v2.1.1699392533/LuaJIT-2.1.19668-win64.msi
Response status code does not indicate success: 404 (Not Found).
(Automated message - build 748)


Refactor blas/ext/base/dsort2sh to follow project conventions

Resolves Issue #1497
Description

What is the purpose of this pull request?

This pull request:

refactors dsort2sh to follow project conventions.

Related Issues

Does this pull request have any related issues?

No
Questions

Any questions for reviewers of this pull request?

No
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines

@stdlib-js/reviewers

Hey @Planeshifter @kgryte.
Can you please review the changes that I have made ?

/stdlib merge


Swag fmt not throws error

Hello, Thank you for the project.
When I try to run swag fmt I have the following error:

Run<EMAIL_ADDRESS>  with:
    command: fmt ./...
    swagWersion: 1.8.7
Installing swag tool 1.8.7 ...
Unexpected error attempting to determine if executable file exists '/home/runner/swag/tar': Error: ENOTDIR: not a directory, stat '/home/runner/swag/tar'
Unexpected error attempting to determine if executable file exists '/home/runner/swag/tar': Error: ENOTDIR: not a directory, stat '/home/runner/swag/tar'
/usr/bin/tar xz --warning=no-unknown-keyword --overwrite -C /home/runner/work/_temp/346a8a4f-967e-4b65-9165-4a542d01cfea -f /home/runner/work/_temp/a554da27-47ee-43a3-bcb8-ef0f7c4f7c5e
/home/runner/swag fmt ./...

Could you please share some fix. Thank you.

Hi @Nibba2018 . Thanks for the feeback.
Seems that machine/image you are using for the action does not have tar tool.
You can add tar to your image or use another github action to install it.
I tested with default ubuntu-latest image that has tar by default and it works:
jobs:
  testIssue:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v3
      - uses: ./
        with:
          command: fmt -d example_test ./...
          swagWersion: 1.8.7

Code: https://github.com/yegorrybchenko/go-swag-action/blob/swag-fmt-fix/.github/workflows/test-issue.yml
Result: https://github.com/yegorrybchenko/go-swag-action/actions/runs/3543316092/jobs/5949656924
I will update docs for that


Implement Clone trait for serde_json::Error

Related requests: #972 and #335.

A brief investigation regarding the failing checks showed most alternative solution exclusively compatible to either pre-nightly builds or nightly builds. To pass the checks, the PR needs to implement a backward-compatible solution for cloning std::io::Error.


why so similar?

so why the ic_launcher.png have a similar shape to a paw with baidu? making me anxious is baidu that is potentially harm to computer equipment etc...

It's just a coincidence, you don't get anxious when you see a real dog's paw in real life, why here?

That is why you are welcome to compile from source.
pzdr
es
2 lut 2017 11:15 "Hybrid Song"<EMAIL_ADDRESS>napisał(a):

so why the ic_launcher.png have a similar shape to a paw with baidu?
making me anxious is baidu that is potentially harm to computer equipment
etc...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pylerSM/YouTubeBackgroundPlayback/issues/301, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGisqJ0SW7PMWs-Wn9r5LhavxXoLVvILks5rYazNgaJpZM4L07aQ
.

AFAIK that paw is taken from actual Google apps. Google introduced something called "Dogfood" which is basically test/unpublished/slightly modified app build. Dogfood's logo is that little paw - that is why it is used in module logo :P
Anyway - module is not even shown in launcher so why do you have problem with that? Just recompile module from source with changed logo as @esgie suggested :P


A few basic a11y fixes

Added document language and landmarks. To be honest I'm not happy with the added landmarks being nested in the container and would explore layout in more details and seek advice from a11y friends at TPG or Deque..
I left as is because I'm a Dev not a Designer but just had to make the basic a11y fixes :)
Also am in UK with no plans to move. Though am exploring opportunities to work in UK for Microsoft and just about to submit an application for AI for Accessibility for  AlwaysInMind.
So feel free to delete this PR if you'd rather.

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

Excellent, thank you!
Please still submit your application–adding this PR may help you as well!

Woah! Fast work! :)

Please still submit your application–adding this PR may help you as well!

Do you mean the AI for A11y one?

Yes! Good luck 😄


Remove RequiredPolicy

Fixes https://github.com/aspnet/AspNetCore/issues/9398

@pranavkm test failure which doesn't look related to this change, is this a known issue?  Should I just retry?
   at System.Net.Http.HttpContent.LoadIntoBufferAsyncCore(Task serializeToStreamTask, MemoryStream tempBuffer)
   at System.Net.Http.HttpClient.FinishSendAsyncBuffered(Task`1 sendTask, HttpRequestMessage request, CancellationTokenSource cts, Boolean disposeCts)
   at Microsoft.AspNetCore.Mvc.FunctionalTests.HttpClientExtensions.GetHtmlDocumentAsync(HttpClient client, String requestUri) in /_/src/Mvc/test/Mvc.FunctionalTests/Infrastructure/HttpClientExtensions.cs:line 18
   at Microsoft.AspNetCore.Mvc.FunctionalTests.FlushPointTest.FlushFollowedByLargeContent() in /_/src/Mvc/test/Mvc.FunctionalTests/FlushPointTest.cs:line 45
--- End of stack trace from previous location where exception was thrown ---
----- Inner Stack Trace -----
   at System.IO.Pipelines.PipeCompletion.ThrowLatchedException()
   at System.IO.Pipelines.Pipe.GetReadResult(ReadResult& result)
   at System.IO.Pipelines.Pipe.GetReadAsyncResult()
   at Microsoft.AspNetCore.TestHost.ResponseStream.ReadAsync(Byte[] buffer, Int32 offset, Int32 count, CancellationToken cancellationToken) in /_/src/Hosting/TestHost/src/ResponseStream.cs:line 114
   at System.IO.Stream.CopyToAsyncInternal(Stream destination, Int32 bufferSize, CancellationToken cancellationToken)
   at System.Net.Http.StreamToStreamCopy.<>c.<DisposeSourceWhenCompleteAsync>b__1_0(Task completed, Object innerSource)
   at System.Threading.Tasks.ContinuationTaskFromTask.InnerInvoke()
   at System.Threading.Tasks.Task.<>c.<.cctor>b__274_0(Object obj)
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.Tasks.Task.ExecuteWithThreadLocal(Task& currentTaskSlot, Thread threadPoolThread)
--- End of stack trace from previous location where exception was thrown ---
   at System.Net.Http.HttpContent.LoadIntoBufferAsyncCore(Task serializeToStreamTask, MemoryStream tempBuffer)
----- Inner Stack Trace -----

@aspnet-hello triage https://dev.azure.com/dnceng/public/_build/results?buildId=159145

Yeah, I kicked off a retry. It's a fairly new test, so perhaps there's a bug :(

This comment was made automatically. If there is a problem contact ryanbrandenburg.
I've triaged the above build. I've created/commented on the following issue(s)
https://github.com/aspnet/AspNetCore-Internal/issues/2300


Training error

Hi,
I'm getting the below error while I try to train the model. I checked in internet, the sources say that it can happen because of multiple cuda versions. I delete other cudas and tried, the issue still persists. Can you please help me get through this?


Test Changes

This run looks nice -- is it the output you expected, @LaurentDardenne?
https://github.com/potatoqualitee/psmodulecache/runs/8246200070?check_suite_focus=true

sweet, 8 passes. i havent yet dug into the tests -- are the old version tests still included?

The Previous tests included tests for old implementations as a reassurance that the expected behavior continues even in older versions. I've added them back and we'll see if it works 🤞🏼

Nice, it does 👍🏼  I'll likely do a new release within the next few hours.

thanks for the note about restarting the workflows. hopefully powershellget 3 fixes this 😞

>>The Previous tests included tests for old implementations as a reassurance that the expected behavior continues even in older versions.
I thought the action code was linked to the version of the repository (Tag) as shown here.
In this case the old versions continue to work.
>>hopefully powershellget 3 fixes this
I will try to find out the cause, but locally I never had this problem.
For PSGet3 I don't know how MS will manage the runner update, in my opinion it's a bit early to worry about it.

I thought the action code was linked to the version of the repository (Tag) as shown here

If I create a workflow that accidentally deletes all of my releases, then my previous versions will no longer work and I will be alerted to it. I do not see a reason to delete tests in general unless they are problematic.

I will try to find out the cause, but locally I never had this problem.

PS Get is flaky as are runners, unfortunately. Combine the two and some weird stuff can surface.


Add command to run extension tests as well as native tests.

I think this makes sense overall, with some minor suggestions for readability, etc. One idea that could be useful: rather than adding more extension points to the setup.py file, we could have a single entry point. Something like:
entry_points = {
    "eregs_plugin": ["atf = atf_parser.plugin"]
}

and then define atf_parser/plugin.py something like
# These are decorators which would be picked up by the plugin system
from regparser.plugins import xml_preprocessor, test_runner
from atf_parser import preprocessors, tests

xml_preprocessor(preprocessor.USCode)
xml_preprocessor(preprocessor.SomethingElse)

test_runner(tests)

Not at all for this PR, but an alternative to tinker with. I'd recommend this route if we wanted to cordon off stevedore knowledge (e.g. if it feels particularly unwieldy)


Check for updates cached

Describe the bug
I'm monitoring few programs for current releases, lens is one of them. I expect that when Lens is released it should display notification about new version and prompt for update. It may check sometimes for new version but I should be able to force update from menu or tray. Right now something is cached. With new version I have no prompt and manually checking for new version does not find anything. If I leave it it will display upgrade dialog sooner or later, I never checked how long does it take to refresh it's cache. If I exit lens on tray icon and relaunch it finds new version perfectly.
To Reproduce
Steps to reproduce the behavior:

Leave it running on background, use sleep for night
on next day when there is new release try to update it from menu or tray - it would not find new version
relaunch including tray - now it will find new version OR leave it and measure when information about new version will be displayed

Expected behavior
I expect that when new version comes it should find it ASAP, when it was released at night then it should display dialog soon after app is active. For sure when I know about new version it should not cache when I manually check for new version using menu or tray.
Environment (please complete the following information):

Lens Version: 5.4.3 after morning upgrade
OS: Win10
Installation method (e.g. snap or AppImage in Linux): installer

Additional context
If You don't use sleep or maybe hibernation then You won't notice that - checking for update works and lens starts with upgrade dialog. I intentionally left it once and it took it about one day to refresh this cache. I think that it cached to much and it should not cache manual update checks.

Related to #4948
The reason why it doesn't show something immediately might be because it is downloading the update so you don't have to wait after clicking "yes"

Lens now checks for updates every hour so this should be resolved then.


superfluid charge icon to pool card

What is the purpose of the change

ClickUp Task
ClickUp Task URL
Brief Changelog

Testing and Verifying

Documentation and Release Note

stale, can reopen


Repetitive tiny routes

Hello again!
I just realized that there are a lot of the same tiny routes (generated by the code) and the data generated in these routes seem nearly the same after visualization, which might waste the disk space? Is there any reason not to filter out those routes, e.g., they are useful because of something?
for example, below are two same tiny routes on Town01

Thanks in advance!
Xinshuo

To generate these routes, I first sampled start and end waypoints at 90 degrees at each intersection (link) and then extracted each tiny route by explicitly finding the first waypoints where the navigational command changes (link). In this way, I got 8 routes at each intersection (4 pairs of waypoints and start & end points can be interchanged).
While going through the code, I noticed some bugs. The 8 routes should correspond to: (x-d,y)->(x,y-d), (x-d,y)->(x,y+d), (x,y-d)->(x-d,y), (x,y+d)->(x-d,y), (x+d,y)->(x,y-d), (x+d,y)->(x,y+d), (x,y-d)->(x+d,y), (x,y+d)->(x+d,y), but I mixed up some of these due to which there are duplicates in the tiny routes.
https://github.com/autonomousvision/transfuser/blob/720c9b566f8b6fdeec991881e291833b1566af9f/tools/generate_intersection_routes.py#L21-L51
The main reason for including these tiny routes was to improve the distribution of turnings and intersections in the dataset (check Fig. 1(a) of suppmat) since we noticed a significant performance difference between training on the original leaderboard routes and our routes. Removing some of these routes would change the distribution but I don't know if the resulting difference will have a significant impact on the performance.

Yep, definitely understand the point of having tiny routes in general. Was only curious about why not filtering out the "repetitive" routes, which however has been clarified by the bug you mentioned.
It looks like the current code is not updated to resolve this bug yet, right? Would be great if you can update it. I will try on my side as well!
Thanks again!

We are currently working on the codebase to remove more bugs and add more functionalities, we'll update it soon.

Sounds great! Looking forward to it!


Respect autoNotify flag on Android

Goal
Fixes Android's handling of the autoNotify flag so that it is respected when set as false for JVM errors.
Changeset
AbstractConfiguration  set AutoNotify to true, which enabled the JVM exception handler. This meant that the call to setEnableExceptionHandler in the Android implementation of Configuration was overridden as soon as the base method of SetupDefaults() was called.
This fix passes the autoNotify value supplied through Bugsnag.Init to the base method of SetupDefaults() so that the handler is not enabled.
Testing
Manually verified in an example project with autoNotify = false that an error is no longer sent.

CI temporarily failing due to issues unrelated to this changeset - Travis CI appears to have changed behind the scenes again

CI temporarily failing due to issues unrelated to this changeset - Travis CI appears to have changed behind the scenes again

Refactored changeset so that autoNotify is passed down toSetupDefaults, and confirmed that AutoNotify is respected for the original reproduction case.
It may be worth consider some general regression testing in this area, as the changeset now touches all the platforms after the refactor.

Refactored changeset so that autoNotify is passed down toSetupDefaults, and confirmed that AutoNotify is respected for the original reproduction case.
It may be worth consider some general regression testing in this area, as the changeset now touches all the platforms after the refactor.


add support for Tuba.fm

Tuba.fm

I have sent an email to<EMAIL_ADDRESS>requesting they broadcast track information so SLS can scrobble it.


Generator: update demo to use focus-visible

Is your feature request related to a problem? Please describe.
Update the demo from the generator to use focus-visible feature.
Dependency: https://github.com/AlaskaAirlines/WC-Generator/issues/69

dupe


Feature 260 client default allowed file extensions

define whitelist in text file
automatically include whitelist into documentation
script for easily copy+pasting into constants.go
#258 fixed
#267 fixed
main branching changed from if-else-if to switch statement (easier to read)
some tweaking of debugging output
go testing: test for new feature #256
go testing: adapted existing tests because of new default behavior (only source files are added to the zip)
rename additionalSourceExts -> additionalFilenameExtensions
adapt documentation
adapt example config
fix java test (integrationtest)

closes #256
closes #258
closes #266
closes #267

Sven Dolderer<EMAIL_ADDRESS>Daimler TSS GmbH, imprint

Review has been done, PR will be merged

Albert Tregnaghi<EMAIL_ADDRESS>Daimler TSS GmbH, imprint


Hierarchy [WIP]

Closes #28

[x] Removed duplicate function: insert_dynamic.
[x] Removed duplicate code: get_ptr
[x] Added ChangeList.
[x] Integrated hierarchy storage.
[x] Add automatic HierarchicalStorage initialization.
[x] Integrate get API so to support LOCAL and GLOBAL fetch.
[x] Integrate the automatically "sub flush" at the end of each system that takes an HierarchicalStorage mutably.
[x] Integrate the automatically "sub flush" at the end of each system that takes the Hierarchy.
[ ] Make sure it's impossible take Child using a Query.
[x] Integrate new Global filter.
[x] Make sure that the Query uses the correct function get() or get() const depending on the Query type.
[x] Finish write the testing for in System Hierarchy mutation feature.
[x] Expose Space::Global to GDScript.

https://user-images.githubusercontent.com/8342599/107147366-f17f9280-694d-11eb-95c8-588dd1d4ff4c.mp4


[VOTE] Release Apache TVM v0.11.0.rc0

Dear TVM community,
This is a call for vote to release Apache TVM version v0.11.0.  https://github.com/apache/tvm/issues/13899 for any edits you would like to see included in the release notes.
Link to release notes:
https://github.com/apache/tvm/issues/13899
Link to release candidate:
https://dist.apache.org/repos/dist/dev/tvm/tvm-v0.11.0-rc0/
Link to github pre-release page:
https://github.com/apache/tvm/releases/tag/v0.11.0.rc0
Everyone is welcomed to vote. Please vote by replying to this thread explicitly.
+1 = approve
+0 = no opinion
-1 = disapprove (provide reason)
NOTE: this thread is being mirrored in dev@
Vote will close Mar. 1st at 23:59PM GMT.

I checked
+1 , I checked
Signatures and hashes good
LICENSE, NOTICE
Signatures and hashes
No unexpected binary files
Code compiles and import in python

+1

+1

+1

+1

+1

+1

Thanks for all who voted, I'll summarise the results and publish a [RESULT] ticket.


Use different font list and exposures for "lat" language training in language-specific.sh

See also tesseract-ocr/langdata#23.

All tests failed because @egorpugin made changes in his leptonica repo.
https://github.com/egorpugin/leptonica
is empty right now.
The link in the test should be updated to:
https://github.com/egorpugin/leptonica_old

Keep kalm. We're setting an official leptonica repository this time.

Ok, you can try to recreate this PR. It should pass CI checks. We'll modify scripts lates, but egorpugin/leptonica is available again.


steam: gamescope fails to launch when used from within steam

Describe the bug
gamescope crashes with this error when run from within steam's fhsenv:
wlserver: Running compositor on wayland display 'gamescope-0'
wlserver: [backend/headless/backend.c:18] Starting headless backend
wlserver: [xwayland/sockets.c:99] /tmp/.X11-unix not owned by root or us
wlserver: [xwayland/sockets.c:99] /tmp/.X11-unix not owned by root or us
[...]
wlserver: [xwayland/sockets.c:99] /tmp/.X11-unix not owned by root or us
wlserver: [xwayland/sockets.c:216] No display available in the first 33

This check in wlroots causes that error
https://gitlab.freedesktop.org/wlroots/wlroots/-/blob/0.15.1/xwayland/sockets.c#L98
because inside steam's fhsenv the  /tmp/.X11-unix directory is owned by nobody:nogroup.
I have tried simply removing the check from wlroots but in that case gamescope simply hangs.
(Note that it's gamescope that depends on wlroots so this bug happens under x11 window managers too.)
Steps To Reproduce
Steps to reproduce the behavior:

start steam
add gamescope -- %command% to the launch options of any game
run the game

Notify maintainers
@jagajaga @jonringer
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.7, NixOS, 23.05 (Stoat), 23.05pre446447.1b1f50645af`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.12.0`
 - channels(root): `"nixos"`
 - channels(hadis): `"home-manager, nixos"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

This seems to be a very old issue with NixOS https://github.com/NixOS/nixpkgs/issues/42117

https://github.com/NixOS/nixpkgs/pull/215837 would allow gamescope to run with steam-run gamescope without having to manually chown /tmp/.X11-unix but running gamescope from within steam as described above is still broken.
gamescope dies with this error:
Xwayland: symbol lookup error: /nix/store/ip8l0dbilm0vayy5wqgghn05vsqp8130-libkrb5-1.20.1/lib/libcom_err.so.3: undefined symbol: k5_os_mutex_destroy, version krb5support_0_MIT
wlserver: [xwayland/server.c:247] Xwayland startup failed, not setting up xwm

However :
[nix-shell:/nix/store/ip8l0dbilm0vayy5wqgghn05vsqp8130-libkrb5-1.20.1/lib]$ readelf -Ws libcom_err.so.3 | awk '{print $8}' | grep k5_os_mutex_destroy
k5_os_mutex_destroy@krb5support_0_MIT
k5_os_mutex_destroy@krb5support_0_MIT

I've encountered this issue too!
I know @jonringer got gamescope to run on NixOS - maybe we're missing something?

start steam
add gamescope -- %command% to the launch options of any game
run the game

Hi, I just wanted to real quickly add some context with my knowledge of how Steam works.
this part is not the issue and is an ""issue"" about the steam overlay, it happens because there is both a 32 bit and 64 bit version of the overlay both of which are loaded when you launch a 32bit or 64bit game, so this is not related. It happens every time you launch a game on any distro without fail since a game cannot both be 32 and 64 bit.
 ERROR: ld.so: object '/home/me/.local/share/Steam/ubuntu12_32/gameoverlayrenderer.so' from LD_PRELOAD cannot be preloaded (wrong ELF class: ELFCLASS32): ignored.
Everything else in this log seems normal, outside of this part:
Xwayland: symbol lookup error: /nix/store/lkbxlllgchqavfl6ldz60vx0p7gkzc08-libkrb5-1.20.1/lib/libcom_err.so.3: undefined symbol: k5_os_mutex_destroy, version krb5support_0_MIT which could point to a missing library. the mentioned "libcom_err.so" might be what we need to look at here.
Also, thank you so much to  @GrabbenD for posting the workaround too, I was getting a headache dealing with this.

Hi, i wanted to ask if someone is working on this issue?

@urioTV It's an open-source project, so unless someone with the necessary knowledge is interested in the issue (could be yourself), then you'll probably have to wait.

I ran into this same issue before. The only thing that let me run gamescope through steam so far has been the solution from this comment, but I am not sure why it works.

I'm trying to add the patch in https://github.com/NixOS/nixpkgs/issues/162562#issuecomment-1229444338 in #312268 , eagering for testing from you.


Apt server is broken/down, gives 403 Forbidden

Required Info

Camera Model
D435

Firmware Version
Intel® RealSense™ SDK 2.0)

Operating System & Version
Ubuntu 22.04.1 LTS

Kernel Version (Linux Only)
Linux 5.15.0-46-lowlatency x86_64

Platform

SDK Version
2.0

Language
n/a

Segment
n/a

Issue Description
Following the instructions at:
https://github.com/IntelRealSense/librealsense/blob/master/doc/distribution_linux.md#installing-the-packages
I receive:
Err:6 https://librealsense.intel.com/Debian/apt-repo jammy InRelease
403  Forbidden [IP: 2600:9000:218c:c000:1a:eb13:66c0:93a1 443]
I notice there are similar issues that are now closed #9349 - but the solutions there do not work for me.
Thanks & regards

Hi @devany  On a 22.04 Jammy installation discussion at https://github.com/IntelRealSense/librealsense/issues/10439#issuecomment-1171965106 there is an unofficial test patch to add support for kernel 5.13 and 5.15, and also a pre-built DKMS test package.

But I can't download the SDK, so I can't apply a patch.
Also, or in any case, the instructions I originally followed have now broken apt such that I always receive a forbidden IP error when attempting apt update... item 2 on the instructions.

Have you expanded open the 'Assets' file list on that instruction page that contains the file quoted in item 2, please?

Yes, I have that file, but i can't successfully apt update without throwing "Forbidden IP".
i.e.
sudo apt-get update Hit:1 http://archive.ubuntu.com/ubuntu jammy InRelease Hit:2 http://archive.ubuntu.com/ubuntu jammy-updates InRelease Get:3 http://archive.ubuntu.com/ubuntu jammy-backports InRelease [99,8 kB] Hit:4 https://dl.google.com/linux/chrome/deb stable InRelease Get:5 http://security.ubuntu.com/ubuntu jammy-security InRelease [110 kB]                              Err:6 https://librealsense.intel.com/Debian/apt-repo jammy InRelease   403  Forbidden [IP: 2600:9000:218c:5400:1a:eb13:66c0:93a1 443] Reading package lists... Done                              E: Failed to fetch https://librealsense.intel.com/Debian/apt-repo/dists/jammy/InRelease  403  Forbidden [IP: 2600:9000:218c:5400:1a:eb13:66c0:93a1 443] E: The repository 'https://librealsense.intel.com/Debian/apt-repo jammy InRelease' is not signed. N: Updating from such a repository can't be done securely, and is therefore disabled by default. N: See apt-secure(8) manpage for repository creation and user configuration details.

I will discuss your problem with my Intel RealSense colleagues.  Thanks very much for your patience!

Hi @devany  My colleagues provided the following information about your issue.

Ubuntu 22.04.1 LTS is not yet formally supported in LRS, so the apt repository not available yet.  The customer can install the pre-release Debian packages with dpkg commands
sudo dpkg -i <deb package to be installed>
sudo dpkg -r <name of the deb package to be removed>

There are two ways to resolve the Forbidden IP error:

Method #1 - remove the realsense apt server from apt with command below:

sudo add-apt-repository --remove "deb https://librealsense.intel.com/Debian/apt-repo $(lsb_release -cs) main" -u
Then sudo apt update should work.

Method #2 - remove the realsense apt repository server from apt manually:

Modify the /etc/apt/sources.list file directly, delete the lines with the realsense apt repository, save the file back.
Check the repo list with this command:
cat /etc/apt/sources.list

Hi @devany  Do you require further assistance with this case, please?  Thanks!

Thanks for all your help, the issue can be closed now.
Cheers!

You are very welcome, @devany - thanks very much for the update!


Refactor ReaddirHandle to lazily create inodes

The motivation here is that readdir calls from FUSE generally only
retrieve a small number of entries, but ListObjectsV2 calls to S3
retrieve many entries. In the current code, we instantiate inodes for
those entries as soon as the ListObjectsV2 call completes, but because
of the small batch size of readdir, most of these inodes will not be
returned to FUSE until some time later. Because the TTL of an inode's
attributes is chosen at inode creation time, it will have expired by
this point, and so the kernel will immediately re-lookup these entries
when they're returned to readdir. That makes readdir do O(n)
lookup calls back to S3, defeating the purpose of ListObjectsV2.
To solve this problem, this change refactors the ReaddirHandle
implementation to defer creating inodes until an entry is actually
returned to readdir. The big idea is that we now just create a new
[ReaddirEntry] struct for each entry returned by ListObjectsV2, and all
the readdir logic works in terms of those things rather than directly
on inodes. The conversion from [ReaddirEntry] to an actual inode then
happens right before returning the entry to readdir.
I took this opportunity to refactor this code (hopefully for the
better!) since this change was adding yet another layer of complexity
into the logic for iterating directory entries. I also tried to capture
my understanding of this code in a new module comment. There is still a
bug here around page sizes (#190) that I think this comment and refactor
will make it easier to fix, but let's do that separately to avoid making
this change bigger.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and I agree to the terms of the Developer Certificate of Origin (DCO).

The TTL duration hasn't changed. It's just that the inode itself is constructed later, and since the TTL is computed at inode construction time (when we create the InodeStat), it now happens right before returning the inode rather than as soon as it was received from ListObjectsV2. So it's inode_construction_time + ttl_duration rather than list_objects_time + ttl_duration.

My high level understanding here, correct me if I am wrong:
Current status

FUSE call on readdir retrieve x entries whereas FUSE Damon implementation(Mountpoint-s3)ListObjectsV2 retrieves, let's say 10x entries.
Inode creation happens after ListObjectsV2 completed, where readdir can only deal with x entries as a batch at a time, so FUSE experiences some delay when it received rest 9x entries because of TTL expired. (The assumption here is we are not going to change TTL for some reason?)
If TTL expired, linux kernel conducts lookup on the rest of 9x entries.

Problem

The nuance of capability of handling the number of directory entries between readdir and ListOjbectsV2.
More specifically, the size ofreaddir entry buffer cannot hold the number of directory at a time before TTL expired.

Solution:

Another level of indirection
Introduced ReaddirEntry so readdir don't have to deal with inode immediately, instead, convert ReaddirEntry to inode on demand basis, so it prevent re-lookup during readdir

Nope, you're right! This is presupposing a future change that actually uses the expiry field correctly.


PSBT base64 (de)serialization with Display & FromStr

Was quite surprised that the lib can't serialize and read Base64 PSBTs as required by BIP-174 (while having base64 as a dependency). Since this is a way of putting PSBT into a string as defined by the standard I used it as a way for implementing FromStr/Display.
Seems like an API break since we need to add one psbt::Error variant (feature-gated, though).
Provided one other commit https://github.com/rust-bitcoin/rust-bitcoin/pull/557/commits/a6399e7350a1b2ec6f91fc64001239402f4164ae that makes the PR non-API-breaking, suitable for 0.26.1 (really like to have base64 encoding ASAP). After the minor release that commit can be reverted.

@sgeisler thank you for the review! The reason for minor version is the following: 0.27 will take months to be published and will do a large-scale refactoring of the repo; while this thing is needed to be published within days/week, if possible.

@sgeisler thank you for the review! The reason for minor version is the following: 0.27 will take months to be published and will do a large-scale refactoring of the repo; while this thing is needed to be published within days/week, if possible.

As long as you take ownership of reverting the commit I'm ok with it, I'd just be unhappy with it staying that way and we don't exactly have a track record of being super organized as an org :smile:

As long as you take ownership of reverting the commit I'm ok with it, I'd just be unhappy with it staying that way and we don't exactly have a track record of being super organized as an org :smile:

Improved display with your suggestion - and yes, I will create revert PR right after this one will be merged

Improved display with your suggestion - and yes, I will create revert PR right after this one will be merged

@sanket1729 done with https://github.com/rust-bitcoin/rust-bitcoin/issues/564. I propose to merge this for now and discuss the best way of handling errors in that issue

@sanket1729 done with https://github.com/rust-bitcoin/rust-bitcoin/issues/564. I propose to merge this for now and discuss the best way of handling errors in that issue

@dr-orlovsky, I prefer not to merge unless I get a sign of approval from @apoelstra's powerful machine which runs more powerful checks than the CI, checks that every commit builds on the respective rustc versions that we care about.
Maybe this is turning up too much work on @apoelstra and we need to find a better way. I am awaiting @apoelstra's comment before I merge this, anyways we need him for publishing the minor release.

@dr-orlovsky, I prefer not to merge unless I get a sign of approval from @apoelstra's powerful machine which runs more powerful checks than the CI, checks that every commit builds on the respective rustc versions that we care about.
Maybe this is turning up too much work on @apoelstra and we need to find a better way. I am awaiting @apoelstra's comment before I merge this, anyways we need him for publishing the minor release.

@sanket1729 Are there any practical limits for on GH actions? I think we could test a lot more if we wanted to, possibly gated by an initial smaller stage to not burn CPU needlessly.

@sanket1729 Are there any practical limits for on GH actions? I think we could test a lot more if we wanted to, possibly gated by an initial smaller stage to not burn CPU needlessly.

@sgeisler, I think it also checks every single commit builds/tests on different rust versions which is I only do for the stable rust versions.
But we should see if we get per commit build/test checks in the CI script. Strangely enough, I can't easily find any project with CI that checks on every commit.

@sgeisler, I think it also checks every single commit builds/tests on different rust versions which is I only do for the stable rust versions.
But we should see if we get per commit build/test checks in the CI script. Strangely enough, I can't easily find any project with CI that checks on every commit.

I tested b4077213ab311bed2a4faaaae3600d9949b867a6 quite extensively now (full feature matrix stable+nightly and multiple fuzzing rounds), I think we can just merge except if we want @apoelstra's ok regarding the release-and-revert strategy.

After a lot of confusion with the latest revert commit preventing API break, I have restructured the PR in this way:

The current PR is not API breaking and reuses existing PSBT Error type, targeting 0.26.1 release
It is followed by https://github.com/rust-bitcoin/rust-bitcoin/pull/574 which adds a custom error type specific for PSBT FromStr parsing, which is API breaking

Improved after discussion in https://github.com/rust-bitcoin/rust-bitcoin/pull/574#discussion_r579687827

Needs a fix for 1.29.0, but I like the error solution.

As long as you take ownership of reverting the commit I'm ok with it, I'd just be unhappy with it staying that way and we don't exactly have a track record of being super organized as an org smile

@sgeisler we should use the issues feature more and have a release board or so at least. But well I'm not really the one to speak, I have been really bad keeping up with review lately.


Track Location of Chatting User Talk-IN

User can Track the Location of the User with whom they are chatting .

@Pranishkafle6175
Conflicting files
TalkIn/app/src/main/java/com/example/talk_in/ChatActivity.kt
TalkIn/app/src/main/res/layout/activity_chat.xml
And can you please share some screenshots?

@Pranishkafle6175 recently we have introduced locate button in each registered user so please delete that icon shift the same action to the new one

Done Updated the Map icon to menu

@Pranishkafle6175 please share screenshot

What I have done is when user selects the LocateNow a function will call where the location of the sender will be fetched  if he/she has provided location permission and then the user location will be updated in database in node named location and then the receiver location will be fetched and an intent will be passed  with latitude and longitude of both sender and receiver and located in maps.

@Pranishkafle6175 Due to some uncertain issue, currently I'm not merging it but soon it will be merged as the issue is solved.
Thank You!

Can you tell me the isssue! Is the issue from my side!
On Sun, 19 May 2024, 19:38 Shivam Jaiswal, @.***> wrote:

@Pranishkafle6175 https://github.com/Pranishkafle6175 Due to some
uncertain issue, currently I'm not merging it but soon it will be merged as
the issue is solved.
Thank You!
—
Reply to this email directly, view it on GitHub
https://github.com/Ijaiswalshivam/Talk-In/pull/155#issuecomment-2119246620,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2IMHPUUHAN7YBSVYUBDUQLZDCVENAVCNFSM6AAAAABH6BLFDWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJZGI2DMNRSGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

no no, not from your side

but please make sure that you worked on the latest committed files.

Ohh okay please let me know if there is anything i can do!!
On Sun, 19 May 2024, 19:46 Shivam Jaiswal, @.***> wrote:

no no, not from your side
—
Reply to this email directly, view it on GitHub
https://github.com/Ijaiswalshivam/Talk-In/pull/155#issuecomment-2119248724,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2IMHPU5QYDSH22GAPOXYATZDCWCJAVCNFSM6AAAAABH6BLFDWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJZGI2DQNZSGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Yeah i worked on the latest committed code but since everyone are
committing it may not be the latest when you merge so i request you to
merge fast if possible!
On Sun, 19 May 2024, 19:47 Shivam Jaiswal, @.***> wrote:

but please make sure that you worked on the latest committed files.
—
Reply to this email directly, view it on GitHub
https://github.com/Ijaiswalshivam/Talk-In/pull/155#issuecomment-2119249153,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2IMHPWLCQ7G57V7LFNJVR3ZDCWHLAVCNFSM6AAAAABH6BLFDWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJZGI2DSMJVGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

Error In chat activity: Unresolved reference: locateNow,
and try to implement by forking it again so that i can merge this as soon as possible.


Doesn't include some templates with grunt build.

Hi, I got an issue with ng-template.
it's doesn't include some templates.
my routes code here:
$stateProvider.state('home', {
  abstract: true,
    url: '/home',
    views: {
    'header': {
      controller: 'HeaderCtrl',
      templateUrl: 'layout/header.html'
    },
    '': {
        templateUrl: 'home/main.html',
    },
    'footer': {
        controller: 'FooterCtrl',
        templateUrl: 'layout/footer.html'
    }
   }
})
.state('home.detail', {
  url: '',
    views: {
        'home' : {
            templateUrl: 'home/home.html',
            controller: 'HomeCtrl'
        },
        'clients' : {
            templateUrl: 'clients/clients.html',
            controller: 'ClientsCtrl'
        }
 ...
}

so the home page would be include multiple views like home, clients, etc more.
my grunt script here:
ngtemplates: {
      dist: {
        options: {
          htmlmin: '<%= htmlmin.dist.options %>',
          usemin: 'assets/js/main.js'
        },
        cwd: '<%= yeoman.app %>',
        src: ['{,*/}*.html','!<%= yeoman.app %>/index.html'],
        dest: '.tmp/templateCache.js'
      }
    },

so after grunt build, in the templateCache.js file doesn't includes for home.html and clients.html..
Hope to help for those.

@tiemin would you mind running the command with --verbose? I'm wondering if there might be a problem with the globbing pattern on your src for the ngtemplates config.

+1