VSCode crashing when sshfs mounted drive is removed

Issue Type: Bug
I often use SSHFS to mount remote drives and then use VSCode to edit files on those drives. When I have a file on a remotely mounted drive open in VSCode and I have to unmount the drive (either problem with sshfs, or just no longer on the required VPN), VSCode crashes. During the crash it asks if I would like to relaunch, and I click yes. A screenshot of the message is below. The button does not relaunch VSCode, so I have to manually open the application again. Once I do so (usually after unmounting sudo diskutil umount force ~/mnt and remounting the remote drive), everything returns to it's previous state. This behavior is 100% reproducible on my computer. Additionally, I have check with all extensions disabled and the same thing occurs.

I would expect that unmounting a drive via sshfs while having a file from that drive open in VSCode would not cause VSCode to crash, but rather just complain that file cannot be saved/found/etc. VSCode already gives this warning when I am not on the correct VPN or sshfs is not working correctly, but then as soon as I force the unmount so that I can remount the drive, it crashes in the background.
Thanks for the help, and if there is anything else I can do to help with this please let me know.
VS Code version: Code 1.59.0 (379476f0e13988d90fab105c5c19e7abc8b1dea8, 2021-08-04T23:14:40.191Z)
OS version: Darwin x64 19.6.0
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4980HQ CPU @ 2.80GHz (8 x 2800)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
82, 21, 9

Memory (System)
16.00GB (0.12GB free)

Process Argv
--crash-reporter-id 6181084c-00e1-40ef-aeda-66f2d6f78f3d

Screen Reader
no

VM
0%

Extensions (6)

Extension
Author (truncated)
Version

matlab
Gim
2.0.1

vscode-cudacpp
kri
0.1.1

python
ms-
2021.8.1105858891

vscode-pylance
ms-
2021.8.1

jupyter
ms-
2021.8.1195043623

latex-support
tor
3.9.0

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
pythonvspyt602:30300191
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyt639:30300192
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
pythonvsuse255:30340121
vscod805:30301674
pythonvspyt200:30340761
vscextlangct:30333562
binariesv615:30325510
vsccppwtct:30329789
pythonvssor306:30344512
bridge0708:30335490
pygetstartedc2:30351386
vstre464cf:30350173
bridge0723:30353136

Looks like duplicate of #114283 which was in turn marked as duplicate of #106879
You should provide your use case there if it wasn't handled by latest workaround.

It does seem to be the same general issue as both of those that you mention, but I don't think any real workaround is provided. This comment here seems to imply that a change was pushed to VSCode, but as the comment suggests, it is only a bandaid as it does not work for drives not mounted in /Volumes/. I do not mount my drive to /Volumes/, but rather a different path, so the workaround should not work.
Should I close this issue and post this comment also on issue #106879?

My assumption is yes, as that issue acknowledges that it's still a problem for other paths. Maybe propose other common paths in that issue if you are using standard or common/popular path for network mounts.
However I'm not a VS Code dev so feel free to wait for reaction/comment from devs.

Thank you very much for the help!


Potential Tick Duplication In extract_pool_events_and_positions module

In viewing the extract_pool_events_and_positions function in the filtering.rs file used by the extract_pool_events_and_positions module, it looks like you might end up creating the same tick more than once. This is because of the or logic for creating both ticks. From my understanding, it is possible that only one of the ticks already exists, so you should run separate conditions for each tick to trigger the new tick creation.
The scenario I am thinking of is there is a new pool, and the first two deposits are as follows:

User 1 deposits $100 on tick 10 to 15
User 2 deposits $100 on tick 10 to 20

When the user makes two deposits, tick 10 already exists, but tick 20 does not. So you would only acknowledge that a new tick was created for tick 20.

Maybe something like this?

@abourget @Eduard-Voiculescu

Was this resolved? Can we close this?

@steegecs Do you need any more clarification?


Ride-able Frostmites Beetles are not showing up in ClassicUO like they do in all other clients.

Describe the bug
Ride-able Frostmites Beetles are not showing up in ClassicUO like they do in all other clients.
Client version: <IP_ADDRESS>
To Reproduce
Steps to reproduce the behavior:
Spawn in one of the new ride-able Frostmites from OSI's version <IP_ADDRESS>
Ride it.
ClassicUO Assistant you cannot see it.

Razor Alone or UOSteam:

Expected behavior
It should be visible in ClassicUO's launcher
Screenshots
Above
Shard

Name of the shard: UOAlive or OSI

Information (please complete the following information):

OS: Win 11
ClassicUO Version: Latest
[ NO ] Using DEV Version?
Launcher: ClassicUO Launcher

Additional context

How many clients do you run in parallel: 1
Screen Resolution: [e.g. 1920x1080] 4k

do you have the anim id?

base(name, 0x590, 0x3EDA,)

added 9821a3fc0a7bfc7efc96926743ca8b50caf98e61

So there is an animation now (thank you!) but it is not lined up properly it seems:


Error: watch ENOSPC     at exports._errnoException (util.js:746:11)

➜  test-project git:(master) ✗ gulp serve
[17:58:12] Using gulpfile ~/test-project/gulpfile.js
[17:58:12] Starting 'styles'...
[17:58:12] Starting 'elements'...
[17:58:12] Starting 'images'...
[17:58:12] Finished 'elements' after 286 ms
[17:58:12] styles all files 98 B
[17:58:12] Finished 'styles' after 682 ms
[17:58:12] images all files 35.41 kB
[17:58:12] Finished 'images' after 352 ms
[17:58:12] Starting 'serve'...
[17:58:12] 'serve' errored after 52 ms
[17:58:12] Error: watch ENOSPC
at exports._errnoException (util.js:746:11)
at FSWatcher.start (fs.js:1172:11)
at Object.fs.watch (fs.js:1198:11)

@sajjadmurtaza-ror Use chokidar instead. Gaze is too buggy :)

@sajjadmurtaza-ror Gaze is gulp3 watcher. gulp4 will use chokidar

@sajjadmurtaza-ror However nospc error means, that you don't have a space on your drive.

https://github.com/gulpjs/gulp/issues/217

Resetting the memory fixed the problem for me. Thanks


Images arent transparent

So the images arent transparent, but why is that?
The images not being transparent causes an issue, and that the widget doesnt blend in with the background.
Now obviously, this is an easy peasy fix, but what is the reason for the images not being transparent?

Initially the images' background were transparent, but it made the theme incredibly laggy on Window$. And Tkinter will put a background behind any (transparent or not) image anyway, so I rather removed the transparency.
TLDR: It wouldn't blend in with the background even if the images' background were transparent.


Cannot convert from Map[_, Any]

While trying out the library, I have realized that there are no implicit conversions for Any or AnyVal.
For example, consider the following code:
case class Container(m: Map[String, AnyVal])
val container = Container(Map("name" -> "John".asInstanceOf[AnyVal], "age" -> 50))
container.asJson

This will not work because there is no implicit mapping for a AnyVal (or for Any, for that matter).
I've been trying to understand how one may write their own implicit conversion guidelines in such cases for converting from objects to JSON and vice versa.
The end goal here is something like being able to convert between any arbitrary JSON to a Map[String, Any], and vice versa.
Note: If such a functionality already exists, please point me to the right direction. I was not able to find this in the documentation or otherwise.

@ChetanBhasin circe is built on the idea of having types drive serialization—you ask for a JSON representation of a value of type Map[String, User], for example, and the appropriate encoders are selected at compile-time—there's no cost or danger of reflection at runtime. The other side of this is that circe is entirely helpless when it comes to Any or AnyVal, where you can't do anything with a value apart from reflect on it at runtime.
This is by design, and has two goals: to keep the library simple by avoiding any use of runtime reflection, and to promote the use of types. In many ways Map[String, Any] is the opposite of typesafe functional programming, and circe aims to make it both possible and desirable to avoid having types like Map[String, Any] show up anywhere in your program.
Of course you could write your own Encoder and Decoder instances for types like Map[String, Any], but they'd be unsafe, unidiomatic, and possibly less performant (because of the necessity of runtime reflection), so circe itself will almost definitely never provide anything like that off the shelf.
I'd recommend trying to avoid Map[String, Any] entirely, but if you're stuck with it and you really want to use circe (as opposed to a JSON library that embraces runtime reflection, which is most of them :smile:), you could try to cast and convert your Map[String, Any] values into something more type-full, like e.g. Map[String, Either[Int, String]], which circe will happily encode and decode.

@travisbrown Thanks for the reply.
I can totally understand why we might not want to do conversions between Any and JSON value and why custom Encoders are a bad idea.
However, I think, it makes sense to have a data type which deals only with JSON primitives. For example, while the only way to represent a JSON object of key-value pairs in vanilla Scala would be to use a Map[String, Any] or Map[String, AnyVal], we could have something like Map[String, JsType].
JsType in this regard, can be one of the JSON primitives (i.e., String, Long, Boolean, Object, List, or null). However, it would make sense to be able to implicitly convert between JSON primitives to Scala, while backward can be achieved using a JSON type. Of course, in this case, it would be wonderful to be able to do some basic operations on these values (either by implicit conversion or implementing methods).
This approach would be type-safe. Now, my only question is weather something like already exists in the library?
I do realise that this is something fairly simple to implement, but would be great if we could have it included as a feature.

I'm sorry. I've been fiddling around with the library and understand this better. Will close the issue.

@travisbrown I tested Map[String, Either[Int, String]] but it created a json with the Left and Right keys:
import io.circe._, io.circe.generic.auto._, io.circe.parser._, io.circe.syntax._
import io.circe._
import io.circe.generic.auto._
import io.circe.parser._
import io.circe.syntax._
import cats.data.Xor

type Message = Map[String, Either[Int, String]]
object Message {
  def apply(elems: (String, Either[Int, String])*): Map[String, Either[Int, String]] = Map(elems: _*)
}
val m1 = Message("name" -> Right("John"), "age" -> Left(50))
println(m1.asJson)

// Exiting paste mode, now interpreting.

{
  "name" : {
    "Right" : {
      "b" : "John"
    }
  },
  "age" : {
    "Left" : {
      "a" : 50
    }
  }
}

And when I to convert a json without this keys the decode do not work:
// ...
val json: String = """
  {
    "id": 1,
    "name": "Foo",
    "city": "NYC",
    "age": 10,
    "height": 175
  }
"""
val m2 = decode[Message](json)

// Exiting paste mode, now interpreting.
m2: cats.data.Xor[io.circe.Error,Message] = Left(DecodingFailure(CNil, List(El(DownField(id),true,false))))

Is that the expected behave?

