#28297 Resolve Same queryset result in two different queries on ORM

Ticket

I did this in the eq, but a lot of tests failed with differing
values. I could shift it up into the loop above and see what happens.
On 12 Jun 2017 17:54, "Simon Charette"<EMAIL_ADDRESS>wrote:
@charettes commented on this pull request.
In django/db/models/sql/query.py
https://github.com/django/django/pull/8631#discussion_r121466856:

@@ -918,6 +918,13 @@ def join(self, join, reuse=None):
"""
reuse = [a for a, j in self.alias_map.items()
if (reuse is None or a in reuse) and j == join]

   if len(reuse) > 1:

       # In some cases more than one suitable alias can be

found. Join.eq

       # only compares on table_name, join_field and

parent_alias. Here we find

       # the alias with a matching join_type.

       reuse = [r for r in reuse if self.alias_map[r].join_type

== join.join_type]
Shouldn't we adjust Join.eq to consider join_type instead? Or maybe
move this condition to the reuse generation above?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/django/django/pull/8631#pullrequestreview-43497733,
or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA-shxh9qS3JL9_nuh4-dMV-t77v1kqJks5sDW03gaJpZM4N3YYY
.

I've fixed up the MR as asked @timgraham. Thank you for the review!

Should we close this since you deassigned yourself from the ticket?

Yes, sorry, I forgot to do this.


Switch the place of "Next Change" and "Open Changes with Next Revision" button

I have mis clicked the button of "Next Change" when I check the changes, it changes to the button "Open Changes with Next Revision", I think we can change the place of those buttons to avoid this happens

@tongxian Not sure I understand. Can you show a screenshot/gif of how these change location? FYI, the Open Changes with Next Revision button are from GitLens, not VS Code.


I continually get this error, which I just ignore

Too old for us: Celebrating (or not) the Centenary of the Russian Revolution [Audio]
Already seen: Celebrity [Audio]
Downloading The French Election and the Left [Audio] from http://media.rawvoice.com/lse_publiclecturesandevents/richmedia.lse.ac.uk/publiclecturesandevents/20170509_1830_theFrenchElectionAndTheLeft.mp3
100%|████████████████████████████████████████████████| 42.5M/42.5M [00:25<00:00, 1.66MB/s]
Moving {/tmp/upodder/cadce213ff5c92391888d1e784a4180c93cad59d} to {/home/Jadamso/Desktop/MiscPersonal/Audio/Podx/The_French_Election_and_the_Left_Audio_.mp3}
Traceback (most recent call last):
File "/usr/bin/upodder", line 9, in
load_entry_point('upodder==0.6.9', 'console_scripts', 'upodder')()
File "/usr/lib/python2.7/site-packages/upodder/upodder.py", line 212, in main
process_feed(url)
File "/usr/lib/python2.7/site-packages/upodder/upodder.py", line 192, in process_feed
EntryProcessor(entry, feed)
File "/usr/lib/python2.7/site-packages/upodder/upodder.py", line 98, in init
SeenEntry( pub_date=self.pub_date, hashed=self.hashed)
File "/usr/lib64/python2.7/site-packages/sqlobject/main.py", line 1301, in init
self._create(id, **kw)
File "/usr/lib64/python2.7/site-packages/sqlobject/main.py", line 1352, in _create
self._SO_finishCreate(id)
File "/usr/lib64/python2.7/site-packages/sqlobject/main.py", line 1381, in _SO_finishCreate
self._init(id)
File "/usr/lib64/python2.7/site-packages/sqlobject/main.py", line 986, in _init
self._SO_selectInit(selectResults)
File "/usr/lib64/python2.7/site-packages/sqlobject/main.py", line 1227, in _SO_selectInit
colValue = _col.to_python(colValue, self._SO_validatorState)
File "/usr/lib64/python2.7/site-packages/sqlobject/col.py", line 1251, in to_python
return datetime.timedelta(seconds=value.seconds)
AttributeError: 'str' object has no attribute 'seconds'
podx.txt

Hi @Jadamso , thanks for writing in. The bug you encounter seems to be located deep inside sqlobject. Did you try updating that package? Which version are you using, when running pip freeze?

None. It was not installed. I apologize for my laziness in debugging. My
script works now.
Jordan Adamson, Ph.D. Economics
https://sites.google.com/a/g.clemson.edu/ja-resources/
``The good of other times let people state; I think it lucky I was born so
late" - Ovid
On Mon, May 15, 2017 at 9:28 PM, Manu<EMAIL_ADDRESS>wrote:

Hi @Jadamso https://github.com/jadamso , thanks for writing in. The bug
you encounter seems to be located deep inside sqlobject
http://www.sqlobject.org/. Did you try updating that package? Which
version are you using, when running pip freeze?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/m3nu/upodder/issues/21#issuecomment-301649356, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHQq7bP4aJPNAkrWIQxrPNdiU6YJYDk_ks5r6PutgaJpZM4Nbb0n
.

Thanks for checking. Closing this issue.


Missing 4.5.3 release notes

Release notes for 4.5.3 are missing.

Good catch, they should have been pushed up during release. Will investigate once at a computer.

This is because the master branch is currently being used for v5 and the branch v4 contains the latest v4 deployments.
You can find the changelog at https://github.com/zemirco/json2csv/blob/v4/CHANGELOG.md


Fix(loader Icon): Update loading spinner (FightPandemics#1421)

What does this pull request aim to achieve? 💯
Resolves #1421

[x] I have checked that no one else is working on similar changes.
[x] I have read the latest README and understand the development workflow.
[x] The name of this branch is: feature/<branch_name>, or hotfix/<branch_name> if this is a hotfix. The branch is created in a cloned version of this repo, not a forked repo.
[x] This branch is rebased/merged with the latest staging (or the latest production if it is a hotfix).
[x] There are no merge conflicts.
[x] There are no console warnings and errors.
[x] On the right hand side of this PR, I have linked the appropriate issue(s) under "Issues" and added a project progress under "Projects".

Please reopen this PR from a cloned repo and not forked one. I will DM in Slack.


per-keyboard profiles

It's not clear if it's possible to support different profiles for different keyboards.  For instance, on my Laptop I'd only like to make caps lock act as esc, but on my windows keyboard that I usually hook up I want to also swap command and option keys.  Is this possible?

I think there are a bunch of issues already open for this:

https://github.com/tekezo/Karabiner-Elements/issues/138
https://github.com/tekezo/Karabiner-Elements/issues/67
https://github.com/tekezo/Karabiner-Elements/issues/31

@Ghazgkull you're right, closing this one


Re-implement the following using intermediate parsers

[ ] Single identifiers
[x] Strings
[x] Integers
[x] Processor statements
[ ] Method invocation
[ ] Control statements
[ ] Method declaration
[ ] Unary operations (generic)
[ ] Binary operations (generic) (parsed as an OperationChain before being broken up into expressions for precedence purposes)
[ ] Parenthesis expressions
[ ] Indexers
[ ] Tables
[ ] Expression chains (like a.b().c[0].d)

Additionally:

[x] Comments (#)
[ ] Compound operators (+=, -=)
[ ] Keywords

How should intermediates be separated from the interpreter itself? What is parsed by the former and the latter?

Moved to private issue tracker


create wiki with examples

The most common usage:

CLI
Node module
Gulp
Broccoli

This depends on 1.1.0.

Done.


Knockback X-Axis

The character knockback does not work as intended when the hit character is on the ground. This is likely because of how velocity is handled inside the actors class.
Take a look at the character actors class. In _determine_movement() the velocity is being directly modified every frame.
So, when you take damage:
[FRAME 0]

velocity = player movement <- as decided by the player
^ Before the body has been hit
velocity = knockback <- player hit, velocity is overwritten
move_and_slide()

[FRAME 1]

velocity = player movement
The player is not hit again
move_and_slide()

The frame after the hit, the velocity of the character is reset to match the input. This doesn't affect the y axis because the _determine_movement() will only set _velocity.x to 0 when it's not pressed. Additionally, the knockback works on the x-axis in mid-air because of the is_on_floor() condition inside _determine_movement.
In addition to this, there could be a compounded issue with friction.
One possible solution is to separate the knockback velocity from the movement velocity and only combine them when they're set in move_and_slide().
This also brings up the question of if the player should be able to move while in knockback.

Perhaps we can add a "knockback" move state that resets to "idle" after a short time.


request: make it compatible with AngularJS 1.7

Using bower, it shows the following alert:
├─┬ angular-modal-service#0.15.1
│ └── angular#1.7.8 incompatible with 1.5.0 - 1.6.x (1.6.11-build.5555+sha.45879a8 available, latest is 1.7.8)

It works great with AngularJS 1.7.8. Please, remove the restriction about from 1.5.0 to 1.6.x versions and include the 1.7.x.

Resolved in #264, released as:
<EMAIL_ADDRESS>
Thanks for the heads up!


Make current stateful widget all to stateless

Make current stateful widget all to stateless since the states are not supposed to store there.
Gesture handler should pass out of object instead defined in-place.

User answers need to be shared over various widgets, therefore they are app states.

since provider package is included, gesture handler could be replaced by accessing directly from context at current place.


Definitions using polymorphism seem incorrect

I'm not sure whether this belongs here, or in the swagger-parser project, or in the swagger2markup project https://swagger2markup.readme.io/, but I'll start here.  We have a number of models that use polymorphism, some of them extend multiple levels.  We're using Java annotations and the JAX-RS integration to create our Swagger definition at runtime.  I'm running into issues running our spec through the swagger2markup tool, and I'm trying to figure out whether the issue is something we have defined incorrectly, or a bug in one of the 3 projects.  Here's our Java classes with the swagger annotations.
@ApiModel(description = "The representation of a Chart-like object.", discriminator = "_class",
        subTypes = { TextChartDTO.class, ChartDTO.class, TableChartDTO.class })
public abstract class BaseChartDTO implements Serializable {
    private String id;
    private String name;
    public String getId() { return id;}
    public void setId(final String id) {this.id = id;}
    public String getName() { return name;}
    public void setName(final String name) {this.name = name};
    // Various other properties
}

@ApiModel(description = "A parent class for all charts.", parent = BaseChartDTO.class, discriminator = "_class",
        subTypes = { PieChartDTO.class, BarChartDTO.class, AreaChartDTO.class, LineChartDTO.class,
            MixedChartDTO.class, GaugeChartDTO.class })
public abstract class ChartDTO extends BaseChartDTO implements Serializable {
    private ChartType chartType;
    public ChartType getChartType() {return chartType;}
    public void setChartType(final ChartType type) {this.chartType = type;}
    // various other properties
}

@ApiModel(description = "An area chart.", parent = ChartDTO.class)
public class AreaChartDTO extends ChartDTO implements Serializable {
    // various properties
}

When our swagger definition is generated, the definition for AreaChartDTO looks like this:
"AreaChartDTO": {
"allOf": [
{
"$ref": "#/definitions/ChartDTO"
},
{
"type": "object",
"properties": {
  "id": {
    "type": "string"
  },
  "name": {
    "type": "string"
  },
},
.... more stuff

Note the existence of "allOf", referencing the ChartDTO definition, and then individual properties like "id" and "name".  The definition for ChartDTO looks like this:
"ChartDTO": {
"allOf": [
{
"$ref": "#/definitions/BaseChartDTO"
},
{
"type": "object",
"properties": {
  "id": {
    "type": "string"
  },
  "name": {
    "type": "string"
  },
.... more stuff

Again note that id and name are defined here.  The definition for BaseChartDTO has them as well.  In effect what we're seeing is that properties from the base classes are being duplicated for each concrete class.
When the swagger definition goes through the swagger-parser, it parses successfully, but if you look in definitions, the id and name fields are duplicated 3 times through existence of multiple ComposedModel and RefModel objects.  When I send the Swagger object that comes back from the parser into the Swagger2markup tool, it throws an exception about having multiple entries with the same key: "id".
Is this something that we are configuring incorrectly in our ApiModel annotations?  If not, does this issue belong in swagger-core, swagger-parser, or the 3rd party swagger2markup project?

hmm, to be honest, this sounds like a swagger2markup issue.  If you could, please open a ticket there.  I don't think this belongs in swagger-core, as the spec you're showing looks accurate.

My question is: Why do all definitions (ChartDTO, AreaChartDTO and ChartableDTO) need a id and name property, if it would be sufficient to have them in the ChartableDTO definition? It looks like a bug in the generator which generates the Swagger JSON? What do you think?

It is legal to have them all contain shared properties, JSON schema does say that which one is used is indeterminate.
If the parser is creating multiple parameters with the same name--well that's a bug of the parser.  I do believe internally that it's using linked hash maps, which means it's technically impossible.
If we can create a sample test then I can get it figured out pretty quickly.  @rdiehl do you think you can send a PR with a test that shows the problem?

Not able to see polymorphism definitions in documentation
Model is
@ApiModel(value = "MovieBean" ,discriminator = "type", description = "Prototype for creating movies", parent = EntertainmentBean.class , subTypes = {DocumentaryMovieBean.class , DramaBean.class})
public class MovieBean extends EntertainmentBean {
@ApiModelProperty(value = "movie Id" , position = 1 , required = true)
private int movieId;
@ApiModelProperty(value = "Name of a Movie",  position = 2)
private String name;
@ApiModelProperty(value = "Movie Rating",  position = 3 , allowableValues = "1,2,3,4,5")
private int rating;
@ApiModelProperty(value = "language", position = 4)
private String language;
@ApiModelProperty(value = "type", position = 5 , required = true )
private String type;
}
and generated doc is
definitions":{"MovieBean":{"type":"object","properties":{"language":{"type":"string","description":"language"},"movieId":{"type":"integer","format":"int32","description":"movie Id"},"name":{"type":"string","description":"Name of a Movie"},"rating":{"type":"integer","format":"int32","description":"Movie Rating"}},"description":"Prototype for creating movies"}}}
I am using swagger 2.4.0. Please advise.

There's no swagger 2.4.0.

apologies Let me correct myself.
springfox-swagger2 version 2.4.0 and swagger version 2.0

@jagruti29 then you need to open a ticket on springfox.

Ok thanks .I will raise a ticket.


Support for multiple occurences of attributes

Description
According to the source code pycsw iterates over a , separated string and creates individual elements for each item in case of dc:subject.
file: csw2.py, line 1484 following

                keywords = util.getqattr(recobj, queryables['dc:subject']['dbcol'])
                if keywords is not None:
                    for keyword in keywords.split(','):
                        etree.SubElement(record,
                        util.nspath_eval('dc:subject',
                        self.parent.context.namespaces)).text = keyword

I expected the same behaviour for all identifiers, that are stated to have 0...n occurences according to the specification of CSW.

But surprisingly this is not the case, as I get for my records the following:
...

<dc:format>image/png,image/tiff,image/jpeg,image/png; mode=8bit,image/vnd.jpeg-png,image/vnd.jpeg-png8,application/x-pdf,image/svg+xml,application/vnd.google-earth.kml+xml,application/vnd.google-earth.kmz,application/x-protobuf,application/json,application/vnd.ogc.wms_xml,text/plain,application/vnd.ogc.gml,text/xml,image/png,image/jpeg,image/png; mode=8bit,image/vnd.jpeg-png,image/vnd.jpeg-png8,text/xml</dc:format>
...
<dct:spatial scheme="http://www.opengis.net/def/crs">EPSG:31466,EPSG:31467,EPSG:25832,EPSG:4326,EPSG:4258,EPSG:3857</dct:spatial>

A closer look in the source code reveals the reason for this:
file: csw2.py, line 1497 following

                val = util.getqattr(recobj, queryables['dc:format']['dbcol'])
                if val:
                    etree.SubElement(record,
                    util.nspath_eval('dc:format',
                    self.parent.context.namespaces)).text = val

Is it possible to add support for these attributes, just like for keywords? Or is it already implemented in a newer version of pycsw, that is not available using pip, yet?
Environment

operating system: Deb10 Buster
Python version: 3.7
pycsw version: 2.4.2
source/distribution

[ ] git clone
[ ] DebianGIS/UbuntuGIS
[x] PyPI
[ ] zip/tar.gz
[ ] other (please specify):

web server

[ ] Apache/mod_wsgi
[ ] CGI
[x] other (please specify): Django Dev Webserver

Steps to Reproduce
Not directly a bug, therefore no steps to reproduce
Additional Information
None

@Rastopapola thanks for the report.  Do you have sample metadata so I can reproduce ?  pycsw only supports output of multiple occurrences of attributes for specific properties (like subjects/keywords, links), which are dependent on the input metadata.  Having said this, if you update, say, a Dublin Core document with multiple repeated properties, search results (with elementsetname=full and Dublin Core as the output) will return the DC document as inserted.

@tomkralidis thanks for the fast reply.
I hooked up pycsw as an interface for my own database, where the metadata records do not hold all data directly in one table. E.g. keywords are stored in an own table as individual records and refer to their described metadata using M2M relations and so on. That's why I can not provide any metadata for reproducing the behaviour.
As far as I understood from the source code e.g. the keywords are expected to be returned for the xml building as one larger string, where all keywords are separated using ,.  Therefore I implemented the same behaviour for other attributes, like the supported spatial reference systems, which should be stored in dc:spatial.
What I did is basically mocking the table properties, using model functions decorated with @property. So I could for example create a single string from all related keywords, which works fine. But the same way does not work for other elements with possibly multiple occurences like formats, which would appear in dc:format.
I'm not sure if I only need another delimiter for my returned strings - but I could not see any hint on that in the source code.

@Rastopapola the pycsw data model as it stands is not built for these constructs beyond some fields (keywords, links) -- see https://docs.pycsw.org/en/latest/administration.html#existing-repository-requirements for more info.
For example, you could patch the code above to additionally output multiple occurences of a field given a comma-separated list, but then you would have to additionally make this work across the rest of the outputs (ISO, FGDC, DIF, Atom, etc.).  Then again if core CSW is good enough, then this may be a moot point.
Having said this, an easier workaround would be to serialize and store a Dublin Core document (with multiple occurences, etc. as you wish) in a text field and map the pycsw:XML mapping to that field.  Then, CSW requests like: https://example.org/csw?service=CSW&version=2.0.2&request=GetRecords&typenames=csw:Record&elementsetname=full&resulttype=results (note that elementsetname=true triggers pycsw to look for the pycsw:XML field/mapping and dump that file back to the client, as opposed to serializing the XML) would work.

@tomkralidis Thanks for your feedback!

the pycsw data model as it stands is not built for these constructs beyond some fields (keywords, links)

So it's expected that one metadata record holds exactly one e.g. format, since multiple dc:format elements would not be supported directly using the MD_CORE_MODEL mapping approach?
Thanks for your suggestions. I may write a patch covering this feature.

@tomkralidis For now I'm only going to extend pycsw:Format and pycsw:CRS. Could I create a branch for this or do you have to create one?

Feel free to fork and create a branch.  You can submit a PR thereafter for review.

Pull request has been created and checks have been passed successfully. #618

@tomkralidis
I was writing tests when I noticed something really bad: pycsw is using the owslib, which doesn't seem to allow multiple formats as well:
file: iso.py, line 657
...
class MD_Distribution(object):
    """ process MD_Distribution """
    def __init__(self, md=None):
        if md is None:
            self.format = None
            self.version = None
            self.distributor = []
            self.online = []
            pass
        else:
            val = md.find(util.nspath_eval(
                'gmd:distributionFormat/gmd:MD_Format/gmd:name/gco:CharacterString', namespaces))
            self.format = util.testXMLValue(val)
...

As you can see, the MD_Distribution object takes a single value - more precisely: the first value and ignores other existing formats. This is pretty bad, since it does not follow the specifications for ISO19115, 19119 or 19138, which allows a multiplicity of * for the format objects.
This means I would have to write a patch for the owslib, which has to be accepted and merged, afterwards I can continue the work on here.
Since our use case for the CSW interface is pretty simple, the work I invest in here outgrows the benefits. The easiest for me would be to write an own interface, which covers the requirements of our use case.
I'm sorry, you might do with the Pull request whatever you like but it's not sufficient to crawl deeper and change even more third party packages.
Anyway, thanks for your help!


An error in infer_wild.py

Hello, I have met a difficulty and do not know how to solve it. My error is as follows. May I ask what happened and how to correct it? I'm using a Windows10 computer. Thank you so much!!
(motionbert) F:\DeepLearning\MotionBERT\MotionBERT-main> python infer_wild.py --vid_path video/me.mp4 --json_path video_json/vis_me.mp4.json --out_path output
Loading checkpoint checkpoint/pose3d/FT_MB_lite_MB_ft_h36m_global_lite/best_epoch.bin
0% |                                                                                | 0/2 [00:00<?, ?it/s]L
oading checkpoint checkpoint/pose3d/FT_MB_lite_MB_ft_h36m_global_lite/best_epoch.bin
0% |                                                                                | 0/2 [00:00<?, ?it/s]
Traceback (most recent call last):
File "< string>" , line 1, in < module>
File "F:\Software\Anaconda\envs\motionbert\lib\multiprocessing\spawn.py", line 116, in spawn_main
exitcode = _main(fd, parent_sentinel)
File "F:\Software\Anaconda\envs\motionbert\lib\multiprocessing\spawn.py", line 125, in _main
prepare(preparation_data)
File "F:\Software\Anaconda\envs\motionbert\lib\multiprocessing\spawn.py", line 236, in prepare
_fixup_main_from_path(data['init_main_from_path'])
File "F:\Software\Anaconda\envs\motionbert\lib\multiprocessing\spawn.py", line 287, in _fixup_main_from_path
main_content = runpy.run_path(main_path,
File "F:\Software\Anaconda\envs\motionbert\lib\runpy.py", line 289, in run_path
return _run_module_code(code, init_globals, run_name,
File "F:\Software\Anaconda\envs\motionbert\lib\runpy.py", line 96, in _run_module_code
_run_code(code, mod_globals, init_globals,
File "F:\Software\Anaconda\envs\motionbert\lib\runpy.py", line 86, in _run_code
exec(code, run_globals)
File "F:\DeepLearning\MotionBERT\MotionBERT-main\infer_wild.py", line 70, in < module>
for batch_input in tqdm(test_loader):
File "F:\Software\Anaconda\envs\motionbert\lib\site-packages\tqdm\std.py", line 1178, in iter
for obj in iterable:
File "F:\Software\Anaconda\envs\motionbert\lib\site-packages\torch\utils\data\dataloader.py", line 439, in iter
self._iterator = self._get_iterator()
File "F:\Software\Anaconda\envs\motionbert\lib\site-packages\torch\utils\data\dataloader.py", line 390, in _get_iterator
return _MultiProcessingDataLoaderIter(self)
File "F:\Software\Anaconda\envs\motionbert\lib\site-packages\torch\utils\data\dataloader.py", line 1077, in init
w.start()
File "F:\Software\Anaconda\envs\motionbert\lib\multiprocessing\process.py", line 121, in start
self._popen = self._Popen(self)
File "F:\Software\Anaconda\envs\motionbert\lib\multiprocessing\context.py", line 224, in _Popen
return _default_context.get_context().Process._Popen(process_obj)
File "F:\Software\Anaconda\envs\motionbert\lib\multiprocessing\context.py", line 336, in _Popen
return Popen(process_obj)
File "F:\Software\Anaconda\envs\motionbert\lib\multiprocessing\popen_spawn_win32.py", line 45, in init
prep_data = spawn.get_preparation_data(process_obj._name)
File "F:\Software\Anaconda\envs\motionbert\lib\multiprocessing\spawn.py", line 154, in get_preparation_data
_check_not_importing_main()
File "F:\Software\Anaconda\envs\motionbert\lib\multiprocessing\spawn.py", line 134, in _check_not_importing_main
raise RuntimeError('''
RuntimeError:
An attempt has been made to start a new process before the
current process has finished its bootstrapping phase.
This probably means that you are not using fork to start your
child processes and you have forgotten to use the proper idiom
in the main module:
if name == 'main':
freeze_support()
...
The "freeze_support()" line can be omitted if the program
is not going to be frozen to produce an executable.

Now, I know, because I only have one GPU, num_worker I should set to 0, by the way, prefetch_factor and persistent_workers, what do those two parameters do? If I set num_worker = 0, what should these two parameters be set to?

Please refer to Pytorch documentation to find the proper dataloader configs for your setting.

same error, have you done?


Split ODataCpp.md into ODataCpp-Server.md and ODataCpp-Client.md

Updated Cpp libraries, by adding ODataCpp-Server.md and renaming ODataCpp.md to ODataCpp-Client.md.
Tested locally using "jekyll build && jekyll serve"

 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: abhishekchandratre sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Fix output encoding error

Fix output encoding error in PowerShell script.  @chaim1221 reported it as an issue since it's polluting the otherwise clean looking build log.  The solution is to start a cmd console so that [Console]::OutputEncoding has something to modify.  Whether or not you encounter the error depends on how you invoke PowerShell.  Some contexts include a cmd console already, while others do not.

Thanks! I'm just getting back into the game here. Thanks for closing the issue.


dot notation could be nice?

i was thinking instead of passing an array  ['a', 'b', 'c'] as location
why not 'a.b.c' ?

What would this do?
$data = [
    'b' => ['c' => 1],
    'b.c' => 2,
];
echo igorw\get_in($data, 'b.c');

:+1:


revert Revert disable Jemalloc Hook

What problem does this PR solve?
It is not easy to remove with-jemalloc-prefix, which may affect the compatibility between third-party and old version codes.
Also, will building failed on Mac, it said can't find mallctl symbol. because jemalloc's default prefix on macOS is "je_", not "".
I have repeatedly fixed the compatibility issues between Jvm and Jemalloc, in #33946 #34107 #34578 #34643 #34750 #34886 #35557 #35694 #45210,
but this is all in vain, need to really understand what the problem is between Jvm and Jemalloc.
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[ ] No.
[ ] Yes.

Does this need documentation?

[ ] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.
Please clearly describe your PR:

What problem was fixed (it's best to include specific error reporting information). How it was fixed.
Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
What features were added. Why was this function added?
Which code was refactored and why was this part of the code refactored?
Which functions were optimized and what is the difference before and after the optimization?

run buildall


Rename & move

This is a wide-scale renaming and file re-org. Been wanting to do it for a long time, but have been concerned about merging it with outstanding branches... so I've kept putting it off. But this seems to merge pretty cleanly with the ones I've tried, so going to just get it over with.
This also hopefully solves a long-standing make annoyance, where browserify would fail trying to generate a list of dependencies when the stringified shaders file is missing (even though it's an earlier ilsted build dependency, make calls out to browserify as part of its dependency generation; the easy solution was to create a temporary file and then delete it).
@meetar @iwillig this moves lots of things around but doesn't change functionality. Let me know if you see anything off.

This merged cleanly into one of @meetar's other branches, so I'm going to go ahead with it now.


Video IO device

This is just based off of the work on Z's current PR branch for now, will update to base when that work lands

Opening this up as a quick request for feedback & comment.

https://github.com/LLK/scratch-vm/pull/1016


Cmake error

cmake ../csrc/ -D_OPENGL_LIB_PATH=/usr/lib/nvidia-410
-- The CXX compiler identification is GNU 5.4.0
-- The CUDA compiler identification is NVIDIA 9.0.176
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Check for working CUDA compiler: /usr/bin/nvcc
-- Check for working CUDA compiler: /usr/bin/nvcc -- works
-- Detecting CUDA compiler ABI info
-- Detecting CUDA compiler ABI info - done
-- Found OpenGL: /usr/lib/nvidia-410/libOpenGL.so  found components:  OpenGL EGL
-- Configuring done
CMake Warning at CMakeLists.txt:42 (add_library):
Cannot generate a safe runtime search path for target rasterise because
files in some directories may conflict with libraries in implicit
directories:
runtime library [libEGL.so.1] in /usr/lib/x86_64-linux-gnu may be hidden by files in:
  /usr/lib/nvidia-410

Some of these libraries may not be found correctly.
-- Generating done

Note that this is a warning not an error; sometimes DIRT will still work when it happens.
The warning means you have two versions of OpenGL, system and nvidia, and they aren't "tied together" by GLVND. To avoid the warning, you need to tell cmake not to look in /usr/lib/x86_64-linux at all when searching for OpenGL libraries, which can't be done (AFAIK) without modifying FindOpenGL itself.
If it does crash at runtime (because system libEGL is picked up at runtime instead of nvidia's), you can try with LD_PRELOAD=/usr/lib/nvidia-410/libEGL.so python square_test.py.
If that fails, please run
import dirt.rasterise_ops
import subprocess
subprocess.call(['ldd', dirt.rasterise_ops._lib_path + '/librasterise.so'])

and paste the output here.

This warning will cause the core dumped error  when  running the testing code
i just solved this by
cmake ../csrc/ -D_OPENGL_LIB_PATH=/usr/lib/nvidia-410 -DCMAKE_LIBRARY_PATH=/usr/lib/nvidia-410
thank you!


Project Roadmap ?

Hello.
Do you have a road map for the project? What to expect changes in the project and when?

Hi Aleksey,
Are you looking for something specific ?
My initial intention was to build an easy to use backup solution for oVirt. It's not fully implemented yet. Final product will have  basic functionalities like, backup/restore, scheduling, retention periods, (maybe backup to tape). I am open for feature requests. Please let me know if you need something specific.
For now, I am the only one developing the project, so it takes some time to make things work.

closing the issue.


Zendesk ownership transfer

https://support.zendesk.com/hc/en-us/articles/226885727-Changing-Support-Account-Ownership


It should be noted ReadManyItemsStreamAsync is unordered

[Enter feedback here]
It should be noted ReadManyItemsStreamAsync is unordered. It will not match the order of the list passed in,

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

ID: 819245a4-835a-64aa-d195-60871cacc13c
Version Independent ID: 5cabd119-cec0-b932-0311-cb739ac51939
Content: Container.ReadManyItemsStreamAsync(IReadOnlyList<ValueTuple<String,PartitionKey>>, ReadManyRequestOptions, CancellationToken) Method (Microsoft.Azure.Cosmos) - Azure for .NET Developers
Content Source: xml/Microsoft.Azure.Cosmos/Container.xml
Service: cosmos-db
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

@brianasu-notdoppler Yes, you are correct, the results give the items in the ordered it is stored in Cosmos DB.


Document Intelligence returns tables with multiple table spans

In some document edge cases, the spans for any given table index could have more than one span depending on the document, the table and it's layout. Until now we only handle the first span in the array.
If you see this and would like a fix, see the code changes made in this Pull Request for next version release 1.1
https://github.com/microsoft/PubSec-Info-Assistant/pull/462
What is the expected behavior?
For it to handle multiple spans if they exist
Screenshots

Information Assistant details
Affected versions v1.0

GitHub branch: [e.g. main]
main

This was merged into main as a hotfix in #478.


WARNING:Shape of head.cls_preds.0.weight in checkpoint is torch.Size([80, 96, 1, 1]), while shape of head.cls_preds.0.weight in model is torch.Size([2, 96, 1, 1]).

Hi Im training my custom datasets in voc format.
training well now but have warning at start.
could this warning have bad result? or just ignore it?
my command and waning below,
$ python3 tools/train.py -f exps/example/yolox_voc/yolox_voc_tiny.py -d 1 -b 64 --fp16 -o -c yolox_tiny.pth.tar

you use the custom datasets you should change the code in the
yolox/data/datasets/voc.py
data/datasets/voc_classes.py
yolox/exp/yolox_base.py

I changed the code in voc_classes.py , yolox_base.py.
which portion of the code should be changed in voc.py?

Hi Im training my custom datasets in voc format.
training well now but have warning at start.
could this warning have bad result? or just ignore it?
my command and waning below,
$ python3 tools/train.py -f exps/example/yolox_voc/yolox_voc_tiny.py -d 1 -b 64 --fp16 -o -c yolox_tiny.pth.tar

It's normal, as the coco pretrained weight has 80 classes but voc only has 20


Facing issue no module named Newspaper

altair==4.2.2
en-core-web-sm @ https://github.com/explosion/spacy-models/releases/download/en_core_web_sm-2.3.1/en_core_web_sm-2.3.1.tar.gz#sha256=06c80936324012d1223291d2af41a5229e746dc2dee8fe31a532666ee3d18aaa
gTTS==2.3.2
h5py==3.9.0
matplotlib==3.7.1
newspaper3k==0.2.8
nltk==3.8.1
numpy==1.23.5
pandas==2.1.0
googletrans==4.0.0rc1
scikit-learn==1.3.0
streamlit==1.22.0
tensorboard==2.12.3
tensorflow==2.12.0
textblob==0.17.1
wordcloud==1.9.2
yake==0.4.8
I am using this in my requirements.txt and also using python version 3.9.17 the same one I used for this locally but when i am deploying in huggingface space it is saying no module named "Newspaper" whereas it is already present in requirements.txt. How can i solve this issue??

Hey @Shoaib-33 could you share which Space it is? And could you check also the build logs to see if the dependencies are installed? Also can it be simply a problem with the import (import newspaper instead of import Newspapers).
Without more information is it hard to help :)


foreach: add a --xfail option

For instance, to check a file is absent,
you may run:
tsrc foreach --xfail ls <the-file>

Yeah, if it's hard to explain, it's probably a bad idea. Let's not merge this


regex for matching words that aren't html tags

This regex gives a much better word count in markdown:
\b(\w+(?![^<>]*>))\b

because it ignore html tags (which don't count as words in the output).

You can add it in configuration now. These days I do not have enough time to upgrade. Next major version 2.0 should support most common mark languages. Please be patient.


ARTEMIS-2773 Dockerfile Improvements

The current Dockerfile based on openjdk:8 does not build due to an unavailable version of libaio1 specified. It explicitly references version 0.3.110-3 of libaio1 however, it appears that 0.3.112-3 is the current stable version and when attempting to build the Dockerfile the following error occurs:
Step 9/20 : RUN groupadd -g 1000 -r artemis && useradd -r -u 1000 -g artemis artemis && apt-get -qq -o=Dpkg::Use-Pty=0 update && apt-get -qq -o=Dpkg::Use-Pty=0 install -y --no-install-recommends libaio1=0.3.110-3 && rm -rf /var/lib/apt/lists/*
 ---> Running in f56ff568c477
E: Version '0.3.110-3' for 'libaio1' was not found

In addition, I received this error attempting to run prepare-docker.sh:
cp: cannot stat './{Dockerfile-centos,Dockerfile-ubuntu,docker-run.sh}': No such file or directory
Finally, the Dockerfile states it is Ubuntu, when in fact openjdk:8 is now Debian, so the names of the scripts and the readme should reflect this.

I made one little ammend to the commit name before pushing. Take a look on master just for future PRs.
Thanks a lot.


Update awslogs.md

Signed-off-by: Justin Menga<EMAIL_ADDRESS>
Proposed changes
Add documentation for new AWS CloudWatch logs multiline pattern options
Unreleased project version
Docker Engine 17.06
Related issues
https://github.com/moby/moby/pull/30891

@nathanleclaire @FrenchBen @friism PTAL

@nrdlngr - changes submitted as requested

@FrenchBen PTAL or nominate.

I think this is more of a review by @samuelkarp - if he's happy with it, I don't see any blockers

I interactive rebased and removed the unrelated commit.

This was already addressed in #3615

@mstanleyjones It looks like https://github.com/docker/docker.github.io/pull/3615 lost a lot of the content from this one.  Is that intentional?

Not necessarily, no. I'll review it and see what we need to keep from here. I wasn't focused on this PR when I did the work on that other one.

I'm taking this over to get the important info in here ported over.


Adds tile pre-conditions and validation, closes #122

For #122.
I've added pre-conditions and validation to the tile parsing logic.
This uncovered two existing problems in mercantile

one problem in simplify I could fix, and
one problem in the bounds <-> bounding_tile roundtrip I'm unsure how to fix properly (see inline comments)

Note: I've cherry-picked some commits from #123 in here to develop locally.

There's one test failing right now I'd love your input on how to fix: the bounding tile roundtrip.
https://github.com/mapbox/mercantile/blob/d00e29330663d2ca8dc1f4056f92194286250b38/tests/test_funcs.py#L424
The offending function is bounds with Tile(0,0,0)
https://github.com/mapbox/mercantile/blob/d00e29330663d2ca8dc1f4056f92194286250b38/mercantile/__init__.py#L174-L196
because we then construct Tile(0+1, 0+1, 0) which is an invalid Tile here and call ul on it
https://github.com/mapbox/mercantile/blob/d00e29330663d2ca8dc1f4056f92194286250b38/mercantile/__init__.py#L195
How should we handle the Tile(0,0,0) bounds edge case? Special case and hard-code the -180, -85.., 180, 85.. bounds?

Superceded by #130 .


Deepl API Key Invalid

hello i use the deepl api for foundry v10 after i updated my account it keeps telling me that my api token is invalid but my token now replaced several times always the same problem

Did you get a Pro-Subscription for DeepL? If yes, the module currently does
not support Pro accounts. It should be an easy addition, but I won't be
able to work on it until tomorrow.
Am So., 26. März 2023 um 18:10 Uhr schrieb Vandree1977 <
@.***>:

hello i use the deepl api for foundry v10 after i updated my account it
keeps telling me that my api token is invalid but my token now replaced
several times always the same problem
—
Reply to this email directly, view it on GitHub
https://github.com/Meneltamar/fvtt-comprehend-languages/issues/23, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AE4ZVBWAWFRS2RMKH5LNEE3W6BZ77ANCNFSM6AAAAAAWIIP7JM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Now that I think about it, I don't think I am comfortable implementing the Pro API URL into the module, as I don't want to be responsible for people racking up charges, if anything goes wrong.
I can show you where you have to make a change in the files, to get it to work.
Could you confirm that you upgraded to an API Pro account?

Yes I have performed an update

I have updated to API Pro today

Go into your modules folder.
Under modules/comprehend-languages/scripts open the "main.js" file with any text editor. In line 503 (or search for "api-free"), replace the code like this:
let response = await fetch("https://api-free.deepl.com/v2/translate?" + data, {

becomes
let response = await fetch("https://api.deepl.com/v2/translate?" + data, {

Save the file, restart your world and it should work.
Please note:
I do not take any responsibility for charges you receive on your DeepL Account due to errors in the module. I cannot guarantee that the translations always work and I know that the error handling in the module is not the best at the moment. This could mean that if you try to translate large JournalEntries that the translation completely fails but your account still gets charged. Changing this behavior (to at least have the parts of the translations that worked saved into Foundry) is on my list, but I won't be able to work on it soon. You will have to do this on your own responsibility.

You will have to make this change again whenever the module is updated.

Hello, I am playing on The Forge and somehow I don't have the option to change anything in the script. I cannot access the module. Can you help me?


Text-Range Module's Selection's move() does not see optional options argument when it should.

Version: 1.3.0
Module: Text-Range
For reference... https://github.com/timdown/rangy/wiki/Text-Range-Module#movestring-unit-number-count-object-options-1
When I use rangy.getSelection().move("word", -1, options), I expect the options parameter to be passed. However, my IDE (WebStorm... sorry) gives me a warning that there are only 2 arguments when it should pass 3.
From the documentation....:
move(String unit, Number count[, Object options])
I even use the options parameter for expand(), and it works fine, but move() doesn't work. I know this option parameter doesn't work at all because when I add a different regex and pass it in as an option, it does not use it at all.
Example
Where the pipe is the cursor........
Text:
help-me|
Execution (sorry... in coffeescript):
# a word in this case includes characters and numbers with its apostrophes and/or hyphens
regex = /[a-z0-9]+(['\-][a-z0-9]+)*/gi

rangy.getSelection().move("word", -1, {
  wordRegex: regex
})

Expected:
|help-me
Actual:
help-|me

I'll look into this as soon as I can (hopefully over the weekend) Apologies, I have had other priorities in recent months.


Remove SyncPeriod from webhook server

This is a dead parameter included in the initial implementation of the webhook server

I was always wondering what it did 😅


Flickering issue

I'm using progress circle with dynamic data that's getting updated via my post request, everything is working great but the progress circle keeps flickering
this is the code
 .uploadProgress((written, total) => {
                        this.setState({
                            overlayIsVisible:true,
                            overlayProgress:written / total,
                        });

                    }).then((response) => {

Fixed, something on my end


First cut of the Enum Macro

First cut of the Enum Macro.
Currently it works on named and unamed enum variants. Each field must implement View + Drawable. I included an example that use a random number generator (tinyrng to stick to the no-std depandancies) to pick what variant should be displayed to prove that any can be selected.
One of the problems I am currently running into is that it does now work with LinearLayout. The problem is that LinearLayout does not implement Clone + Copy which causes the impl Transform to fail since the translate is given a &self and usally calls translate which accepts a mut self. Have to see if there is another way to implement it but it seems like I will need to add Clone + Copy to LinearLayout (which in turn means it has to be added to Views and Chain). Not sure if that is doable on Views since it holds a &'a mut [T] . Any feedback/suggestions is greatly appericated. Thanks.

Thanks for working on this!
I think we shouldn't introduce a new macro and instead we should include enums in ViewGroup. Not quite the same thing, but I just find it a bit weird to need two separate macros to essentially turn complex types into something workable. Also some of it overlaps when you have an enum variant with multiple views in it.
I'm not sure how I feel about Clone. I would like to avoid that if possible, but I need to read this PR first.
Lastly, you don't need randomness and 5 enum variants in the example. I would suggest to keep it simple and easy to read, as it serves as a guide for users.

I think we shouldn't introduce a new macro and instead we should include enums in ViewGroup. Not quite the same thing, but I just find it a bit weird to need two separate macros to essentially turn complex types into something workable. Also some of it overlaps when you have an enum variant with multiple views in it.

Move code into the current ViewGroup macro.

Lastly, you don't need randomness and 5 enum variants in the example. I would suggest to keep it simple and easy to read, as it serves as a guide for users.

Removed RNG. Sorry got a bit carried away :smile:

I'm not opposed to implementing Clone for LinearLayout, but it looks like it's not necessary for this PR. We can keep it, but ideally it would be nicer to add it in a different PR.

Moved.


Add python script selection in argparse

with a -f / --file argparse argument, instead of blindly acting on all of them everytime.
This will be useful for #6

just to be sure, if someone use -f and tp_id, we should apply the action to both the files with the -f and the files in the tp_id folders ?

yes

this "and" might be a quiproquo.
tp_id and file are filters. when they are both specified, we should only act on files where tp_id and file match.


Remove old ubiquity linux service, config files and few old deployment files

This PR is for removing old files that are no longer relevant in the project. Below the list:

Remove old ubiqutiy linux service concept. From 1.0 we moved to deploy ubiqutiy as k8s deployment instead of regular linux service. Removing the following related files:
scripts/setup
scripts/ubiquity-server.env
scripts/ubiquity.service
ubiquity-server.conf

Remove old k8s deployment files. Since the full installer is locate in https://github.com/IBM/ubiquity-k8s/tree/master/scripts/installer-for-ibm-storage-enabler-for-containers. Removing the following related files:
deploy/postgres/master-service.json
deploy/postgres/replica-service.json
deploy/postgres/service.json
deploy/postgres/service_account.json
deploy/postgres/set.json
deploy/ubiquity/scbe_ubiquity_deployment.yml
deploy/ubiquity/ubiquity_deployment.yml
deploy/ubiquity/ubiquity_namespace.yml
deploy/ubiquity/ubiquity_service.yml

This change is

Coverage remained the same at 51.479% when pulling 7ccf8998326cbd7f4e852c06d3d0c87f280a1d17 on cleanup_old_installation_method into da0d975cc81f8a7b35fc0f1a697151927eee15ec on dev.


Folders on LiveShare

Type: Feature Request
My team and I are having issues on opening multiple folders whilst using LiveShare. So let's say I'm a host and I have "Personal" folder open, my team will be able to see that on liveshare and then I might want to open another folder to show them, like "Thank you" folder, they can't see the second folder I open whilst on my liveshare.
Can we get that feature or can we get assistance if it is there.
Thank you.
VS Code version: Code 1.81.1 (6c3e3dba23e8fadc360aed75ce363ba185c49794, 2023-08-09T22:22:42.175Z)
OS version: Windows_NT x64 10.0.22621
Modes:

Please file the issue directly against the LiveShare extension: https://github.com/MicrosoftDocs/live-share/issues


init pygame

init the pygame
test if we have installed pygame successfully

PR Reviewer Guide
After analyzing the pull request, here are my findings:
Overall Feedback:
The initial setup for integrating the Pygame library is a good start for the project. The code is straightforward, but there are a few improvements that can enhance code quality and maintainability.
Score: 85/100
Labels: Enhancement, Other
Code Suggestions:

File: main.py

Language: Python
Suggestion Content: Add a newline at the end of the file to adhere to PEP 8 standards.
Relevant Line: +if __name__ == "__main__":
Existing Code:+if __name__ == "__main__":

Improved Code:+if __name__ == "__main__":
+    main()
+

File: main.py

Language: Python
Suggestion Content: Consider adding error handling for Pygame initialization to ensure that the library is set up correctly.
Relevant Line: +    print("Starting asteroids!")
Existing Code:+    print("Starting asteroids!")

Improved Code:+    try:
+        pygame.init()
+        print("Starting asteroids!")
+    except Exception as e:
+        print(f"Failed to initialize Pygame: {e}")

Good start with the Pygame integration! Here are some suggestions:

Add a newline at the end of main.py to comply with PEP 8 standards.
Implement error handling for Pygame initialization to ensure proper setup.

Persistent review updated to latest commit https://github.com/ngduyanhece/asteroids/commit/711812e80ffb09e2c3db79464d0cb0653a3b8287

PR Reviewer Guide
After analyzing the pull request, here are my findings:
Overall Feedback:
The initialization of Pygame is a good start for the project. The error handling during initialization is a positive addition, ensuring that any issues are reported. However, there are a few improvements that can enhance the code quality and maintainability.
Score: 85/100
Labels: Enhancement, Bug fix
Code Suggestions:

File: main.py

Language: Python
Suggestion Content: Ensure that the Pygame quit function is called to properly clean up resources when the program exits.
Relevant Line: +        pygame.init()
Existing Code:pygame.init()

Improved Code:pygame.init()
# Your game logic here
pygame.quit()

File: main.py

Language: Python
Suggestion Content: Consider adding a return statement in the main function to indicate successful completion.
Relevant Line: +    main()
Existing Code:main()

Improved Code:return 0

File: main.py

Language: Python
Suggestion Content: Add a newline at the end of the file to adhere to PEP 8 style guidelines.
Relevant Line: +    main()
Existing Code:main()

Improved Code:main()

Good start with Pygame initialization! Here are some suggestions:

Ensure to call pygame.quit() to clean up resources when the program exits.
Consider adding a return statement in the main function to indicate successful completion.
Add a newline at the end of the file to adhere to PEP 8 style guidelines.

Persistent review updated to latest commit https://github.com/ngduyanhece/asteroids/commit/a891145e610c96633a2af0d51921cf0815dbbadf


How to Merge Code Within Parentheses into a Single Line Using Black Formatter in VSCode?

How can I use the black-formatter plugin  in VSCode to merge code within the same parentheses into a single line?
For example, I want the following two lines of code:
nccl_kernel = NcclKernel(kernel_id, node_proc, nccl_comm, nccl_rank,
                         func_index, launch_order, channel_count)

to automatically merge into one line after pressing Ctrl+S:
nccl_kernel = NcclKernel(kernel_id, node_proc, nccl_comm, nccl_rank, func_index, launch_order, channel_count)

Do you have following settings in your settings.json?
  "[python]": {
    "editor.defaultFormatter": "ms-python.black-formatter",
    "editor.formatOnSave": true
  }

Yes, I have the following settings
    "black-formatter.args": [
        "--line-length", "300",
        "--skip-magic-trailing-comma", "true",
    ],
    "[python]": {
        "editor.defaultFormatter": "ms-python.black-formatter",
        "editor.formatOnSave": true
    },

Thank you, it's working now. Additionally, when I press Ctrl+S, the single quotes in Python (' ') automatically change to double quotes (" "). How can I disable this automatic change?

Additionally, when I press Ctrl+S, the single quotes in Python (' ') automatically change to double quotes (" "). How can I disable this automatic change?

This extension just runs black for you. You will need to see if black has a command line option for things like this. You can add —skip-string-normalization. Like this:

"black-formatter.args": [
        "--line-length", "300",
        "--skip-magic-trailing-comma",
        “— skip-string-normalization“
    ],
    "[python]": {
        "editor.defaultFormatter": "ms-python.black-formatter",
        "editor.formatOnSave": true
    },


Cargo.toml is missing "repository" link

I noticed that this downstream services like docs.rs and crates.io are unable to link back to this repository because the Cargo.toml file is missing the repository URL.
Could you please add the link?

Nthing this issue. I was looking for the repository for this crate so I could watch it, but I couldn't find the link without Googling.
This would likely be resolved by #5

@eira-fransham Hi! Could you please look at this issue and merge proposed fix?


[FALSE-POSITIVE]  dlvr.it

Domains or links
dlvr.it
More Information
How did you discover your web site or domain was listed here?
Tried to visit a site served via dlvr.it
Have you requested removal from other sources?
This is the only source
Additional context
This is a URl shortener, not a phishing site.

@mitchellkrogza This is back on the active list again.


[Fix-10452] Serial wait for policy recovery

Purpose of the pull request
Serial wait for policy recovery
Brief change log
close #10452
Verify this pull request
Manually verified the change by testing locally.

Hi @WangJPLeo , there are some ut failed.

just fixed UT, thank you @SbloodyS

Codecov Report

Merging #10453 (db6c0fe) into dev (190f253) will increase coverage by 0.33%.
The diff coverage is 45.83%.

@@             Coverage Diff              @@
##                dev   #10453      +/-   ##
============================================
+ Coverage     40.55%   40.88%   +0.33%
- Complexity     4774     4849      +75
============================================
  Files           878      883       +5
  Lines         35741    35978     +237
  Branches       3969     3993      +24
============================================
+ Hits          14493    14710     +217
- Misses        19805    19814       +9
- Partials       1443     1454      +11

Impacted Files
Coverage Δ

.../server/master/runner/WorkflowExecuteRunnable.java
7.73% <0.00%> (-0.03%)
:arrow_down:

...nscheduler/service/process/ProcessServiceImpl.java
30.64% <57.89%> (-0.13%)
:arrow_down:

...org/apache/dolphinscheduler/remote/utils/Host.java
37.77% <0.00%> (-2.23%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
51.38% <0.00%> (-1.39%)
:arrow_down:

.../org/apache/dolphinscheduler/api/enums/Status.java
100.00% <0.00%> (ø)

...er/api/service/impl/TaskDefinitionServiceImpl.java
24.01% <0.00%> (ø)

...olphinscheduler/common/utils/ClusterConfUtils.java
0.00% <0.00%> (ø)

...scheduler/api/service/impl/ClusterServiceImpl.java
88.28% <0.00%> (ø)

...hinscheduler/api/controller/ClusterController.java
89.47% <0.00%> (ø)

... and 6 more

Continue to review full report at Codecov.

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


Changes around block-modal

Make "Include the transcript of this chat" checked by default when checking "Report user" in block modal.

This does not imply we are sending transcript by default whenever you block anyone - you still need to check "Report user" for transcript to even be considered.

Add "Block user" menu item into message-popup menu (the ... menu for text messages and exploding messages).

There is some logic around when to say "Block user" and "Report user", and when to use alternative label in block modal for blocking user: "Block user from messaging me directly" (instead of "Block user").

Ran into troubles with this thing regarding storybooks. I'm not sure if my fixes are good there.


go get failure

Hi,
I just tried to Go get grump, but ran into the following error. Just to let you know.
$ go get -v github.com/dhulihan/grump
go: downloading github.com/dhulihan/grump v0.0.0-20201015025354-2413accc6e5c
get "golang.org/x/sys": found meta tag vcs.metaImport{Prefix:"golang.org/x/sys", VCS:"git", RepoRoot:"https://go.googlesource.com/sys"} at //golang.org/x/sys?go-get=1
go get: github.com/dhulihan/grump@none updating to
<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>invalid version: unknown revision 58c0f4d7150f

Cheers,
Robin
Edit: oh, and just as a note, seek support for FLAC is soon to be added in Beep. https://github.com/faiface/beep/pull/111

Thanks for the catch @mewmew ! Forgot to run go mod tidy recently 😅 . Go ahead and try go get -v github.com/dhulihan/grump again. Let me know if that doesn't address this problem.
Also, that's exciting that flac seek support is coming soon! I'll keep an eye on that issue.

Thanks for the catch @mewmew ! Forgot to run go mod tidy recently sweat_smile . Go ahead and try go get -v github.com/dhulihan/grump again. Let me know if that doesn't address this problem.

Thanks, now it works!

Also, that's exciting that flac seek support is coming soon! I'll keep an eye on that issue.

Definitely, this was the work of @cswank.


feat: add Concourse#recall(...) implementation in Concourse.java

Php/python/ruby implementations are still a WIP, since my approach doesn't use thrift

Tagging jira task: CON-231
@jtnelson

@dearyhud I haven;t forgotten about this one. I'll get it merged in this week!

Sounds good! Updating the PHP, Ruby, and Python drivers is not as straight forward as I thought.

@jtnelson do you want to bail this pr?

nah i promise i'm going to merge it :)
JEFF NELSON
Founder & CEO
www.cinchapi.com
Follow Cinchapi on Twitter https://twitter.com/cinchapi | Github
https://github.com/cinchapi | AngelList https://angel.co/cinchapi
On Tue, Aug 30, 2016 at 11:45 AM, Deary Hudson III<EMAIL_ADDRESS>
wrote:

@jtnelson https://github.com/jtnelson do you want to bail this pr?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cinchapi/concourse/pull/147#issuecomment-243484697,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABbRmrHGcKMH7Tp-5RhH-uDGItSnrUi2ks5qlFAvgaJpZM4IOVdX
.

Okay cool!

yo - @jtnelson any updates for this pr? or should we close


Fix erroneous reading ref; cf, BCP 992.

User found an error; fortunately and amusingly, even the longer Mark ends precisely before the verse specified.

Thanks very much!


chore: fix typo

Closes #
Describe your proposed changes here.
In tsdb/engine/tsm1/reader.go, I found a word misspelling in line 733. The word "minium" should be "minimum".

[ ] CHANGELOG.md updated with a link to the PR (not the Issue)
[ ] Well-formatted commit messages
[ ] Rebased/mergeable
[ ] Tests pass
[ ] http/swagger.yml updated (if modified Go structs or API)
[ ] Feature flagged (if modified API)
[ ] Documentation updated or issue created (provide link to issue/pr)
[ ] Signed CLA (if not already signed)

Hi @LLThomas! Thanks for catching this typo. Would you mind updating the commit message so the Semantic Pull Request test passes? You should be able to use something like chore: fix typo.

Everything looks good now.


extra_host fix for linux

linux users were having issues running sphinx-stack on their local because /etc/hosts wasn't mapping host.docker.internal to their localhost this fixes this issue without having the user having to manually edit the /etc/host file

oh i missed this PR, looks good. U think its good to merge @kevkevinpal ?


Compatibility with visibility

Thank you for your crate! One thing that prevents me from using it is the fact that it doesn't seem to handle visibility (or that I don't know how to do it!).
nest! {
    pub enum Level0 {
        Level1(enum Level1 {
                Level2(enum Level2 {
                    A,
                    B,
                }),
        }),
    }
}

Level0 will be public, but not Level1 and Level2. How can I make Level1 and Level2 public as well?

I will work on finding a solution to that!

Issue should be fixed per #3. This issue will be closed within 48 hours


Assess whether or not there is a  hardcoded dependency on "LowParcelID" in the application and in the GP tools

Validate if there is a hardcoded dependency on "LowParcelID" for the GP tools and \or the application
@busbus This was an issue in the old app. I think we fixed it, but I am not sure. We should test it out. If there is a hardcoded dependency then we need to remove it.

@LindsayThomas
We investigated this and can confirm that there is no hard coded dependency on "LowParcelID" in the gptools and application. It is a required configurable parameter.

The lowest tax parcel identification number is an integral part of the Local Gov Information Model, and is used to uniquely identify the condominium polygon in the parcel fabric.
To verify that the field name is not hard coded, we altered the field name LOWPARCELID without modifying the ids in the table and updated the same in configuration file. The app worked as expected.
Please confirm if this is what you were trying to accomplish.

@CTLocalGovTeam Okay. Thanks that makes sense.
========================
Note to LT and CB this works with naperville data (Using one Parcel ID field)
        ParcelInformation: {
            LowParcelIdentification: "PARCELID",
            ParcelIdentification: "PARCELID",
            AliasParcelField: "Parcel Identification Number",
            SiteAddress: "SITEADDRESS"


Add package_ensure parameter

init.pp has service_ensure and service_enable, but no package_ensure. install.pp thus always ensures the package is present. There is no way to use this module without having it install the package, or to enforce the installed version (like latest).
This would be a useful feature, like in our case where we want to use puppet to create the configuration, but don't want it installed because we want to run bind in a Docker container.

pull requests welcome :)

This can be achieved as well by passing an empty string to the dns_server_package parameter. Patch is still welcome because users might want to ensure latest.

Closing since it's open for a while and there's a workaround. A PR is still welcome.


Kotlin coroutine update.

The Flow interface is not stable for inheritance in 3rd party libraries. Updated code to use the AbstractFlow as recommended instead via the sealed MongoAbstractFlow class.
JAVA-4950

Hi @zigzago - just wanted to double check you are happy with this PR

If you look at the implementation of MongoAbstractFlow, it adds some behaviour
public final override suspend fun collect(collector: FlowCollector<T>) {
        val safeCollector = SafeCollector(collector, coroutineContext)
        try {
            collectSafely(safeCollector)
        } finally {
            safeCollector.releaseIntercepted()
        }
    }

Also in the doc of AbstractFlow (https://kotlinlang.org/api/kotlinx.coroutines/kotlinx-coroutines-core/kotlinx.coroutines.flow/-abstract-flow/):
"Base class for stateful implementations of Flow". I'm not sure that mongo flows are stateful.
I don't say your implementation will not work, but it would be cleaner for my point of view to use the kotlin delegate pattern:
class AggregateFlow<T : Any>(private val wrapped: AggregatePublisher<T>) : Flow<T> by wrapped.asFlow()

Then you rely completely on Publisher.asFlow() kotlin implementation (which is safe).
HTH
Julien

Hi @vsct-jburet,
Good points, they are stateful in that they contain the cursor state, but that is all managed by the Publisher and not the Flow.  Will update to MongoFlow and delegate instead.
Ross


Modify graphical user interface

Currently, InternBuddy inherits the GUI from AB3. In order to stand-out let's try to improve the GUI.
The improved GUI

Changes:

Add InternBuddy header for better recognizability
Add list title showing the current list type
Rearrange command box, command result box, and list view to give users better focus on the list
Recolor components as an attempt to make the GUI look better
Size adjustments to status bar footer and command result box
Change app icon and title to InternBuddy

Codecov Report

Merging #89 (6ddf080) into master (90d08ec) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master      #89      +/-   ##
============================================
- Coverage     47.45%   47.41%   -0.05%
  Complexity      451      451
============================================
  Files           104      104
  Lines          2221     2223       +2
  Branches        318      318
============================================
  Hits           1054     1054
- Misses         1089     1091       +2
  Partials         78       78

Impacted Files
Coverage Δ

...c/main/java/seedu/address/ui/CompanyListPanel.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/EventListPanel.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/address/ui/PersonListPanel.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/UiManager.java
0.00% <ø> (ø)

Continue to review full report at Codecov.

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


CriticalCSS

Probably overkill here, but I got a reputation to maintain.

How would I do this with 11ty/Nunjucks? I don’t think I can import/include/whatever the generated styles into head. I bet somebody has done this before.

Am I happy with just (“just”) using HTTP/2 server push, for this?

Calling it done.


fix for tooling errors

All my tooling regarding panda stopped working as soon as I added this preset. That is @pandabox/prettier-plugin, @pandacss/eslint-plugin and the VS Code plugin. The error was all the same, related to node-eval. This fixed it for me on all instances.

Hi! Sorry for the wait. Thank you for the PR! I'll get it merged and published right away.


chore(lint): unite eslint rules

Closes #2689
Even though original thought was to have only a single rule for each version, but it turned out to be incorrect from user's perspective. Pretty much we have 3 different rule cases:

Deprecated paths (not auto fixable)
Deprecated methods (auto fixable)
Deprecated alias (auto fixable)
If we combine all of them into one, there might be a case where the user has only path error. Since we declare our new combined rule as auto fixable, it will misinform user about us automatically fixing his problem. Which is false information.

That's why it was decided to only combine alias rules into one + refactor other rules to accept array of rule cases

:tada: This PR is included in version 5.0.0-beta.37 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


How does this differ from VideoJS?

Can someone briefly explain why would I use this, how does it differ from VideoJS? I thought VideoJS was a html5 player? Does this package just add more options or skins or something? It is very confusing navigating the javascript libraries for recordrtc and video recording

it's a plugin for video.js. video.js provides a skinnable player and component structure, and videojs-record extends this with cross-browser/platform recording for audio, video and image files.


Discussion: Float encoding

At the risk of causing bikeshedding, I wanted to raise a point of discussion regarding how floats are encoded -- given that integers are encoded in Base62 and therefore take up less space than their decimal form, it seems a bit odd that floats are left as-is.
My first idea was that they could also be encoded in a similar compacted form, such as base36, which JavaScript can do natively with .toString(36), but that isn't particularly great if the float's magnitude is not close to 0 because the expontent e+n notation can't be used unambiguously, and parsing them back from such a format probably isn't easy (definitely not in JavaScript.)
I'd love to hear more about the rationale behind this decision (if any)! All in all, for the goals that it sets out to accomplish, JCOF is really neat.

So, having a way to do base62 encoding of floats has struck my mind, but I've disregarded it since I wanted to use JavaScript's float generator and parser. I didn't consider the fact that JavaScript lets you convert floats to strings with different bases.
I think the base 10 syntax has to stay. That doesn't necessarily mean we can't add a base 36 (or base 62 or anything else) in addition, but here are the reasons why we can't replace the base 10 syntax with a base 36 syntax:

Currently, the characters 0 through 9 and - indicate that the coming value is a float. This means we can encode floats without a leading character. Some of the lower-case characters are used as a leading character to indicate other values; i for positive base64 integer, n for null, b for true boolean, s for string reference. Now, we could replace al those characters with upper case characters so that the characters 0 through 9, - and a through z indicate float, and maybe that would be worth doing, except:
Base 36 turns out to be larger in a lot of cases. Numbers aren't always randomly picked from the real number line; often, our datasets contain numbers which look okay in base 10. For example, one test dataset contains a lot of numbers rounded to 2 base-10 decimal places. Moving from base 10 to base 36 means encoding 1.58 as 1.kvoha2voha instead, or encoding 2.2 as 2.7777777778.
 Removing a way to represent numbers as base 10 would make it much harder for a human to write a JCOF document.

I implemented a naïve change which just replaces the current value.toString() with value.toString(36) to see what kind of difference it makes. Here it is, with jcof being the old base 10 floats and jcff being the new base 36 floats:
tiny.json:
  JSON: 299 bytes
  jcof: 134 bytes (0.448x)
  jcff: 134 bytes (0.448x)
circuitsim.json:
  JSON: 8315 bytes
  jcof: 2093 bytes (0.252x)
  jcff: 2090 bytes (0.251x)
pokemon.json:
  JSON: 219635 bytes
  jcof: 39650 bytes (0.181x)
  jcff: 39643 bytes (0.180x)
pokedex.json:
  JSON: 56812 bytes
  jcof: 23132 bytes (0.407x)
  jcff: 25861 bytes (0.455x)
madrid.json:
  JSON: 37960 bytes
  jcof: 11923 bytes (0.314x)
  jcff: 11618 bytes (0.306x)
meteorites.json:
  JSON: 244920 bytes
  jcof: 87028 bytes (0.355x)
  jcff: 95370 bytes (0.389x)
comets.json:
  JSON: 51949 bytes
  jcof: 37480 bytes (0.721x)
  jcff: 37471 bytes (0.721x)

Now, there is a use-case for a more efficient float encoding. Storing the state of a physics-based game, for example, would have a lot of entities with position and velocity vectors which contain values that are essentially picked arbitrarily from the real number line without regard for any base. And base 36 or 62 floats would certainly be an advantage there. I wouldn't be against adding a new float syntax, maybe one which works like the base 62 integer syntax today; f followed by a base 36 or 62 float for positive floats, F for a negative float. My concern there would be: All languages probably have a base 10 float parser, but some languages may not have a base 36 float parser. I think I want to avoid making people hand-write float parsers and generators, since parsing and generating floats efficiently in a way which can round-trip all values without loss seems like one of those classic hard problems. I should do some research in this space.

I was wondering about the same thing. Here's one idea I had, maybe it would work for your usecases? If it clashes with some design nuance I missed in my brief exposure to the library feel free to disregarded.
Since we already have the possibility of exponential notation via toExponential, e.g. (0.000123).toExponential() becomes 1.23e-4, why not think of that notation as a triplet of values (1, 23, and 4) that each can be converted to Base62? If we start with an fprefix to indicate we'll have a floating point value we can drop thee, instead using +or-as a separator. In our example that would result inf1.n-4`.
But we can do better: we can get rid of the . between 1 and 23, as long as we add the change in powers of ten to the coefficient to the exponent to compensate. So in this case, changing 1.23 means multiplying by 100, so 10⁻⁴ has to become 10⁻⁶ to compensate.
So 0.000123 becomes 1.23e-4 becomes 123-4 becomes 123-6, which becomes f1Z-6
I have a working but totally unsafe implementation here:
https://observablehq.com/d/e3010313d892b36e
(unsafe as in "it does not test and bail for invalid string input and probably can result in infinite loops as a result". But you already have a B62 parser yourself so that's the hard part handled)
Number.prototype.toExponential() is so old that even IE5 supported it, so I don't think you have to worry about it not being available.

Hahaha, dammit, all that work for a test implementation and I just realized that there is much simpler approach that seems to work a lot better:

@mortie, thanks for the detailed response! Your reasoning makes a lot of sense.

Removing a way to represent numbers as base 10 would make it much harder for a human to write a JCOF document.

Well, this doesn't entirely hold for base 62 integers either, but given that in JS there's no difference between integers and floats, I suppose at the moment the float notation works as a makeshift solution for that :)
I respect that any change should be justified by both a use-case and data, and that the present notation is good enough for what JCOF sets out to achieve -- though as you said, there are some cases where a lot of floats might need to be passed around, and there's not currently a dataset among benchmarks that could represent that case, so perhaps one could inform design decisions.

@JobLeonard, I really like your proposals! If I might riff on them a bit:
Reusing the existing base 62 encoder/decoder makes a lot of sense. I'd've tried it too; the base 36 was meant as an illustrative example, and doing either of your proposals fits much better along with rest of JCOF.
For the first proposal, an equivalent of Number.prototype.toExponential could probably be put together with a bit of logarithms, so it's unlikely to be a problem in other languages; though the algorithm might require some time to figure out, afterwards it can be ported easily and basically only requires the logarithm function and exponentiation, both of which are likely to be supported. It'd also be possible to try to calculate it in a binary exponent form (so using 2 as the exponentiation base), and that might allow for very slightly faster parsing if you have access to the IEEE 754 binary representation, but that might be going in a bit too deep.
I like the sublimity of the second one, given that it's essentially just a compressed form of what JCOF does at the moment :)
Either way, all of these solutions (both the current decimal scheme, naïve base 36, as well as both proposed by Job) suffer to some extent from the base-incompatibility problem if trying to encode numbers like 1/3 without losing accuracy. Of course, then a way to reduce this is to decide a cutoff point of accuracy that'd be acceptable -- after all, 2.2 isn't exactly that in the float representation too, just that stringification/parsing algorithms skip all the insignificant zeroes and don't display decimals after the 17th-or-so position that would arise from the binary value.
That said, just truncating after some n digits can be done just as well with the current decimal encoding, and the gains attained by the rest of these are fairly minimal (or so it seems to me, not having benchmark data of my own), so perhaps the added complexity becomes not really worth it by that point...

A proper float to string algorithm doesn't just choose some number of sufficient digits; the string "2.2" represents exactly the one floating point number which is the closest to being 2.2, and a float parser will return that exact floating point value when parsing "2.2". And the expression 1/3 returns a floating point value; (1/3).toString() returns the shortest string representation where (1/3) is the closest float, so that the float parser returns the exact same float.
My problem is that I only know about these challenges, not how they're solved, or even how difficult they are to solve.
FWIW, I tried out the algorithms in https://observablehq.com/d/e3010313d892b36e, and a lot of numbers aren't round-tripped correctly. For example, the base 10 float 1.1344e-319 stringifies to "f2WY-5d", which parses to 1.12094e-319, and 9.399999999999983 stringifies to "fH3egP4fQc-f" which parses to 9.399999999999984.
Here's a snippet which can be used to test all the floats:
// Return the next representable double from value towards direction
// From https://stackoverflow.com/a/27661477
function nextNearest(value) {
  if (!isFinite(value))
    return value;

  var buffer = new ArrayBuffer(8);
  var f64 = new Float64Array(buffer);
  var u32 = new Uint32Array(buffer);

  f64[0] = value;

  if (value === 0) {
    u32[0] = 1;
    u32[1] = 0;
  } else if ((value > 0) && (value < Infinity) || (value < 0) && (value > Infinity)) {
    if (u32[0]++ === 0xFFFFFFFF)
      u32[1]++;
  } else {
    if (u32[0]-- === 0)
      u32[1]--;
  }

  return f64[0];
}

let f = 0;
while (f != Infinity) {
  let str = stringifyF62(f);
  let parsed = parseF62(str);
  if (parsed == f) {
    console.log("OK:", f, "from string", str);
  } else {
    console.log("ERR: expected", f, "got", parsed, "from string", str);
  }

  f = nextNearest(f);
}

Replace with stringifyF62(f) with f.toString() and parseF62(str) with parseFloat(str), and you can see that the JavaScript float stringify and parse functions round-trip all numbers you try perfectly.

@mortie yeah, after thinking a bit more about the "v1" approach that checks out - I do a multiplication which can lead to rounding errors.
The second version that first uses toString(10) and then converts the two separate strings to b62 integers should roundtrip correctly for all values, since it is a lossless transformation of two integers. I'll update the code to use that approach instead. Gimme a few minutes.

Ok so first, thanks for that test function, that's been a major help in debugging!
There's one major flaw in the implementation though: it allocates new TypedArrays every call :scream:. ArrayBuffer allocation is just about the slowest thing you can do on JS right now (TypedArrays are only fast in usage after allocation). So do yourself a favor and hoist it out for faster testing.
Anyway, that took longer than a few minutes because I bumped into two edge-cases:

For very small or large numbers .toString(10) returns exponential notation, so we need support for that after all
leading zeroes on the fractional part, e.g 1.00456789.

The first point wasn't too much of a pain, since it was basically just adapting the ideas from before but in a way that guarantees that we'll never have longer strings in the end.
The second point took more time. If I naively convert this the leading zeroes get dropped, resulting in 1.456789 after the roundtrip. Bad news.
To keep the output compact my solution was to introduce a second separator to indicate leading zeros: ,. In that case the next character encodes the number of leading zeroes in b62 (so up to 62). So:

1.456789 converts to "f1.1UPz"
1.0456789 converts to "f1,11UPz"
1.00456789 converts to "f1,21UPz"

I don't think it's possible for any double to be converted to a number with a fraction with 62 leading zeroes, and vice versa a handcrafted number string with 62 leading zeroes would not convert to a double with the same value. In that case this I would consider this safe (I'm still letting the tests run in the background to verify this, it's taking forever even with that TypedArray hoisting).
Oh, and I also added a simple check to see if the converted string is longer than a plain .toString(10) conversion. In that case the latter is returned. Note that this does require that on the parsing side we don't just check of the first numbers are /[0-9-]/, we also need to check for optional exponential notation suffixes, so the strings 1e100, 1e+100, 1e-100 must all be recognized. The benefit would be that it's easier to manually insert numbers, which is also a goal, right?

(updated code can be found at the same link as before: https://observablehq.com/d/e3010313d892b36e)

Ok, dammit, found another roundtrip bug:
Error: Expected 0.10000000000000005, got 0.10000000000000003 from string "f.JNBBGsgRe"

The cause is that 0.10000000000000005 is a valid number, but<PHONE_NUMBER>0000005 converts to<PHONE_NUMBER>0000004.
I think that's easy enough to fix though, just have to parse the separate integer, fraction and exponent strings as BigInt values.

Ok so this is as much as I'll write today - I already got nerdsniped on my free Saturday for too long now, although I had fun doing it :).
I'm sure there is still an edge-case I missed, but I think it pretty much works now.
Updated code still at https://observablehq.com/d/e3010313d892b36e.
To summarize, the notation logic is as follows:

if the string starts with 0 to 9 and - it is assumed to be a "plain" number string
otherwise, i and I are integer prefixes followed by a base62 number
f and F are floating point values that may have a fractional and exponential component (so ./, and e in number string notation
. is a separator for an integer and a fractional part, both base62 and the latter without leading zeros
, is a separator for an integer and a fractional part, both base62 and the latter with leading zeros, which are encoded as the first b62 digit of the fraction
e is a separator that indicates a fractional part.

Essentially, f{integer}[.,]{fraction}[e]{exponent} is a reversible transformation of the standard way JavaScript may convert numbers to base-10 strings.
Currently stringifyN62 should always returns the shortest possible number string, and parseN62 assumes the numerical substring has already successfully been extracted from the full string.

... actually, given how I currently coded it we could replace i, I, f and F prefixes with a universal n and N prefix, since the optional ., ,, + and - separators can be used to indicate if we're dealing with a pure integer or with a floating point number.


MacOS Monterey uses port 5000 for Airplay Receiver

Codelab info

Firebase web codelab - step 5
Project setup

Directory: web-start
Operating System: macOS Monterey Version 12.2
Describe the problem

The Airplay Receiver sharing functionality uses port 5000 and is enabled by default. Therefore, the command firebase serve --only hosting does not work, and going to it, the user will receive the HTTP error 403 Forbidden.
Suggested fix
Serve it on a different port. For instance: firebase serve --only hosting --port 5005 and then try the app on http://localhost:5005. You can also disable Airplay Receiver, and it will work as intended.

Thank you!

Thanks for posting a workaround @Drblessing! The Firebase CLI can now detect this issue and automatically pick another port, starting in version 10.7.0 (if you're interested in the specifics of the fix, take a look at https://github.com/firebase/firebase-tools/pull/4415).

Thanks!

Interesting I'm using firebase cli 12.9.1 and it still fails silently.


More Slider Thumb Related Tests

🙋 Feature Request
Please add some more tests. As we improved clicking on the slider and moving the nearest slider to that position. The coverage is good, but they don't cover all the edge cases with multiple thumbs and disabled.
🔦 Context
We created a multiple thumb with disabled slider story and the tests don't cover all the cases found in this story.
🧢 Your Company/Team
RSP

To be added when we pick up Slider again


Update Composer dependencies (2023-06-27-00-12)

Visual regression test passed!
View the visual regression test report


Revert "[embedded] Resolve empty -sdk path warning in embedded stdlib build"

This reverts commit 1b1db053d58de4e694fd1365741def4fc3b18e67.
Looks like there is some build breakage at https://ci.swift.org/job/oss-swift_tools-R_stdlib-RD_test-simulator/.

@swift-ci please smoke test and merge


Linux musl builds?

First off, thanks for the fantastic tool!
It'd be mildly convenient if there were published binaries for musl in addition to the existing glibc one. Would you consider doing so?
Background: we run JVMs (Azul Zulu) out of Docker images based on Alpine (which ships with musl). Currently, when we want to use the profiler, we need to grab the source, install some packages (g++, linux-headers, and make), and compile it. That's all perfectly workable, but it'd be nice if we only needed to install the runtime dependencies (libstdc++ and bash) and use a binary.
In case it's helpful for you, I put together a simple dockerfile to build x86 musl binaries:
FROM alpine:3.9

RUN mkdir -p /usr/local/zulu \
    && wget https://cdn.azul.com/zulu/bin/zulu11.33.15-ca-jdk11.0.4-linux_musl_x64.tar.gz -O - \
    | tar -xz -f - -C /usr/local/zulu

ENV JAVA_HOME=/usr/local/zulu/zulu11.33.15-ca-jdk11.0.4-linux_musl_x64
ENV PATH $PATH:$JAVA_HOME/bin

RUN apk add g++ linux-headers make

RUN mkdir /async-profiler
WORKDIR /async-profiler

COPY Makefile ./
COPY src src/

RUN make

With that in the project dir as, say, alpine.Dockerfile, building with docker build -f alpine.Dockerfile . will produce an image id at the end (or use tags, etc). You can then pull out the build products with:
mkdir x86-musl
for f in jattach libasyncProfiler.so; do
docker run --rm [the image id above] cat /async-profiler/build/$f > x86-musl/$f
done

Thanks!

Yes, I think I can provide musl builds starting from the next release.

Excellent, thanks. Feel free to close this if you like.

I added async-profiler to testing repository of Alpine Linux.

Added musl build in 1.6 Early Access release:
https://github.com/jvm-profiling-tools/async-profiler/releases/tag/v1.6-ea

Just updated our internal tooling to use the 1.6 musl builds; thanks again!
@jchipmunk btw it looks like that alpine build only lists bash as a dependency but AFAICT it also needs libstdc++. Did it work for you with just bash?

@marshallpierce
This shouldn't be needed since abuild tool pulls in build-base automatically which depends on gcc.