@robsonpeixoto The Either constructors are case classes, so generic will produce instances for them just like any other case classes (there are no automatically-provided Either instances, and it's not special-cased).
If you want an untagged representation, you can do something like this:
import io.circe._, io.circe.generic.auto._, io.circe.jawn._, io.circe.syntax._

implicit val encodeIntOrString: Encoder[Either[Int, String]] =
  Encoder.instance(_.fold(_.asJson, _.asJson))

implicit val decodeIntOrString: Decoder[Either[Int, String]] =
  Decoder[Int].map(Left(_)).or(Decoder[String].map(Right(_)))

And then:
scala> println(m1.asJson)
{
  "name" : "John",
  "age" : 50
}

scala> decode[Message](m1.asJson.noSpaces)
res1: cats.data.Xor[io.circe.Error,Message] = Right(Map(name -> Right(John), age -> Left(50)))

Does that work for you?

Very well, @travisbrown. Thanks


feat: Support for code generation from the command line

I have a scene that I want to use to generate some classes. But I don't need to go through the analyzer's quick-fix in VS code to create these classes. Instead, create these class files with some boilerplate.
This Issue is only used as a flag and discusses whether it is necessary to add a command line to generate. Thanks for providing such a great tool.

@spideythewebhead I'm posting some issues for reference.
https://github.com/odroe/prisma-dart/issues/154
https://github.com/odroe/prisma-dart/issues/166

Hi @medz
Currently the tool is only providing code generation for editable files through suggestions in IDEs (using the analysis server & analyzer plugin).
For your case you would need to generate the boilerplate for your code and run the data_class_plugin generator for all the rest.

Unfortunately the tool doesn't support generating the dart code on the above example. It only generates based on the code you provide


Flaky Smoke test failure: Cannot find module '...\app\product.json'

Build: https://dev.azure.com/monacotools/a6d41577-0fa3-498e-af22-257312ff0545/_build/results?buildId=248427
Changes: https://github.com/Microsoft/vscode/compare/57c5c88...2d82689

/duplicate https://github.com/microsoft/vscode/issues/200348

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Support 100million - 1 billion small files in AI/ML training

Is your feature request related to a problem? Please describe.
Training against Alluxio has good performance under 100 million small files.
When the training dataset reaches 100 million to 1 billion small files, training performance is largely impacted, especially when the training job does global data shuffle between epochs and multiple nodes are involved
The impacted training performance comes from that Alluxio Fuse client may not be able to store all the cached metadata in process memory. All local metadata cache is invalidated between epochs. Alluxio master needs to serve global metadata requests during each epoch.
Describe the solution you'd like

Improve the ability for Alluxio master to serve metadata requests. One approach is enabling standby masters to serve metadata read requests.
Improve the ability of the Alluxio client to cache metadata locally.

see how much file metadata can be cached locally, how much RAM is needed.
If RAM is not enough, support client-side metadata cache using ROCKS?
support pinning client-side metadata? if local metadata can only cache 40% total metadata, 40% always serves locally and 60% always serves remotely. Not all locally cached metadata will be useless between shuffled epochs.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Urgency
Explain why the feature is important
Additional context
Add any other context or screenshots about the feature request here.

@yyongycy @yuzhu @maobaolong @ssz1997 @apc999
Create a github issue for tracking the ideas of how to support 100 million or 1 billion small files.
Please share your suggestions. We will have discussions in the future

A few heuristic questions:

Are all files modified/created? How many static files(only file atime change)?
Can one Master support that many metadata in performing way?
Are all cached metadata not usable in next epoch?
Is the hardware config(cpu/memory/network) matching the data scale that is managing?

How to access files where goosefs does not exist but UFS exists？【follower-readOnly】


SURN tag does not allow comma separated surname list.

GEDCOM v7.0.9 does not indicate that a comma separated list of surnames is allowed in the SURN tag just like the same tag under GEDCOM v5.5.1 which was defined as:
NAME_PIECE_SURNAME:=
[ <NAME_PIECE> | <NAME_PIECE_SURNAME>, <NAME_PIECE> ]
Surname or family name. Different surnames are separated by a comma.
Individuals with multiple surnames for example:
1 NAME Juan /Hernandez Martinez/
2 GIVN Juan
2 SURN Hernandez, Martinez
The SURN tag should have a comma between surnames for indexing purposes AND maintain consistency and backward compatibility with GEDCOM v5.5.1

This was an intentional change and listed as an ambiguity correction in the 7.0.0 changelog. It was one of several ambiguities that were identified caused not by a poorly-written spec but by prevalent use that did not align with the spec. In particular, we identified many applications that did not respect the comma and some researchers who incorrectly typed commas as a substitute for letters or diacritics they did not know how to type properly leading to enough 5.5.1-invalid data in the wild that some careful application developers were choosing to violate the 5.5.1 spec to conform to common practice instead.
In version 7.0, the mechanism provided to indicate that there are several name parts with the same type is using multiple substructures of that type, as
1 NAME Juan /Hernandez Martinez/
2 GIVN Juan
2 SURN Hernandez
2 SURN Martinez

A spec-conforming 5.5.1 / 7.0 converter could switch between comma-separated payloads in 5.5.1 and multiple structures in 7.0, though again as noted above a file claiming to be 5.5.1 may have violated the 5.5.1 spec in this regard.

Discussion12 JUL 2022:
Add entry to the migration FAQ on gedcom.io with this answer

As of GEDCOM v7.0.8 the specification does not provide for multiple SURN tags specifically stating {0:1} for this tag.
Therefore the above solution and proposed changes to the converter would not generate valid v7.0 GEDCOM.  Without a comma separated list of specific surnames, GEDCOM v7.0 currently provides no way for indicating a difference between individuals with two or more family surnames vs an individual with a single multipart family surname.

We have a new ambiguity in GEDCOM 7.0 as we allow space separated payload in the NAME_PIECES, too. The example above may be exported in the way it was exported in 5.5.1 by most applications without modification:
1 NAME Juan /Hernandez Martinez/
2 GIVN Juan
2 SURN Hernandez Martinez
`

The primary issue not totally resolved with this "fix" is that GEDCOM does not differentiate between surname customs where 1) two or more family surnames are used to identify an individual, and 2) where an individual has a two part (but singular) surname.  Not all individuals follow the custom to combine two names in to a surname by placing a dash/hyphen between the two parts.
A singular surname of "Johnson-Allen" may be customary in one region, where Johnson Allen may be a surname customary in another region but they are not considered a multiple surname custom such as Hernandez Martinez.
My take would be that GEDCOM should do the following:
Single Surname Custom:
1 NAME John /Johnson-Allen/
2 GIVN John
2 SURN Johnson-Allen
-or-
1 NAME John /Johnson Allen/
2 GIVN John
2 SURN Johnson Allen
Multiple Surname Custom:
1 NAME Juan /Hernandez/ /Martinez/
2 GIVN Juan
2 SURN Hernandez
2 SURN Martinez
I realize that the double sets of "/" was shot down earlier but is valuable in these situations.  It is very important for genealogy software that manages statistical information and large numbers of family groups to identify the family group each individual participates.  For example the Hernandez family (or the Martinez family) would want to know the number of members in that family group.  If the SURN tags were not separated, statistics would be lost and inaccurate for one or both of the family groups.

You cannot have distinct surnames separated by spaces inside a single /.../ element.
There are many surnames that include punctuation characters (e.g. hyphen or apostrophe), but also ones contains a space. In principle, this should be
a non-breaking space but that would be difficult to enforce and so ordinary spaces must be acceptable.
Examples Include:
/St John/, or /St. John/ (see St John (name) https://en.wikipedia.org/wiki/St_John_(name)), pronounced /Sinjin/or /Sinju//n.
/and Irish names including name particles, such as
/Ó Dónaill/ or /Ó Conchobhair/
Tony
On 16/07/2022 21:15, Norwegian-Sardines wrote:

The primary issue not totally resolved with this "fix" is that GEDCOM does not differentiate between surname customs where 1) two or more family
surnames are used to identify an individual, and 2) where an individual has a two part (but singular) surname. Not all individuals follow the custom
to combine two names in to a surname by placing a dash/hyphen between the two parts.
A singular surname of "Johnson-Allen" may be customary in one region, where Johnson Allen may be a surname customary in another region but they are
not considered a multiple surname custom such as Hernandez Martinez.
My take would be that GEDCOM should do the following:
Single Surname Custom:
1 NAME John /Johnson-Allen/
2 GIVN John
2 SURN Johnson-Allen
-or-
1 NAME John /Johnson Allen/
2 GIVN John
2 SURN Johnson Allen
Multiple Surname Custom:
1 NAME Juan /Hernandez/ /Martinez/
2 GIVN Juan
2 SURN Hernandez
2 SURN Martinez
I realize that the double sets of "/" was shot down earlier but is valuable in these situations. It is very important for genealogy software that
manages statistical information and large numbers of family groups to identify the family group each individual participates. For example the
Hernandez family (or the Martinez family) would want to know the number of members in that family group. If the SURN tags were not separated,
statistics would be lost and inaccurate for one or both of the family groups.
—
Reply to this email directly, view it on GitHub https://github.com/FamilySearch/GEDCOM/issues/169#issuecomment-1186282282, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACDJB3TNXE3AE2FNY7XSPS3VUMJ6HANCNFSM523XKQFQ.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

You cannot have distinct surnames separated by spaces inside a single /.../ element. There are many surnames that include punctuation characters (e.g. hyphen or apostrophe), but also ones contains a space. In principle, this should be a non-breaking space but that would be difficult to enforce and so ordinary spaces must be acceptable. Examples Include: /St John/, or /St. John/ (see St John (name) https://en.wikipedia.org/wiki/St_John_(name)), pronounced /Sinjin/or /Sinju//n. /and Irish names including name particles, such as /Ó Dónaill/ or /Ó Conchobhair/ Tony
…

Is your assertion that a NAME tag must therefore allow for multiple sets of “/“ when an individual has two distinct surname parts? As in my example:
Multiple Surname Custom:
1 NAME Juan /Hernandez/ /Martinez/
2 GIVN Juan
2 SURN Hernandez
2 SURN Martinez

You cannot have distinct surnames separated by spaces inside a single /.../ element.

This is not quite correct. The PersonalName datatype cannot tell if the slashes delimit one or several surname-like components, but with the addition of one or several SURN that distinction can be added.
Additionally, it is worth noting that SURN does not require that its payload be part of the slash-delimited part of the PersonalName value.
Hence, as I read the spec I believe that all three of the following are permitted:
0 @I1@ INDI
1 NAME Juan /Hernandez Martinez/
2 SURN Hernandez
2 SURN Martinez

0 @I2@ INDI
1 NAME Martin /St John/
2 SURN St John

0 @I3@ INDI
1 NAME Joaquin /Perez/ Quinones
2 SURN Perez
2 SURN Quinones

Based on the above from tychonievich, would this also be valid?

0 @I3@ INDI
1 NAME Joaquin Perez Quinones
2 SURN Perez
2 SURN Quinones

Would this also be valid?

0 @I3@ INDI
1 NAME Joaquin Perez Quinones
2 SURN Perez
2 SURN Quinones

I don't think so. The PersonalName datatype says the slashes are

used to delimit the portion of the name that most closely matches the concept of a surname, family name, or the like.

If there is something close enough to the concept of a surname to deserve the SURN tag, I read the above as implying it should also use slashes.
The reason I had an example with more SURN than the slashes cover is based on how names are given in many Spanish-speaking societies: people are given two surnames, where the first is more significant. Joaquin in this example might have been called "Mr. Perez" or "Mr. Perez Quinones" but never "Mr. Quinones", and will give his children the surname Perez but not Quinones. Because of that I could see arguments both ways on how to put slashes about the part that "most closely matches" a family name: either Joanquin /Perez/ Quinones or Joaquin /Perez Quinones/.

Discussion 16 AUG 2022: committee discussion was that the farm/house name should not be put into the SURN payload since it is not a surname or family name.   It may be a gap in the standard tags, but could have a TYPE for farm name, and/or an extension tag for the farm name, like we discussed having an extension tag for _RUFNAME for the rufname purpose.   A similar issue has arisen around royalty, where that would be in the TITL tag on an individual.
The following are in use today (correctly or incorrectly):
1 NAME John /England/
1 TITL King

1 NAME John /England/
2 SURN England
1 TITL King

1 NAME John
1 TITL King of England

The last one would be the most correct.
One could have
1 NAME Ole Jansen
2 GIVN Ole
2 _FARMNAME Bruflott

I already model it as a PLAC, but the intention of associating “Bruflott” as part of a name is that:
a) family units both residing at the farm and residing at a different farm still associate themselves with the Bruflott farm name as a family (not as part of their public identity),
b) after a law was put in place in 1924 in Norway (and at other times in Europe) some people (not all) took the farm name as their required surname.
This association is valuable for outlining a genealogy for a family group over the 300 to 400 years of their existence.  The name “Bruflott” was not a surname in 1701, but could have been used as a surname in 2001.
This has some of the same aspects of family grouping like the  “House of Tutor”, “House of Windsor” where individuals did not really have a surname but associated themselves with other family members by saying they belonged to “The House Of”.
I would like to have entered:
1 NAME John
2 SURN Windsor
1 TITL King

Or, but not my favorite
1 NAME John
1 NATI Windsor
2 TYPE House
1 TITL King

The last being correct for the committee, but not supported in family lists, charting, or any other grouping mechanisms by any of the major or minor genealogy software, so it becomes useless in practice!

Discussed by steering committee

By merging #187 we can now use the NATI solution given in the last comment
We know that the current NAME structure does not capture every use case, but cannot change it significantly in 7.x because doing so would be backwards-incompatible. We anticipate introducing a much more flexible name structure in 8.0

We are closing this issue, but if additional problems come up or if there is a recommendation for a new name part type to introduce in 7.1, please re-open the issue (or file another one)

What is the time frame for v7.1 recommendations regarding new name parts?
Obviously, a person’s identity begins with their name and I would include identity items like.
Rufname, Patronymic, Farm/Location, Clan, House, Tribe to name a few.
Rufname has been discussed here by the German delegation,
Patronymic is common in many cultures and still used in Iceland and probably other.
Farm/Location is also historically common.
Clan, House, Tribe are similar and used in European and Asian (probably all cultures) and represent individuals that participate in a grouping that is part of their name and identity.
An example of both Patronymic and Location in one famous name is; Leonardo di ser Piero da Vinci  (aka Leonardo da Vinci).  Leonardo is his given name, di see Piero is a Patronymic reference to his father Ser Antonio Piero, and da Vinci is an indicator of birthplace.
Korea has approximately 288 family names and almost 50% of individuals have one of 3 family names.  To differentiate between unrelated individuals with the same family names they use a Korean clan village name (called  Jipseongchon) to differentiate origin of the family name by village.  This has some parallels in ancient times for use when doing census based on town of birth.

@Norwegian-Sardines  asked

What is the time frame for v7.1 recommendations regarding new name parts?

While not set in stone, our current working estimate is that we'll have an open meeting associated with RootsTech 2023 to get input from voices that might not be active on github, then use that input as well as the various next-minor tagged issues here to draft 7.1. I expect that will take us a few months, and then I expect a few months of a comments period prior to release; if everyone is happy with what we draft 7.1 might be released around this time next year.
Things could go faster if we decide to draft 7.1 prior to the RootsTech meeting, or slower if we run into difficulty writing it or disagreement with what we write.

The use cases identified by @fisharebest and the additional parts requested by @Norwegian-Sardines appear to be largely independent.

Display -- I think this is covered by the current name payload.

Sort -- partially covered by the / in the name payload, but not in all cases; for example, it will not help sort by root name instead of name form.
Proposal: add INDI.NAME.SORT_AS in 7.1

Group -- I'm not familiar with this use case. Is it enough to identify which name part types are "grouping" (e.g. SURN, CLAN, HOUSE, TRIBE) and which are not (e.g. GIVN, PATRONYMIC,PLACE)?

Search folding -- I think this might be subsumed by SORT_AS?

Many more name parts
GEDCOM-X has a solution to this that we might be able to copy. They have just four or five name types (prefix, suffix, given, surname; implicitly also a "none of the above" as these four are all optional) but a large and extensible set of name qualifiers.
Could we do something like that ourselves? Instead of adding name part tags, add just one more (OTHER or PART or something like that) and give all name parts a {0:M} enumeration value substructure with all of GEDCOM-X's name part qualifiers as permitted values?

"Invisible" names -- some name parts are not included in the written or spoken form of the name, and which ones varies by person not by name part type. See #169 for example place names that are (da Vinci) and are not (Bruflott) written in the name.
Proposal: add INDI.NAME.(part).HIDDEN Y substructure, or a HIDDEN qualifier if we decide to add qualifiers per the previous item.

Sort -- partially covered by the / in the name payload, but not in all cases; for example, it will not help sort by root name instead of name form.
Proposal: add INDI.NAME.SORT_AS in 7.1

I will observe that that for dates, GEDCOM introduced a parallel structure (SDATE) rather than a substructure like DATE.SORT_AS.  As such, it would seem inconsistent for names to use a substructure instead of a parallel structure for indicating a sort form.
I think this discussion should move to issue #190 rather than here though.

That's right!
I am also strongly in favour of sticking to the original V7 cardinality for GIVN and SURN so that specific properties (e.g. call-name) can be applied
to each instance. NB: a call-name is not specific to German, and even in US, UK, and Ireland, the preferred given-name is not necessarily the first.
Tony
On 17/07/2022 15:09, Norwegian-Sardines wrote:

You cannot have distinct surnames separated by spaces inside a single /.../ element. There are many surnames that include punctuation characters
(e.g. hyphen or apostrophe), but also ones contains a space. In principle, this should be a non-breaking space but that would be difficult to
enforce and so ordinary spaces must be acceptable. Examples Include: /St John/, or /St. John/ (see St John (name)
https://en.wikipedia.org/wiki/St_John_(name) <https://en.wikipedia.org/wiki/St_John_(name)>), pronounced /Sinjin/or /Sinju//n. /and Irish names
including name particles, such as /Ó Dónaill/ or /Ó Conchobhair/ Tony
… <#>

Is your assertion that a NAME tag must therefore allow for multiple sets of “/“ when an individual has two distinct surname parts? As in my example:
Multiple Surname Custom:
1 NAME Juan /Hernandez/ /Martinez/
2 GIVN Juan
2 SURN Hernandez
2 SURN Martinez
—
Reply to this email directly, view it on GitHub https://github.com/FamilySearch/GEDCOM/issues/169#issuecomment-1186529775, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACDJB3USEKQLEYCY6RFYC63VUQHYBANCNFSM523XKQFQ.
You are receiving this because you commented.Message ID: @.***>


How to exclude crates not part of the root crate from analysis

I have a project that includes a git submodule for .proto files.
Inside of the submodule there is an example rust crate that has nothing to do with the main repository I am running cargo machete on.
How can I exclude this unrelated crate from the cargo machete analysis?
I can explicitly pass the paths to the projects I want to include, but I just don't like having to do so.
I see multiple possible implementations:

only analyze crates that are members of the workspace if it exists
allow adding paths or directories to ignore in the file system discovery

Thanks for opening an issue! For background: the --ignore-target option was introduced as a minimal way to ignore directories, so instead we could just use a plain list of directories that we want to ignore, instead, as a generalization of the --ignore-target. That is your option 2, if I understand correctly! Is this something you'd be interested in implementing too?


Add TCGC: TCGCoin

[o] I understand that token listing is not required to use the Polygon Wallet with a token.

Please provide the following information for your token.
**Token Name:
Token Name: TCGCoin
Token Symbol: TCGC
Decimals: 18
Polygon Address: 0x44ACD96620B708162af4A90524F29A6839675533
Ethereum Address: 0x2FFdE077455f81E28bAa675a46B9c085740216d4
Logo URI (svg):https://tcgverse-assets.s3.ap-northeast-1.amazonaws.com/tcgcoin_logo.svg
Project Name:TCGCoin
Project Summary:TCGCoin is a token issued by BOBG ltd. A basic token that can be used with services on CryptoSpells and TCGVerse.
Project<EMAIL_ADDRESS>Project Website:https://tcgverse.gitbook.io/tcgverse/tcgc/tcgcoin

Added token to AllTokens v2.5.0.


Local tiles

Uses local tileserver-gl-light server to publish a tile end point.
Tile server is installed on RaspberryPi using tileserver/setup-tileserver.sh script.
Tile server is launched at startup, using an mbtiles file on the Pi. Edit mbtiles file in startup.sh
tileserver-gl-light tileserver/2017-07-03_us_hawaii.mbtiles &
URL for locally hosted tile endpoint is hardwired into map.js tile source:
app.ui.map = new mapboxgl.Map({
	container: 'map',
	style: "http://<IP_ADDRESS>:8080/styles/basic-preview/style.json",
	hash: true
});

Tileserver-gl-light is preinstalled, along with the sample mbtiles file on the latest SD image build here: https://curblr-www.s3.amazonaws.com/wheel/images/curbwheel_image_r2.img.gz

added simulator support for the tile server. just run the setup command:
sh setup-tileserver-simulator.sh
and then npm run simulator will automatically start the tile server with the mbtiles files in the test directory

App has now fully incorporated the map with new local tile endpoints

one failing test with a timeout, but good to merge for now. will pursue that in subsequent ticket 🚢


[3.2.4 beta 4] new pixel snap jitters with odd sized sprites

Godot version:
3.2.4 beta 4
OS/device including version:
windows 10, geforce 1080
Issue description:
the new pixel snap jitters with odd numbered sized sprites (ex. 31x31) with a smooth camera
3.2.3 stable:
https://user-images.githubusercontent.com/55656526/103137899-e80bd500-46d5-11eb-94d7-9284c285c842.mp4
3.2.4 beta 4 (only 64x64 and 32x32 works properly)
https://user-images.githubusercontent.com/55656526/103137902-f0641000-46d5-11eb-8000-2a35e7510cc1.mp4
Steps to reproduce:
(3.2.4 beta 4) new project, enable pixel snap, stretch mode to viewport, add pixel art sprite with odd size (filter off), add smooth camera and move it in game.
side notes:
i added pixel perfect script to the main scene.
the new 'transform snap' does not solve my issue.
Minimal reproduction project:
pixel snap jitter Minimal reproduction project.zip

I just tested and it seems to be the extra changes in #43554 (not the m6502 change).
Following on from reduz PR #43194, some more of the snapping is now controlled by the transform snapping project setting, rather than the pixel snapping project setting. I'm not quite sure on the reasoning for this (as this is just a straight backport of his, aside from the canvas (camera) snapping).
We could consider keeping control of these with the pixel snap option in 3.2, to keep compatibility. Or have another option to control this snapping, but it would be better to avoid too many settings to prevent things becoming overcomplicated. Hopefully I can find out from reduz the reasoning for moving these settings and the best compromise.

with 'transform snap' on, walking with my character at a certain speed makes its head jitter (not visible in video)

with 'transform snap' on, walking with my character at a certain speed makes its head jitter (not visible in video)

Ah this may be because of the camera not being snapped. I will look again at this.
The problem is that currently snapping the camera in reduz method causes another bug to exhibit, which results in a subtle jitter. This can be prevented by doing the camera snapping manually. But this may be too much for many users to understand. Maybe just forcing the camera snap will get someone interested to fix the other bug.

Fixed by #44690.

Fixed by #44690.


Y-axis scale error

In some cases, the y-axis scale is not displayed correctly in terms of the numbers shown and the layout.
To Reproduce

https://int.visualize.admin.ch/de/v/a5Jjfyd4-hAy
compare values in mouse-over with values indicated on the scale.

Expected behavior
Like on PROD
open https://www.visualize.admin.ch/de/v/4OFt2R6jxnyr

Environment (please complete the following information):

Visualize INT : v3.22.5 ([615675d]
edge

and the layout Problem : the left side is cutoff.

Hi @ortnever, thanks for reporting the bug 🙇
After the initial investigation, I believe the numbers are shown correctly, but the Y axis tick values are cut off, which leads to the impression they're wrong – I'll fix the layout problem and hopefully it will be fine then 🤞

exactly...I discovered the layout problem later ;-) !

Hi Bartosz, I did a test on test.visualize and everything was fine. But not on INT. Is trhe last release installed?
Could check the problem with y-Axis. I tried with line chart. Could it be the reason?
https://int.visualize.admin.ch/de/v/fk-ItSBbitF1


Validate repository label names.

Fix #112

Fails

:no_entry_sign:
Unknown label blocked

  Generated by :no_entry_sign: dangerJS against 3643c042cef4758f5befbab12d99eb862f2cc23c

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

2.0.0
GitHub release

Your semantic-release bot :package::rocket:


Rsync disappear when rebooting the boot2docker VM.

Hi there,
We have been notified of a bug since the last version you released,
In order to get docker-osx-dev working we have to do the following :
boot2docker start --vbox-share=disable
boot2docker ssh
tce (then s, rsync i)
Then it works like a charm.

I'm not sure I follow. Were you getting some sort of error? What is "tce (then s, rsync i)"?

We have to install rsync with tce directly in the VM each time we boot it.
The rsync install is not persisted to the boot2docker machine.

It's not persisted after what? After a reboot?

Yes, after a reboot.

Heh, I guess I haven't rebooted my computer in a long time and didn't even think about it :) Thanks for reporting.
Probably the right solution is to move the boot2docker init logic from setup.sh to the docker-osx-dev script.

I will try to do it, do you want it to be really moved ?
I would suggest to move the removing of the vbox shared folders too.

Sure, that would be great. Probably the right fix is to move the init_boot2docker function in setup.sh, plus the functions it calls, into the docker-osx-dev script.

PR send :-)

Confirmed the bug and would love to hear the PR getting merged soon :-) BTW, @brikis98, this tool does work like charm, thank you for all your effort put into this.

@tuanmh: Great to hear the script is working for you. I'm just waiting on a response from @Simperfit in #36, and then it'll be ready to merge.

I believe #36, which is now merged in, fixes this issue.


Update 2022-11-01.md

链接就是 gh 仓库下面的子文件夹，已经附在后面了
报告是按照 0928 的报告改的

From: xijing @.>
Sent: Monday, October 31, 2022 3:12:08 PM
To: isrc-cas/tarsier-oerv @.>
Cc: eval Nya @.>; Author @.>
Subject: Re: [isrc-cas/tarsier-oerv] Update 2022-11-01.md (PR #451)
@xijing21 commented on this pull request.

In biweekly/2022-11-01.mdhttps://github.com/isrc-cas/tarsier-oerv/pull/451#discussion_r1009089419:

@@ -241,6 +241,14 @@ OBS提交：

https://build.tarsier-infra.com/request/show/465
https://build.tarsier-infra.com/request/show/466
+#### ptr_0f_invalid

+撰写 Eulaceura 22H1 的测试报告。
+- kiran 桌面环境
将链接附上更好
―
Reply to this email directly, view it on GitHubhttps://github.com/isrc-cas/tarsier-oerv/pull/451#pullrequestreview-1161412092, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOK426OI6YTYN6NULZ3EBNLWF5WMRANCNFSM6AAAAAARSVIEIA.
You are receiving this because you authored the thread.Message ID: @.***>


[release-1.23] Make new stat regex backwards compatible

Manual cherrypick required.
#52031 failed to apply on top of branch "release-1.23":
Applying: Make new stat regex backwards compatible
Applying: Small optimize
Using index info to reconstruct a base tree...
M	pilot/pkg/networking/core/route/route.go
M	pilot/pkg/networking/util/util.go
Falling back to patching base and 3-way merge...
Removing pkg/bootstrap/testdata/default122_golden.json
Removing pkg/bootstrap/testdata/default122.proxycfg
Auto-merging pilot/pkg/networking/util/util.go
CONFLICT (content): Merge conflict in pilot/pkg/networking/util/util.go
Auto-merging pilot/pkg/networking/core/route/route.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0002 Small optimize
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

fixed in https://github.com/istio/istio/pull/52167


Unspent count is confusing

Better name might be "unfolded count", also the help is too advanced and confusing.

Possible change of text:

You get Mining rewards every 60 seconds. And that is like getting a check. So your balance is comprised of hundreds of thousands of checks. So we are now writing technology called folding, that cashes all the checks so your total balance is spendable in 1 transaction if you wanted. But right now you have the equivalent of 203255 “checks” that need to be folded. In the next 2 weeks that will be fixed

fixed in 24e52ce6f4566f794371e71d972b65e46bf42173


feat: Add survey

Fixes #

Thank you for opening this PR! Please answer our quick survey:

How confident are you in this change? (React with 👍 for high, 👎 for low)
Does this PR require extensive testing? (React with 👀 for yes, 🚀 for no)

Your feedback helps us improve our process. Thanks!


How to execute post install composer scripts

Hi.
I need to execute Incenteev\\ParameterHandler\\ScriptHandler::buildParameters interactively. Is this possible?
My deploy config is:
---
- name: Deploy project to S6 production server
  hosts: s6
  roles:
    - { role: servergrove.symfony2 }
  vars:
    # custom config
    vhost_name: "<HIDDEN_SECRET>"
    vhost_path: "<HIDDEN_SECRET>"
    # mandatory role config
    symfony_project_root: "{{ vhost_path }}/{{ vhost_name }}"
    symfony_project_composer_path: "{{ vhost_path }}/{{ vhost_name }}/shared/composer.phar"
    symfony_project_repo: "<HIDDEN_SECRET>"
    symfony_project_env: "prod"
    # optional role config
    symfony_project_branch: "master"
    symfony_project_keep_releases: 3
    symfony_project_composer_opts: "--no-dev --optimize-autoloader"
    symfony_project_shared_folders:
      - { name: logs, src: app/logs, path: app/logs }

You mean interactively like in "need a user input"?
Could you explain a bit more specific?
What you can do is add a "hook" file in your config which will be included and executed at a specific point of the deployment, but no interactivity there.

Thanks @maschmann
Maybe, it would be interesting how to execute a 'composer install' example in hooks sections.

@davidromani You meant like that?
This is already included in the flow :-)

yes @maschmann, but my problem is how to set parameters.yml interactively in my deploy server with Incenteev\ParameterHandler\ScriptHandler::buildParameters script after composer install execution

This sounds solveable @davidromani ...
Consider something like this:
ansible-playbook --limit $ENVIRONMENT -i ${CURRENT_PATH}/${CLIENT} --extra-vars "composer_option=$my_option_value" ${CURRENT_PATH}/${CLIENT}.yml $VERBOSITY
And then build your hook like this:
- name: do composer stuff
  shell: cd {{symfony_current_release_dir}} && export SYMFONY_ENV={{symfony_project_env}}; {{symfony_project_php_path}} {{symfony_project_php_options}} {{symfony_project_composer_path}} install {{composer_option}}

IMHO this is the most interactive thing you can get, if you pack it in the extra vars of the deployment call.
In my case, I've built a bash wrapper script for the ansible call, for our devs to have a more streamlined command and less options to add. That's where you could place some logic, too.


Added Hover Effect To The Buttons!

This is a test PR, no need to merge it!

We regret to inform you that we are closing the pull request you submitted as it does not include an issue number and appears to be a case of spam. Kindly ensure to check our guidelines and procedures for creating pull requests in the future.
Thank you for your understanding.


外部调起Files略显迟缓

What feature or improvement do you think would benefit Files?
files已设置接管文件管理，比如浏览器下载文件后点击打开，会触发打开files软件，此过程等待时间略长略显迟缓。
Requirements
在files内部右键 在新标签中打开 则是相对更快的。帮忙看看还有没有优化空间
Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.4541
Comments
No response

Thanks for the feedback, Are you talking about Files opening or making a new tab?

What feature or improvement do you think would benefit Files?
files已设置接管文件管理，比如浏览器下载文件后点击打开，会触发打开files软件，此过程等待时间略长略显迟缓。

“Files has been set to be the default file explorer. For example, clicking to open a downloaded file in an internet browser will trigger to open Files, but the opening process seems to be slow and unresponsive.”

Requirements
在files内部右键 在新标签中打开 则是相对更快的。帮忙看看还有没有优化空间

“In contrast, opening a new tab in an existing Files window is relatively faster. Is there any space for Optimizations?”
Just a translation. It would be difficult to optimize this process. However, turn on the Leave app running in the background option in settings may help a little if haven’t already.

Thank you for your feedback! We're continuously working on performance improvements. You can follow our progress in issue #4180.


Proposal: Enhance select field

The current select field has a single line input for choices. Which can be hard to add or update the values especially for different (RTL) languages as Arabic.
That could be enhanced by adding an option field to set choices by line instead of comma separated, here's a working prototype I have made:

As I have read that a complete refactoring is in plan. So, if this minor enhancement is acceptable at the moment, I would make a PR.
Thank you.
UPDATE: Making changes to MultipleValueInput will affect other fields components: File, Email and URL. I have refactored them to support the new approach.
File (Single)

File (Multiple)

Email + URL

Btw, it will not have any breaking changes.

@ganigeorgiev
Please consider this request as it is a struggle to set values for mixed languages: English and Arabic (LTR, RTL)

Also setting values with commas such as addresses will split the address into values.
I will be happy to make a PR if it will be an accepted enhancement.
Thank you.

That would be nice; I am also willing to contribute on full RTL Support for pocketbase


Moving forward with C++17

Hi all!
SPIRV-Tools is building in cpp11 since the codeplay code drop. Is there a good reason for us not to move up to a more recent standard, like cpp17, and follow LLVM's example?
(note: bumping standard from 11 to 17 doesn't cause any compilation issues at HEAD).

SPIRV-Tools moved to c++17, closing this.


Truncated text post

Posts with More link are parsed only with text before this link.

@truephp can you post an example post ? and the script you are using to get it ?

@truephp can you post an example post ? and the script you are using to get it ?

I've created a page for test, try this code

from facebook_scraper import get_posts
from facebook_scraper import exceptions
from facebook_scraper import enable_logging, write_posts_to_csv, get_profile
import simplejson as json
import os
import logging

enable_logging()
location = os.path.dirname(__file__)

try:
    #posts = get_posts(61567187803636, pages=1, cookies=os.path.join(location, "cookies.txt"), options={"posts_per_page": 5})
    posts = get_posts(61567187803636, start_url="https://mbasic.facebook.com/61567187803636?v=timeline", pages=1, cookies=os.path.join(location, "cookies.txt"), options={"posts_per_page": 10})
    json_object = json.dumps(posts, iterable_as_array=True, default=str)

    posts_file = os.path.join(location, 'poststext.json')
    with open(posts_file, "w") as outfile:
        outfile.write(json_object)
except exceptions.TemporarilyBanned:
    print("Temporarily banned")
except exceptions.AccountDisabled:
    print("Account Has Been Disabled")
except exceptions.InvalidCookies:
    print("Cookies are not valid")
except Exception as e:
    print(e.args)

This script only parse the next text
Once upon a time in a quiet, misty village nestled between ancient woods and a winding river, there was a curious girl named Maren. She loved wandering through the fields and forest paths, often straying far from the village. She sought the unknown, the hidden, the stories no one else could tell.\n\nOne late autumn day, as the sun dipped beneath the horizon and a soft orange glow filled the sky, Maren stumbled upon a peculiar sight. A fox, sleek and silver, with piercing eyes like moonlight, was...

but the original text is
Once upon a time in a quiet, misty village nestled between ancient woods and a winding river, there was a curious girl named Maren. She loved wandering through the fields and forest paths, often straying far from the village. She sought the unknown, the hidden, the stories no one else could tell.

One late autumn day, as the sun dipped beneath the horizon and a soft orange glow filled the sky, Maren stumbled upon a peculiar sight. A fox, sleek and silver, with piercing eyes like moonlight, was sitting by an old, twisted tree. It didn’t flee at her approach. Instead, it looked at her as if it had been waiting.

“Hello,” Maren whispered, unsure if the fox could understand. But to her amazement, the fox tilted its head and spoke in a voice as soft as rustling leaves.

“I’ve been watching you, Maren. You walk these lands with a heart full of wonder, and it is rare to find such hearts. Will you listen to a tale that most would never believe?”

Maren, stunned but intrigued, nodded. She sat on the mossy ground, her eyes wide with anticipation.

The fox began, “Once, long ago, this land was home to more than just humans. Spirits roamed the forests, their laughter echoing through the trees, their songs as wild as the rivers. But over time, they faded, forgotten by those who couldn’t see or hear. Yet a few of us remain, carrying memories and whispers of that forgotten world.”

As the fox spoke, Maren felt as if the very air was alive with magic, as if the forest itself held its breath. She listened until the sky turned dark, absorbing tales of ancient spirits, hidden paths, and lost realms. When the fox’s tale ended, it rose and gazed at her kindly.

“You are part of these stories now,” it said. “If you listen closely, you may hear the voices that few remember. Keep your heart open, and you’ll find more than you ever imagined.”

With that, the fox vanished into the shadows, leaving Maren alone but forever changed. She returned to the village, feeling as if the whole world had grown richer, more mysterious, and brimming with secrets that only a rare few would ever know. And as she grew older, she often returned to that tree, hoping to see her fox friend again, always listening for the stories hidden within the forest's quiet whispers.

@truephp can you post an example post ? and the script you are using to get it ?

I've created a page for test, try this code

from facebook_scraper import get_posts
from facebook_scraper import exceptions
from facebook_scraper import enable_logging, write_posts_to_csv, get_profile
import simplejson as json
import os
import logging

enable_logging()
location = os.path.dirname(__file__)

try:
    #posts = get_posts(61567187803636, pages=1, cookies=os.path.join(location, "cookies.txt"), options={"posts_per_page": 5})
    posts = get_posts(61567187803636, start_url="https://mbasic.facebook.com/61567187803636?v=timeline", pages=1, cookies=os.path.join(location, "cookies.txt"), options={"posts_per_page": 10})
    json_object = json.dumps(posts, iterable_as_array=True, default=str)

    posts_file = os.path.join(location, 'poststext.json')
    with open(posts_file, "w") as outfile:
        outfile.write(json_object)
except exceptions.TemporarilyBanned:
    print("Temporarily banned")
except exceptions.AccountDisabled:
    print("Account Has Been Disabled")
except exceptions.InvalidCookies:
    print("Cookies are not valid")
except Exception as e:
    print(e.args)

This script only parse the next text
Once upon a time in a quiet, misty village nestled between ancient woods and a winding river, there was a curious girl named Maren. She loved wandering through the fields and forest paths, often straying far from the village. She sought the unknown, the hidden, the stories no one else could tell.\n\nOne late autumn day, as the sun dipped beneath the horizon and a soft orange glow filled the sky, Maren stumbled upon a peculiar sight. A fox, sleek and silver, with piercing eyes like moonlight, was...

but the original text is
Once upon a time in a quiet, misty village nestled between ancient woods and a winding river, there was a curious girl named Maren. She loved wandering through the fields and forest paths, often straying far from the village. She sought the unknown, the hidden, the stories no one else could tell.

One late autumn day, as the sun dipped beneath the horizon and a soft orange glow filled the sky, Maren stumbled upon a peculiar sight. A fox, sleek and silver, with piercing eyes like moonlight, was sitting by an old, twisted tree. It didn’t flee at her approach. Instead, it looked at her as if it had been waiting.

“Hello,” Maren whispered, unsure if the fox could understand. But to her amazement, the fox tilted its head and spoke in a voice as soft as rustling leaves.

“I’ve been watching you, Maren. You walk these lands with a heart full of wonder, and it is rare to find such hearts. Will you listen to a tale that most would never believe?”

Maren, stunned but intrigued, nodded. She sat on the mossy ground, her eyes wide with anticipation.

The fox began, “Once, long ago, this land was home to more than just humans. Spirits roamed the forests, their laughter echoing through the trees, their songs as wild as the rivers. But over time, they faded, forgotten by those who couldn’t see or hear. Yet a few of us remain, carrying memories and whispers of that forgotten world.”

As the fox spoke, Maren felt as if the very air was alive with magic, as if the forest itself held its breath. She listened until the sky turned dark, absorbing tales of ancient spirits, hidden paths, and lost realms. When the fox’s tale ended, it rose and gazed at her kindly.

“You are part of these stories now,” it said. “If you listen closely, you may hear the voices that few remember. Keep your heart open, and you’ll find more than you ever imagined.”

With that, the fox vanished into the shadows, leaving Maren alone but forever changed. She returned to the village, feeling as if the whole world had grown richer, more mysterious, and brimming with secrets that only a rare few would ever know. And as she grew older, she often returned to that tree, hoping to see her fox friend again, always listening for the stories hidden within the forest's quiet whispers.


refactor: tekton.VerifyEnterpriseContract

Description

tekton.VerifyEnterpriseContract now has a top-level app.SnapshotSpec (Snapshot) field
two new helper methods on tekton.VerifyEnterpriseContract: - WithComponentImage - AppendComponentImage
tekton.VerifyEnterpriseContract.Bundle renamed to tekton.VerifyEnterpriseContract.TaskBundle
tekton.PipelineRunGenerator returns an error from Generate method
SSLCertDir set to a fixed value, less repetition this way
commented out code removed

Issue ticket number and link
Ref. https://issues.redhat.com/browse/HACBS-2200
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Run tests against quickcluster instance
Checklist:

[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("STONE-123456789 devfile source")
[ ] I have updated labels (if needed)

/retest

/retest

/retest

/retest

/retest


Leverage Signal Handler & Thread Events

Couple minor additions:

Added signal handler for SIGINT and converted threads to daemons for immediate shutdown.

Removed sleeps and leverage thread events instead to wait/notify on.

Thanks!


Installing Storybook 2.35.3 results in babel deprecation warning

Here are the relevant portions of my package.json:
  "dependencies": {
    "babel-plugin-transform-runtime": "~6.23.0",
    "babel-polyfill": "~6.23.0",
    "classnames": "~2.2.3",
    "react-icons": "~2.2.3",
    "react-pure-render": "~1.0.2"
  },
  "devDependencies": {
    "@kadira/storybook": "~2.35.3",
    "babel-cli": "~6.24.1",
    "babel-core": "~6.24.1",
    "babel-eslint": "~7.2.1",
    "babel-preset-es2015": "~6.24.1",
    "babel-preset-react": "~6.24.1",
    "babel-preset-stage-0": "~6.24.1",
    "chai": "~3.5.0",
    "chai-enzyme": "~0.6.1",
    "css-loader": "~0.28.0",
    "enzyme": "~2.8.0",
    "eslint": "~2.10.2",
    "eslint-config-arista": "0.0.1",
    "file-loader": "~0.11.1",
    "jsdom": "~9.12.0",
    "less": "~2.7.2",
    "less-loader": "~4.0.3",
    "lesshint": "~3.1.1",
    "mocha": "~3.2.0",
    "mocha-jenkins-reporter": "~0.3.7",
    "react": "~15.4.2",
    "react-addons-test-utils": "~15.4.2",
    "react-dom": "~15.4.2",
    "sinon": "~2.1.0",
    "style-loader": "~0.16.1",
    "url-loader": "~0.5.7",
    "webpack": "~2.3.3"
  }

If I blow away node_modules and perform a clean npm install, I get this warning:
npm WARN deprecated<EMAIL_ADDRESS>💥  preset-latest accomplishes the same task as babel-preset-env. 🙏  Please install it with 'npm install babel-preset-env --save-dev'. '{ "presets": ["latest"] }' to '{ "presets": ["env"] }'. For more info, please check the docs: http://babeljs.io/docs/plugins/preset-env 👌. And let us know how you're liking Babel at @babeljs on 🐦

I'm not installing babel-preset-latest or babel-preset-env directly; they're both coming from Storybook. Please let me know if you need any more info.

Thanks for the report!

I've worked on moving devDependencies to the root today, mostly babel. So Will need to check if this is still something that happens. I'm not getting any error-messages like this.

If someone could check if this still happens and if so when, that'd be a great help!

I'm happy to help out, but I don't see any releases after 2.35.3, which is the version I'm installing that exhibits this issue.

Hey thanks @kaiyoma !
We recently moved to a monorepo structure managed by lerna.
What you can do to help is reproduce the issue like this:

Clone the repo
perform an npm install (this should bootstrap lerna)
If it fails initially, retry it by running npm install again. (I'm working on that issue).
It should roughly look like this:➜  react-storybook git:(master) yarn
yarn install v0.22.0
[1/4] 🔍  Resolving packages...
success Already up-to-date.
$ lerna bootstrap --hoist
Lerna v2.0.0-rc.3
Encountered a cycle in the dependency graph. This may cause instability!
Bootstrapping 19 packages
Preinstalling packages
Installing external dependencies
Symlinking packages and binaries
Postinstalling packages
Prepublishing packages
Successfully bootstrapped 19 packages.
✨  Done in 62.68s.

Go to /examples/test-cra/
Make whatever changes to need to to reproduce the problem.
Share how you reproduced the problem, if simple by a message, if complex a branch I can look at is preferred

Thanks so much! Let me know if you need help.

I can't repro the issue by cloning the repo and installing. To repro the issue, you have to install storybook from inside another project. You can do so with this barebones config:
package.json:
{
  "name": "test",
  "devDependencies": {
    "@kadira/storybook": "2.35.3",
    "react": "15.0.0",
    "react-dom": "15.0.0"
  }
}

.npmrc:
@kadira:registry=https://registry.npmjs.org/

When you npm install, the first line of output will be the babel warning.

Right so that will install an older version, I'd love to know if what's on master still exhibits this behavior..

I wonder if that requires some npm link magic? I'm not familiar with that process, so if you have some steps to follow, let me know.

I'm seeing this warning related to babel-preset-react-app:

Looks like we're using babel-preset-latest somewhere we should be using babel-preset-env

@diegohaz Do you think you could try it with this branch? https://github.com/storybooks/storybook/pull/1008

@kaiyoma We now have some steps described here: https://github.com/storybooks/storybook/blob/master/CONTRIBUTING.md#testing-against-master
I hope this helps you reproduce the issue.

I'm getting an error when running npm install inside storybook:
$ npm install
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\chokidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})
npm ERR! Windows_NT 10.0.15063
npm ERR! argv "C:\\Program Files\\Node.js\\node.exe" "C:\\Program Files\\Node.js\\node_modules\\npm\\bin\\npm-cli.js" "install"
npm ERR! node v6.9.1
npm ERR! npm  v3.10.9
npm ERR! code E404

npm ERR! 404 no such package available : @types/react
npm ERR! 404
npm ERR! 404  '@types/react' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
npm ERR! 404 It was specified as a dependency of 'jest-enzyme'
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

How?! https://www.npmjs.com/package/@types/react

In my team's project, we need an entry in our .npmrc to be able to install Storybook:
@kadira:registry=https://registry.npmjs.org/

Maybe you need the same thing in your source tree?

Looking at your log, It's clear the package it's failing to install is a dependency of 'jest-enzyme'.
I'm guessing you need to map:
@types:registry=https://registry.npmjs.org/

I have no idea why this would be necessary, your node & npm versions seem fine to me.

Adding .npmrc with the line above fixed the install issue. I'm guessing you guys need to add that file as part of your project?
In the interest of full helpfulness, I noticed a few (probably benign) warnings during the installation:
npm WARN deprecated<EMAIL_ADDRESS>Use mz or fs-extra^3.0 with Promise Support
...
npm WARN storybook@ No repository field.
npm WARN storybook@ No license field.

During npm run bootstrap, I encountered resolution errors similar to the ones we saw before:
lerna ERR! execute npm ERR! 404 no such package available : @kadira/storybook-addon-links

However, this time, adding a line to .npmrc does not fix the issue. Here's my .npmrc now:
@kadira:registry=https://registry.npmjs.org/
@types:registry=https://registry.npmjs.org/

And here's what happens when I try npm run bootstrap:
lerna ERR! execute Error: Command failed: npm install
lerna ERR! execute npm ERR! Windows_NT 10.0.15063
lerna ERR! execute npm ERR! argv "C:\\Program Files\\Node.js\\node.exe" "C:\\Program Files\\Node.js\\node_modules\\npm\\bin\\npm-cli.js" "install"
lerna ERR! execute npm ERR! node v6.9.1
lerna ERR! execute npm ERR! npm  v3.10.9
lerna ERR! execute npm ERR! code E404
lerna ERR! execute
lerna ERR! execute npm ERR! 404 no such package available : @kadira/storybook-addon-links
lerna ERR! execute npm ERR! 404
lerna ERR! execute npm ERR! 404  '@kadira/storybook-addon-links' is not in the npm registry.
lerna ERR! execute npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
lerna ERR! execute npm ERR! 404
lerna ERR! execute npm ERR! 404 Note that you can also install from a
lerna ERR! execute npm ERR! 404 tarball, folder, http url, or git url.

That error repeats about 4 times and I also see this:
npm ERR! 404 no such package available : @kadira/storybook-deployer

Any ideas?
(BTW, how come you guys require the @kadira at all? I haven't seen that with any of the other dozens of NPM packages that we use.)

Hey @kaiyoma, I'm at a loss..
The .npmrc stuff you apparently need to do is not normal, It's not something everyone needs to do because https://registry.npmjs.org/ is the default registry of npm.
I have no idea why you have to add all organisations to your .npmrc file like that.

I'm aware of this one:
npm WARN deprecated<EMAIL_ADDRESS>Use mz or fs-extra^3.0 with Promise Support

But these are unknown to me:
npm WARN storybook@ No repository field.

We're migrating away from the @kadira organisation to a @storybook organisation on npm. But this is still in progress.

We have an internal NPM registry here at my organization. Would that explain things?

Yeah that would have to be it! Maybe you you can ask for support there.
We're using artifactory as a custom registry, and it's able to pass though to the regular registry (including organisations) just fine.

Our internal repository does the same thing actually. I'm wondering if the lerna layer requires more configuration to resolve all the names. In any case, I'll try testing master on my personal laptop later, which will be much easier than wrangling with my work environment.

I'm running into issues on my personal laptop as well. I believe I made it past the name resolution part and I'm now seeing an error relating to a missing directory:
$ npm run bootstrap

> storybook@ bootstrap C:\Users\Kyle\Downloads\storybook
> lerna bootstrap

lerna info version 2.0.0-rc.4
lerna info ignore [ 'test-cra' ]
lerna WARN ECYCLE Encountered a cycle in the dependency graph. This may cause instability!
lerna info Bootstrapping 18 packages
lerna info lifecycle preinstall
lerna info Installing external dependencies
lerna info Symlinking packages and binaries
lerna ERR! execute Error: ENOENT: no such file or directory, stat 'C:\Users\Kyle\Downloads\storybook\packages\react-storybook\dist\server\build.js'
lerna ERR! execute     at Error (native)
lerna ERR! execute  callback with error
lerna ERR! execute  { Error: ENOENT: no such file or directory, stat 'C:\Users\Kyle\Downloads\storybook\packages\react-storybook\dist\server\build.js'
lerna ERR! execute     at Error (native)
lerna ERR! execute   errno: -4058,
lerna ERR! execute   code: 'ENOENT',
lerna ERR! execute   syscall: 'stat',
lerna ERR! execute   path: 'C:\\Users\\Kyle\\Downloads\\storybook\\packages\\react-storybook\\dist\\server\\build.js' }

I looked at the directory structure manually and I can verify that there is definitely no dist directory underneath storybook/packages/react-storybook:
$ ll packages/react-storybook/
total 306
-rw-r--r-- 1 Kyle 197121   218 May 11 20:10 addons.js
-rw-r--r-- 1 Kyle<PHONE_NUMBER>3 May 11 20:10 CHANGELOG.md
-rw-r--r-- 1 Kyle 197121  3225 May 11 20:10 CODE_OF_CONDUCT.md
drwxr-xr-x 1 Kyle 197121     0 May 11 20:10 config/
drwxr-xr-x 1 Kyle 197121     0 May 11 20:10 demo/
drwxr-xr-x 1 Kyle 197121     0 May 11 20:10 docs/
-rw-r--r-- 1 Kyle 197121  1096 May 11 20:10 LICENSE
drwxr-xr-x 1 Kyle 197121     0 May 11 20:22 node_modules/
-rw-r--r-- 1 Kyle 197121  2301 May 11 20:10 package.json
-rw-r--r-- 1 Kyle 197121  1830 May 11 20:10 README.md
-rw-r--r-- 1 Kyle 197121   290 May 11 20:10 ROADMAP.md
drwxr-xr-x 1 Kyle 197121     0 May 11 20:10 scripts/
drwxr-xr-x 1 Kyle 197121     0 May 11 20:10 src/

This will be a known issue in the 3.0.0-alpha.0 release

It seems that  babel-preset-react-app is still using babel-preset-latest, but I could not find it

When I do a clean CRA install and then add storybook via cli, I do not get this message anymore.

npm ERR! code E404
npm ERR! 404 Not Found: bable-preset-env@latest
npm ERR!      C:\Users\HP PC\AppData\Roaming\npm-cache_logs\2018-06-30T08_40_00_824Z-debug.log
how do i install babel-cli babel-preset-env

Did you copy that output exactly? Because you have a typo here:
npm ERR! 404 Not Found: bable-preset-env@latest

That should be babel-preset-env.

thanks for that i will rerun it


[WIP] Skip Audit - Rendering Library

Description
WIP - identifying unattributed skipped tests
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

I am going to close and re-open this as a new PR.

Actually not necessary, I found what was wrong. :)


Update example configuration

These are the changes I had to make to get my config to work (see https://github.com/vinniefranco/exandra/discussions/50).

Thank you @restlessronin 💟


Added support for eris

This will allow people to use patron.js with eris, and paves the way to use patron.js with the other js libs as well in the future.
This is completely untested and still has a few things that need to be done.

So apparently the travis CI build will fail because eris/discord.js aren't dependencies so it doesn't install them. Should I add them both as optional dependencies to imply that only one should be used?

make them peer dependencies, optional ones are automaticallu installed unless the flag --no-optional is added

also, no library handler at all in the typings, or else it will appear to be available from the default exports in VSC intellisense, even tho library handler isn't exported. It's only used internally, it should just be a ghost property on the registry, remove from the typings

ok, since we only need to apply extensions for the library they are using, put the requireAll inside the registry constructor, and put the lib name as the second foldee inside the extensions, then remove the eris and d.js deps all together

You might wanna squash merge this so the commits aren't so messy lmao.


the code 2 ** 8 not work in IE 11.

in the pulgins "buffer", The code below is not work in IE 11.
Buffer.prototype.readBigUInt64LE = defineBigIntMethod(function readBigUInt64LE (offset) {
offset = offset >>> 0
validateNumber(offset, 'offset')
const first = this[offset]
const last = this[offset + 7]
if (first === undefined || last === undefined) {
boundsError(offset, this.length - 8)
}
const lo = first +
this[++offset] * 2 ** 8 +
this[++offset] * 2 ** 16 +
this[++offset] * 2 ** 24
const hi = this[++offset] +
this[++offset] * 2 ** 8 +
this[++offset] * 2 ** 16 +
last * 2 ** 24
return BigInt(lo) + (BigInt(hi) << BigInt(32))
})

This issue also affects certain older Android 6 devices, like the Oukitel K6000. The exponentiation operator was introduced in ES2016, which makes it newer than things like the import keyword or arrow functions.
We are going to work around this issue on our end by running buffer through Babel during our build step, but it might be nice to skip this.

That's expected because buffer@6 dropped support of IE11: 840160f

should the readme mention this as it currently says 'all browsers'?

As of 4th of August 2021, after 8 months of release of the v6, weekly downloads stats has spoken:

v6 (not IE 11 compatible) = 920k
v5 (IE 11 compatible) = 10,2M

You can check any other npm package, no one - that has such a success like this one - has so slow migration in such a long time.
I think it's clear that community chose version compatible with IE 11 (and old mobile devices).
It would be nice to take these feedback in account, but I also understand you don't force anybody to upgrade - luckily.
Of course we can use some tricks while building, but in my case I prefer to not complexify and maintain extra build steps.


Core language design: variable declaration as an example

Current syntax is very verbose, more like a macro for readability:

wenyan
JavaScript

吾有一數。曰三。名之曰「甲」。
var a = 3;

有數五十。名之曰「大衍」。
var dayan = 50;

昔之「甲」者。今「大衍」是矣。
a = dayan;

吾有一言。曰「「噫吁戲」」。名之曰「乙」。
var b = "alas!";

吾有一爻。曰陰。名之曰「丙」。
var c = false;

吾有一列。名之曰「丁」。
var d = [];

吾有三數。曰一。曰三。曰五。名之曰「甲」曰「乙」曰「丙」。
var a=1,b=3,c=5;

To be more concise, or to be more useful for programming:
數「甲」-> 甲 ("數" is not needed, it defaults to number if not inside a single quote)
「甲」 # means a string as it is inside a single quote, just like in UNix shell
wenyan | JavaScript
-- | --
甲曰三。 | var a = 3;
大衍曰五十。 | var dayan = 50;
甲為大衍」 | a = dayan;
「乙」曰「「噫吁戲」」 | var b = "alas!"; # here 「乙」is in a single quote so it's of a string type
爻丙曰陰。 | var c = false;
列丁（為空）。 | var d = [];
甲曰一。乙曰三。丙曰五。 | var a=1,b=3,c=5;
Keywords above are in bold.
If this is the way to go, should we write macros for all the above, or refactor the existing code and write another set of macros for existing / more verbose expressions?

Hi @kaiyuan01 , thanks for bringing this up, but:
This sounds more like a language philosophy issue. I could argue about the numerous merits of the current style like I did before, but more importantly I think wenyan-lang has already grown out of this stage. Take a cat for example, after being a cat for 10 years, the cat suddenly wonders why she wasn't a dog in the first place. Too late to reincarnate isn't it. Better start with a new dog instead of hacking the cat to make it look like a dog.
I think our efforts will have better value if we acknowledge what we currently have, and focus on improving the compiler, the standard libraries, the build/test tools, essential features, and fixing bugs.
Moreover, now with the addition of macros, users can define new syntaxes for themselves and do whatever they like with their own code. Just like in C, you can #define all the keywords with emojis or whatever, just don't expect the C committee to incorporate them into its standard :)
Thanks!


Fix recipe to use get_env_var

PR Checklist
Notebook requirements

[x] Notebook outputs cleared: Ensure all notebook outputs are cleared.
[x] Standard access to secrets and variables Include !pip install git+https://github.com/ibm-granite-community/utils in the first code cell in order to make get_env_var available to accessing secrets and variables in the recipe.

GitHub

[x] Commits signed: All commits must be GPG or SSH signed.
[x] DCO Compliance: Developer Certificate of Origin (DCO) applies to the code, documentation, and any example data provided. Ensure commits are signed off.

PR works with replicate key. See https://github.com/bjhargrave/granite-legal-cookbook/actions/runs/11712139963

As discussed, I will create a follow-on ticket to make sense of the bare get_env_var


How to ensure concurrency？

var dummy = 1;
function onrev(
){
dummy=3;
};
gofunc();
log(dummy);
I currently have this type of code，the gofunc is Go Thread ，this callback onrev，the i use the lock  for  vm.run,then gofunc is  it
func gofunc(){
go runscript()
}
func gofunc(){
lock();
vm.runscript()
unlock();
}
then the onrev is run after  the script Running completed，
how to i want?

In order to achieve concurrency with ECMAScript you must use an event loop, for example this one:  https://github.com/dop251/goja_nodejs/tree/master/eventloop

为了实现ECMAScript的并发性，您必须使用一个事件循环，例如:https://kgithub . com/DOP 251/goja _ nodejs/tree/master/event loop

I can understand it this way，the vm runscript is not need lock，the lock need for the vm in vaule？

You can't do go vm.RunProgram(prg). In fact, you should not use vm anywhere outside the function that runs on loop.


Suggestions : Documentation and Tests

Part of JOSS review : https://github.com/openjournals/joss-reviews/issues/6970
Dear developers,
Do not know how much of additional work it can be. I was thinking would it be feasible to add some examples for dihedral fitting using other forcefields i.e.,  CHARMM, TraPPE, AMBER, Mie, and Exp6  to the documentation ?
And also include some tests for these. I noticed MIE and OPLS are already included in tests, but could not find tests for other forcefields mentioned in the paper. (It is possible I could be missed the tests, kindly correct me if am mistaken on this.)

Hello!
Thank you for your review and comments!  We will complete all of these that are feasible to completed. We started a issue checklist with all the reviewer comments, so we can address them and write our teams notes here https://github.com/GOMC-WSU/MoSDeF-dihedral-fit/issues/101.
If we need any clarifications, we will kindly ask for them.  Thank you!

Addressed in #108


Create anchor links for methods in API reference

It would be useful to have an anchor for individual methods in a class, for cross-links from the documentation.

Duping against https://github.com/pulumi/docs/issues/492.


ingest/transform: add apply-geolocation-rules

Applies geolocation rules provided in the geolocation rules TSV file to
the geolocation fields (region, country, division, location).
This allows us to create general rules for geolocations that are not
tied to specific record accession (i.e. not user annotations).
Currently fetches the geolocation rules TSV from ncov-ingest, but in the
future this can point to a more centralized location for general rules.
Monkeypox data specific rules in ingest/source-data/geolocation-rules.tsv
will be concatenated to the general rules. Since these local rules are
added at the end, they can overwrite existing general rules.
Closes #34

This looks very good to me!


Adding data to existing categories

I added data into the categories [book, city, country, color, music, electronic, language]

Of course I will remove them now. Sorry for the inconvenience.

It's ok @NouranBakry 🙂  but there is still a failing test in the countries category. I've put a comment on but there could be others

Codecov Report

Merging #37 into main will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #37   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           48        48
  Lines          409       400    -9
=========================================
- Hits           409       400    -9

Impacted Files
Coverage Δ

qual_id/categories/city.py
100.00% <ø> (ø)

qual_id/categories/color.py
100.00% <ø> (ø)

qual_id/categories/electronic.py
100.00% <ø> (ø)

qual_id/categories/instrument.py
100.00% <ø> (ø)

qual_id/categories/music.py
100.00% <ø> (ø)

qual_id/categories/book.py
100.00% <100.00%> (ø)

qual_id/categories/country.py
100.00% <100.00%> (ø)

qual_id/categories/language.py
100.00% <100.00%> (ø)

qual_id/pattern.py
100.00% <0.00%> (ø)

test/utils/category_helper.py
100.00% <0.00%> (ø)

... and 1 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 fbcdf47...12692d3. Read the comment docs.

Thanks a lot for the contribution @NouranBakry ! You've went above and beyond  🙂  🎉

Thank you so much for giving me the chance to make my first contribution 😊


[WIP] Added serde optionnal dependency.

Opened early for feedback to know

Is this wanted?
What other types should serde derives be added to?

Had issues with this branch. This PR is now available here: https://github.com/rustsim/nphysics/pull/184


BootstrapNodeMetadata unmarshalJSON fails

when PROXY_VIA_AGENT is set due to missing
,string tag in struct
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ x ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

Hi @piotrkpc. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

When is PROXY_VIA_AGENT set to string and causing the unmarshal error?

@piotrkpc: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
288a0508e3fcc888038b8b062b54f3322bafec1c
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@piotrkpc: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
288a0508e3fcc888038b8b062b54f3322bafec1c
link
/test release-notes_istio

unit-tests_istio
288a0508e3fcc888038b8b062b54f3322bafec1c
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@piotrkpc: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
288a0508e3fcc888038b8b062b54f3322bafec1c
link
/test release-notes_istio

unit-tests_istio
288a0508e3fcc888038b8b062b54f3322bafec1c
link
/test unit-tests_istio

lint_istio
288a0508e3fcc888038b8b062b54f3322bafec1c
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@piotrkpc: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
288a0508e3fcc888038b8b062b54f3322bafec1c
link
/test release-notes_istio

unit-tests_istio
288a0508e3fcc888038b8b062b54f3322bafec1c
link
/test unit-tests_istio

lint_istio
288a0508e3fcc888038b8b062b54f3322bafec1c
link
/test lint_istio

gencheck_istio
288a0508e3fcc888038b8b062b54f3322bafec1c
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

hi @bianpengyuan we are getting this error on istio-ingressgateway startup
2021-07-07T11:11:55.657154Z	warning	envoy config	StreamAggregatedResources gRPC config stream closed: 2, failed to unmarshal node metadata ({"CLUSTER_ID":"Kubernetes","INSTANCE_IPS":"<IP_ADDRESS>","ISTIO_PROXY_SHA":"istio-proxy:5952f5fb5f85f495ac6d0925539dcd445cd618a0","ISTIO_VERSION":"1.10.2","LABELS":{"app":"istio-ingressgateway","chart":"gateways","heritage":"Tiller","install.operator.istio.io/owning-resource":"unknown","istio":"ingressgateway","istio.io/rev":"default","operator.istio.io/component":"IngressGateways","pod-template-hash":"59798cf48","release":"istio","service.istio.io/canonical-name":"istio-ingressgateway","service.istio.io/canonical-revision":"latest","sidecar.istio.io/inject":"false"},"MESH_ID":"cluster.local","NAME":"istio-ingressgateway-59798cf48-h58cj","NAMESPACE":"istio-system","OWNER":"kubernetes://apis/apps/v1/namespaces/istio-system/deployments/istio-ingressgateway","PILOT_CERT_PROVIDER":"istiod","PILOT_SAN":["istiod.istio-system.svc"],"PLATFORM_METADATA":{"azure_Name":"shoot--kyma-stage--c-67001a4","azure_kubernetes.io-cluster-shoot--kyma-stage--c-67001a4":"1","azure_kubernetes.io-role-node":"1","azure_location":"westeurope","azure_name":"shoot--kyma-stage--c-67001a4-cpu-worker-0-z1-775bd-vrq7s","azure_node.kubernetes.io_role":"node","azure_vmId":"caffae5b-845c-4e97-b442-8762eff5c25a","azure_worker.garden.sapcloud.io_group":"cpu-worker-0","azure_worker.gardener.cloud_pool":"cpu-worker-0","azure_worker.gardener.cloud_system-components":"true"},"PROV_CERT":"var/run/secrets/istio/root-cert.pem","PROXY_CONFIG":{"binaryPath":"/usr/local/bin/envoy","concurrency":"0","configPath":"./etc/istio/proxy","controlPlaneAuthPolicy":"MUTUAL_TLS","discoveryAddress":"istiod.istio-system.svc:15012","drainDuration":"45s","parentShutdownDuration":"60s","proxyAdminPort":"15000","proxyMetadata":{},"serviceCluster":"istio-ingressgateway","statNameLength":"189","statusPort":"15020","terminationDrainDuration":"5s","tracing":{"zipkin":{"address":"zipkin.kyma-system:9411"}}},"PROXY_VIA_AGENT":"true","ROUTER_MODE":"sni-dnat","SERVICE_ACCOUNT":"istio-ingressgateway-service-account","UNPRIVILEGED_POD":"true","WORKLOAD_NAME":"istio-ingressgateway"}): json: cannot unmarshal string into Go struct field BootstrapNodeMetadata2.PROXY_VIA_AGENT of type bool

we are not running pilot-agent and the ingressgateway stuck in this state:
kopec@piorts-nb ~ » k get pods
NAME                                                READY   STATUS    RESTARTS   AGE
application-connector-certs-sync-76965d9c75-wqkq4   1/1     Running   0          45h
istio-ingressgateway-59798cf48-2cgf9                1/1     Running   0          46h
istio-ingressgateway-59798cf48-h58cj                0/1     Running   0          49m
istio-ingressgateway-59798cf48-rwftw                0/1     Running   0          49m
istio-ingressgateway-59798cf48-zfp2x                0/1     Running   0          49m
istiod-5b8f656446-b5wv4                             1/1     Running   0          46h
istiod-5b8f656446-pq68c                             1/1     Running   0          49m

Does not happen to me... Do you happen to set PROXY_VIA_AGENT in env variable or customized bootstrap override?

@bianpengyuan I am assisting @piotrkpc on this issue. I have found what follows:

Both istio-system/ingressgateways and Pod sidecars in user namespaces are affected by the problem: unable to parse string "true" into Boolean
For ingressgateways I was able to provide PROXY_VIA_AGENT env var, because there's a Deployment. And while setting it to "true" yields the same error, setting it to '"false"` surprisingly works, the Pods start normally and seem to work. I don't know all the consequences of switching this value though.
For injected sidecars I was able to "fix" the problem by providing custom bootstrap override. Sample below

Custom bootstrap:
apiVersion: v1
kind: ConfigMap
metadata:
  name: istio-custom-bootstrap-config
  namespace: tomek
data:
  custom_bootstrap.json: |
    {
      "node": {
        "metadata": {
           "PROXY_VIA_AGENT": true
        }
      }
    }

The custom bootstrap fix would require to patch every single Pod in the cluster though. Not really acceptable long-term solution. Is there a way to inject it globally?
Where should we look to find the root cause of this strange behaviour (which configuration sources)?
We provisioned hundreds of clusters and this is the only one with this problem (so far), so we think there must be something in the configuration, or cluster environment, that is causing this errors.

The same happens when I disable automatic sidecar injection and inject manually with istioctl inject. I did that to have full control over sidecar envs. But I don't see anything suspicious there, actually the same set of envs work just fine on another cluster. So this has to be a config external to the sidecar, perhaps something in istiod.

PROXY_VIA_AGENT should already be default and I don't think you need to configure it at all, no?

@bianpengyuan yes, the thing is more complicated - please see this issue for details: https://github.com/istio/istio/issues/33989

@piotrkpc: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Concurrent translation of multiple keys

This is the simplest case of concurrent translation, no error handling, no resume function, can be used with care, if all goes well, it will be very fast
Or wait for https://github.com/yihong0618/bilingual_book_maker/pull/62
Usage: --max_procs num Note that first -h look at
If this parameter is not used, the program functions unchanged
If it is used, the output is modified because it is no longer outputting the content before and after translation because it is too fast.
Instead, it outputs the file inside the epub currently being processed and the apikey used for the current request (so you can see how fast it is now)
Note that it does not care how many keys you have, if you only have 3 keys, but concurrently 100, there will be problems
may be 30 keys, concurrently <= 3
It may output the wrong progress but the result book is correct, I think it should not output the progress bar, but it sometimes outputs... I don't know why.
chatgptapi_translator.py change a lot because https://github.com/yihong0618/bilingual_book_maker/pull/112, actually only add noprint option in this file

My experience I use 30 keys --max_procs 5 to translate animal_farm.epub successfully every time. But when I translated a 300-page English book, the translation failed using --max_procs 5, so I changed it to --max_procs 3 and the translation was successful, but the translation time was only half of --max_procs 5.  --max_procs 3 is not three times as fast as --max_procs 1, but much faster

I will take a look at #112 first maybe tonight

A bit busy right now..

Now a lot of codes have changed, and I am using this function locally and no interest in improving this pr and fix conflicts. I feel that modifying it is not as good as raising a new pr, so I closed it and welcome others to pr


Refactoring (Clean Code)

Problem
Keep the clean code project patterns applied in the initial backend project structure.
Solution
Verify if the project structure until here is ok and correct the needed.

Refactoring done!


add heudiconv

thank you - it will be available in Neurodesk in a few days :) I let you know once it's online


filters: Clear search input on filter delete

Connects-to: #542
Change-type: patch
Signed-off-by: Dimitrios Lytras<EMAIL_ADDRESS>---- Autogenerated Waffleboard Connection: Connects to #542

Tests added. First time working with Enzyme, seems pretty great.

@hedss Do you have any idea on this VB error?


test: enable nexmark q21 in planner test

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
REGEXP_EXTRACT is not a Postgres function. It can be replaced with REGEXP_MATCH with array accessing just like what we did in e2e test and benchmarks.
BTW, we should really find a way to extract these common SQLs. 😕
Checklist For Contributors

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[x] I have demonstrated that backward compatibility is not broken by breaking changes and created issues to track deprecated features to be removed in the future. (Please refer to the issue)
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Checklist For Reviewers

[ ] I have requested macro/micro-benchmarks as this PR can affect performance substantially, and the results are shown.

Documentation

[x] My PR DOES NOT contain user-facing changes.

Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note

BTW, we should really find a way to extract these common SQLs

What do you mean by extract? Can you elaborate?

BTW, we should really find a way to extract these common SQLs

What do you mean by extract? Can you elaborate?

Put the source of truth of these queries into some files, and reference them in planner tests and e2e tests with template substitution. 😄


Updated win2012r2 to use Nov2014 update rollup iso

Updated win2012r2 to use Nov2014 update rollup iso -- Downloadable via
an MSDN subscription.
Tested out builds using packer and the machines built just fine. Using these internally for Chef Windows needs.
Let me know if you need additional validation or tests!

These get overwritten by the values in Makefile

Shouldn't we provide good defaults for consumers of boxcutter?
If you're using winr2012r2 standard this is the current modern release of the platform.

Agreed. My comment was to alert us that we'll need to update the Makefile as well with these new values. Sorry if that wasn't clear.

Thanks for your contribution. Much appreciated!

@rasa Makefile update is included in the PR. Thanks!