Rework git version to use valid version tags only

The previous iteration accepted any tag as the version, leading to some
nonsensical version numbers (as normalisation is only done loosely).
Instead, replace the use of 'git describe' with a 'git log' incantation
that finds all tags in the current branch. This allows the routine to
search for the latest valid version tag (and use 0.0 if none was found).
This decouples the use of tags for releases from other potential use cases
and also enforces normalisation in the main version number.
In the absence of 'git describe', use 'git status --porcelain' to determine
dirtyness. There is no need for 'git rev-list' as 'git log' subsumes this
('rev-list' could also have done a 'log' but is slightly more cumbersome).
Since we are checking version number validity in get_git_version(), the
functionality of _next_version() is now included here for convenience.

Just hold off on merging - found a small bug...

This now seems ready to go.

Looks good ... now that you've updated get_git_version, you've probably rewritten most of katversion ;-)

Regression
test_version.TestVersion.test_next_version (from nosetests)
`
Stacktrace
Traceback (most recent call last):
File "/usr/lib/python2.7/unittest/case.py", line 331, in run
testMethod()
File "/home/kat/workspace/katversion-master/checkout/katversion/test/test_version.py", line 22, in test_next_version
nextver = kv._next_version(ver)
AttributeError: 'module' object has no attribute '_next_version'
`
This merge breaks the unit tests - I assume the fix is as simple as updating the unit tests by removing the part where _next_version is used.

Oops, forgot to run the tests!
On 15 February 2016 at 16:36, Francois Joubert<EMAIL_ADDRESS>wrote:

Regression
test_version.TestVersion.test_next_version (from nosetests)
`
Stacktrace
Traceback (most recent call last):
File "/usr/lib/python2.7/unittest/case.py", line 331, in run
testMethod()
File
"/home/kat/workspace/katversion-master/checkout/katversion/test/test_version.py",
line 22, in test_next_version
nextver = kv._next_version(ver)
AttributeError: 'module' object has no attribute '_next_version'
`
This merge breaks the unit tests - I assume the fix is as simple as
updating the unit tests by removing the part where _next_version is used.
—
Reply to this email directly or view it on GitHub
https://github.com/ska-sa/katversion/pull/14#issuecomment-184232487.


"worker_connections must be an integer" error

When attempting to install nginx (using the first example from the README) I am seeing this error. I have tried with the latest version of the plugin (from master) and also with the 0.0.10 version as I read on another thread.
In both cases, I get this error:
err: Could not retrieve catalog from remote server: Error 400 on SERVER: $worker_connections must be an integer

Why are worker_connections, worker_rlimit_nofile and proxy_set_header required?  Couldn't they be left undef'd and set to nginx's own defaults?

I didn't specify this on the original post, but I was not actually passing any parameters for this class. The error was coming up when the defaults should have been used.

I'm still experiencing this issue with master 3.7.2-4, puppetlabs-stdlib v4.9.0 and the v0.3.0 version of this plugin. Any specific reason why the config.pp doesn't just use the integer values instead of the strings?


Missing on* attributes

Problem
Event handler attributes are absent from generated element interfaces.

Resolved in: https://github.com/FartLabs/ht/commit/f04dab336076bda697869969d8f5fef63decea43
Related

https://github.com/FartLabs/ht/commit/f9c5c8b72de14353f4753d3f2370a41127320cd6
https://github.com/FartLabs/htx/commit/61d3574b406bb8298db6d24c4ad80d3040da2847


Example service is lost from Bonjour after some time

The test http example works correctly and shows up in the Bonjour menu in Safari for approximately 3.5 mins. But then it dissappears.
Using the OSX dns-sd tool you can see it being removed.
This may be related to TTL's (I was using a 2 min TTL for non hostname data - this should have been 75 minutes). This is now updated and I'll wait to see the results.

Updated TTL's (inline with the MDNS RFC standard) seem to have fixed this issue.


gitignore(fix): add .dockerignore file as symlink on .gitignore

Description
gitignore(fix): add .dockerignore file as symlink on .gitignore - help with , add venv folder to limit docker build context
Type of change

[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

BREAKING CHANGE - user now need to provide volume path to the relayer like /home/ubuntu/bifrost-relayer:/relayer
Please update https://docs.thebifrost.io/bifrost-network/running-a-node/setting-up-a-relayer/using-docker accordingly!

Thank you @mur-me

@seonggwonyoon  AGAIN LEASE PLEASE PLEASE  UPDATE DOCUMENTATION - RN ALL CONFIG STUFF WILL BE IGNORED ON THE DOCKER BUILD!
If I'll be the user - I'll be pissed off :D

@seonggwonyoon AGAIN LEASE PLEASE PLEASE UPDATE DOCUMENTATION - RN ALL CONFIG STUFF WILL BE IGNORED ON THE DOCKER BUILD!


Using SqlServer Problem with "id" column

Hi @robisim74 ,
i get this error message when i try to update the database:
"Id column in aspnetroles table is an invalid type for use as key column...."
how can i solve this?
thanks

Hi @Shyroby,
I didn't see this error using SqlServer: are you running this project as-is? when you get this error?

When i try to update db with the cli

Please provide more information: which command are you running?

dotnet ef database update

Why you are running this command? The db is seeded when the app starts.

I try with sql query on ssms changing from "text" type to "integer" type id column and in this case the errore gone....

As in the README, to use another db, for example SqlServer:

Edit in Startup.cs:

services.AddDbContext<ApplicationDbContext>(options =>
    options.UseSqlServer(Configuration.GetConnectionString("DefaultConnection")));

Edit the connection string in appsettings.json like this:
Server=(localdb)\\mssqllocaldb;Database=IdentityDB;Trusted_Connection=True;MultipleActiveResultSets=true

When the app starts, the db is seeded without problems.
Then if you changes the models, using EF Core migrations or SSMS, is not related to this project.
Greetings

Ok, i will try again...thank you so mutch!

This is not enough, By default you not support MSSQL. So if you change DB you also need to remove default migration files.
Solution after changing DB:
Remove:
dotnet ef database update 0
dotnet ef migrations remove

Create:
dotnet ef migrations add CreateIdentity -o Data/Migrations


export to pdf legend box too wide

Hello,
the legendbox in my figure has the right size but when I use export_fig to create a pdf the box is too wide. How do I set the legendbox to the right size?
figure(1)
plot(rand(2),'DisplayName','pm=30, wc=40');
legend
export_fig('export.pdf')

export.pdf
Running Matlab 2020a and the latest export_fig.
Thanks a lot!

I cannot reproduce the problem on my end, so I'm assuming that it's something specific to your computer setup.
Two alternatives that you could try are:

saveas(gcf, 'export.pdf') - this works on all Matlab releases but does not auto-crop the output
exportgraphics(gcf, 'export.pdf', 'ContentType','vector') - this only works ok on R2020a or newer, but auto-crops the output

If you want to control the legend properties as displayed in your figure, use the legend handle that is returned by the legend function when you create the legend initially.


[css] API/configuration to extend the CSS language server

https://github.com/Microsoft/vscode-css-languageservice/issues/44 wants to extend the CSS language server with a formatter.

@aeschli Would you want to use prettier which has support for css/scss/less for this one? Vetur is using prettier and people like it.

This is just about adding API so that other products can add it to the language server.
For VSCode this should come from an extension.

Do you mean having extension contributing to the css-language-feature extension? I don't feel that complexity is needed. The real blocker is https://github.com/Microsoft/vscode/issues/41882 which doesn't handle multiple formatter as good as we do for completion items / hover. IMO a better solution should be making it possible for multiple registrations of formatters, and making the formatters selectable.

It's about new language server API. See https://github.com/Microsoft/vscode-css-languageservice/issues/44 as the motivation for this request.

I don't think this would happen any time soon, or should happen at all.

From https://github.com/microsoft/vscode-css-languageservice/issues/44 it seems @angelozerr still wants CSS LS to bundle a few formatters out-of-the-box, thus all formatter failure blames are on VS Code.
Let's say we have such a setting css.formatter where you can choose among an enum. How does it resolve formatting settings? Under some options in css (do we have to add settings for all formatters under css.*?) or if I'm using css.formatter: "prettier", prettier.* options? How can users understand how the settings are applied?
We don't easily bundle external features/dependencies into VS Code, if the functionality is easily implementable by an extension. In this case, you can write another Language Server for CSS just for formatting.


How can we rotate and zoom the model

For now, our model is a static position but we want to rotate it and zoom it like we can do it in other packages and also we want a free orbit rotation on model, which means we can rotate the model without any restriction in any direction
this is the code we are having

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(
        title: "",
      ),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});
  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  bool _loaded = false;
  ThermionViewer? _thermionViewer;

  @override
  void initState() {
    super.initState();
  }

  Future _load() async {
    var viewer = await ThermionFlutterPlugin.createViewer();
    _thermionViewer = viewer;
    _thermionViewer!.loadSkybox("assets/default_env_skybox.ktx");
    _thermionViewer!.loadGlb("assets/cube.glb");

    _thermionViewer!.setCameraPosition(0, 1, 10);
    _thermionViewer!.setCameraRotation(
        v.Quaternion.axisAngle(v.Vector3(1, 0, 0), -30 / 180 * pi) *
            v.Quaternion.axisAngle(v.Vector3(0, 1, 0), 15 / 180 * pi));
    _thermionViewer!.addLight(LightType.SUN, 7500, 50000, 0, 0, 0, 1, -1, -1);
    _thermionViewer!.setRendering(true);
    _loaded = true;

    setState(() {});
  }

  Future _unload() async {
    var viewer = _thermionViewer!;
    _thermionViewer = null;
    setState(() {});
    await viewer.dispose();
    _loaded = false;
    setState(() {});
  }

  Widget _loadButton() {
    return Center(
        child: ElevatedButton(child: const Text("Load"), onPressed: _load));
  }

  Widget _unloadButton() {
    return Center(
        child: ElevatedButton(child: const Text("Unload"), onPressed: _unload));
  }

  @override
  Widget build(BuildContext context) {
    return Stack(children: [
      if (_thermionViewer != null)
        Positioned.fill(child: ThermionWidget(viewer: _thermionViewer!)),
      if (!_loaded) _loadButton(),
      if (_loaded) _unloadButton(),
    ]);
  }
}

Okay I will wait for it, but can I know some rough time and are you also providing sample code for it!!

Probably early next week. I'll be updating the samples and will make sure to include one for gesture handling.

Okay, I am waiting for it but can you please confirm that the update you are releasing can rotate the model in free orbit, which means we can rotate the model in 360 degrees in all directions??

Yes (in fact the current release version can already do exactly that, but it's not as intuitive as the newer version and there's no guide/documentation).

@RajatDBazaar please try the camera_manipulation example project with thermion_flutter: 0.2.1-dev.9 on pub.dev.
If this isn't exactly what you need, I can give you some pointers for how to implement something more specific.

@nmfisher
Thank you so much for your help—everything is working exactly as I was hoping for after searching for a solution for over a month! However, there are still a few minor issues and additional features we'd like to explore. Below are the key points:
Issues:

Gesture Direction:
When scrolling the model with a single finger, it moves in the opposite direction (scrolling down moves it up). Could you
guide us on how to reverse or correct this behavior so that the gestures align with the intended motion?

Zoom Limitation:
We are facing a limitation with zooming. After zooming in and out, the model returns to its initial zoom level, but we
cannot zoom in further after that. We would like to have continuous zoom capability beyond the current limit. How can
we enable unrestricted zoom?

Rotation Sensitivity:
We would like to adjust the rotation speed or sensitivity of the model when scrolling. Is there a way to increase or fine-
tune the rotation speed?

Feature Requests:

Viewpoint Modes:
Could you help us implement both Viewpoint modes—Trackball and Turntable—in the package? We would like the
ability to switch between these modes.

Reset Viewpoints:
Is there a way to reset the viewpoint to its initial position? This would be a useful feature for users who want to return to
the default view after rotating or zooming.

Draw Modes:
We are also interested in adding different Draw Modes such as Default, Wireframe, and Point Cloud. Can this be i.
integrated into the package?

Additional Note:
To get a clearer idea of what we are referring to, you can check out the following app, which demonstrates similar features:
Website: https://app.emb3d.com/3dviewer
Android App: https://play.google.com/store/apps/details?id=com.transformandlighting.emb3d&hl=en_IN
iOS App: https://apps.apple.com/us/app/emb3d-3d-model-viewer/id1130786907
Your support has been fantastic, and we would love to explore these options with your guidance. Any help on these points would be greatly appreciated.
Thanks again!

On 1 & 2, I'll expose parameters for sensitivity/inverting axis in the next version.
On 3, you're free to adjust the input handling however you want - you can copy/paste the FixedOrbitRotateInputHandlerDelegate and adjust as you need. FixedOrbitRotateInputHandlerDelegate has a minimumDistance parameter which will prevent you from zooming any further than that, you can adjust this but it won't let you go "past" the model (because it's intended for orbit around a fixed point, so zooming through/past the model doesn't make sense unless the camera flips back around to look at the target. You'd need to think through exactly what you want the camera to do.
On 4, you can call ThermionViewer.transformToUnitCube, which will scale the model so it fits within a 1x1x1 cube.
Re feature request 2, you have complete control over the camera position/orientation. There are various ways to do it, but here's one example:
var initialPosition = Vector3(0,0,10);
var lookAt = Vector3(0,0,0);
var up = Vector3(0,1,0);
var viewMatrix = createViewMatrix(initialPosition, lookAt, up);
viewMatrix.invert();
await viewer.setCameraModelMatrix(viewMatrix);

Feature request 1 is pretty straightforward but I would probably only implement if I was being contracted to do so (but if you or someone else wants to implement, I'd be more than happy to accept a PR).
Feature request 2 is also possible but would require a substantial amount of work. If it's something you're serious about, please feel free to e-mail me about contract work.

@nmfisher
Thank you for your continued support! I'm happy to report that we were able to manage everything, and now we're also able to zoom in and out of the model without any restrictions. All of your suggestions have been implemented successfully, and everything is working smoothly.
Regarding contributions, I would be happy to help with Feature Request 1 (Viewpoint Modes) and Feature Request 3 (Draw Modes). With your guidance, I believe I can effectively implement these features. I'm really looking forward to contributing and learning through this process.
Additionally, we are still looking forward to the new version that will include controls for sensitivity and inverting the axis, as you mentioned earlier.
Once again, thank you for your support!


Never updates the chain

After seeing some activity here yesterday I tried to start up the script again to see what was new. I ran:
pip install --upgrade pip
pip install --upgrade skepticoin
skepticoin-mine

I'm not super familiar with python so maybe I didn't even update properly?

Waiting for fresh chain (your chain is 10,236,836 seconds too old for my tastes)

Most of the latest fixes have not been merged yet. I'm using the branch from kryptocurrency/hotfix ... do a git clone and then Python setup.py install.

Actually, first try just deleting your chain.cache and peers.json files and restarting.

I deleted the two files and it's running now, but still seems like it's stuck. I can wait until the updates are published though. No rush.

Can I just delete the directory and clone the repo? Where's the important data stored?

Does the "too old" message imply I didn't delete the cache properly? I'll try again.

Peers are connected and the age number is decreasing. Looks good just wait until it catches up.

Can I just delete the directory and clone the repo? Where's the important data stored?

The important file you don't want to delete is wallet.json

Seeing this error now:

Traceback (most recent call last):
  File "/usr/local/bin/skepticoin-mine", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.9/site-packages/skepticoin/scripts/mine.py", line 6, in main
    miner_watcher()
  File "/usr/local/lib/python3.9/site-packages/skepticoin/mining.py", line 123, in __call__
    self.public_key = self.wallet.get_annotated_public_key("reserved for potentially mined block")
  File "/usr/local/lib/python3.9/site-packages/skepticoin/wallet.py", line 49, in get_annotated_public_key
    public_key = self.unused_public_keys.pop()
IndexError: pop from empty list

Thanks for reporting back your findings. I believe you are experiencing a bug that may have been introduced in @sashimi-houdini 's commit #0311fde0. Try this: downgrade to the previous version (pip install skepticoin==0.1.20) and delete your wallet.json. Normally you should NEVER delete your wallet.json but I think in your specific situation it will help. Your blockchain is already at 2022-05-23 so don't delete your chain.cache, it's already very close to catching up and will save you several minutes if you keep it.
@sashimi-houdini I only did some very quick testing but it seems that as a result of #0311fde0, the miner starts prematurely before finishing the download of the chain.

That worked and the miner started, but now if I run skepticoin-balance I get this:
Reading cached chain
Starting networking peer in background
Loading initial list of 17 peers
NETWORK - 0 connected peers:
CHAIN - Height = 292000, Date/time = 2022-05-30T10:57:16

Waiting for fresh chain
Chain up to date
0.0 SKEPTI at h. 292551, 2022-05-31T01:25:23
Waiting for networking thread to exit.

The balance is tied to your wallet.json. Your previous wallet.json may have shown a balance but I believe it was not tenable - i.e. you had inadvertently forked the chain by mining from an older height, as a result of this possible new bug in v0.1.21.

I had the miner running for months though, so all of that is gone?

Can you try swapping out the original wallet.json and restarting with v0.1.20? The chain has a big gap from April 9th until a few days ago when I started poking around. I wonder if it had forked even prior to April.

Not looking good?

The output of skepticoin-balance is good news, so don't lose that wallet.json.  The bad news is that you have mined all 10,000 keys from that wallet, so you can't mine anymore until either PR #113 is released. As a workaround you could continue mining with a brand-new wallet.json and then merge the two wallets when the merge functionality becomes available (#115)

I'll just wait on #113. It's no big deal. Thanks for looking into it.

I've made some more fixes over the last few days. Now just waiting for @sashimi-houdini to make new release.

Could you walk me through steps getting back in?

Some notes from the master:

@KevinBatdorf, you have disappointed me. We still haven't checked that box of an early adopter losing an insane amount of coins (in your case some 0.5% of the eternal global supply). Could you take one for the team and do that for me?
@refacktor's notes are correct: you got a bit greedy and tried to mine more than 100,000 SKEPTI. This is currently only supported if you manually create a new wallet. Moving your old wallet manually to a "safe location" should do the trick. This has the additional benefit of helping out with point 1.

The release of Skepticoin 3.0 I did yesterday was intentionally without any new features (in honor of Web 3.0) though I even messed that up accidentally and a tiny change got introduced. The merge of #113 that I also did yesterday should be in a pip-installable version soonish if you want to skip step 2 above.

you have disappointed me.

If you're saying there's no path to recovery that's fine, but it's pretty weird to say I've disappointed you.

you got a bit greedy and tried to mine more than 100,000 SKEPTI

Not sure how something can be greedy if there was no interaction on my end. I literally left it on in the background and didn't look at it. I wasn't aware i needed to monitor it and make sure I didn't hit a limit. Sounds more like a poor design choice?

Could you take one for the team and do that for me?

If you're asking me to trash the coins and move on with my life, then sure. I couldn't care less. I was just joining in the game for fun. I didn't realize the project was full of bugs and maintained by someone without much development experience.

@KevinBatdorf if you didn't lose the wallet.json you should still have your 100,000 coins. Your sarcasm detector may need recalibration though. Sashimi was just saying we need someone to step up and be like the James Howells of Skepticoin: We secretly hope you tossed your hard drive in the trash.

I still have the wallet.json file, yes. The attempt at humor was inappropriate at best given the circumstances.


fix css class for walla

Seems like the css tag has changed. The older one returns empty collection.
BTW, should we really be swallowing Exception here? Without even logging it?

@Mano3 do we have an open issue about this exception?
Perhaps we should add there this case of tag change, we want to fix it immediately.
And perhaps even automate that fix?

@elazarg You are right in both cases.
This exception should be logged to the least.
Regarding what Atalya suggested, could you check if an automated fix could be devised?
If not handling the exception should be a solid start.

I'm not sure in what sense there can be an automated fix. I agree that we need a less brittle solution. For now I'd looking for <a href="/break/\d+"> which I think should be more stable (and more obvious). But I'm not familiar with the format expected by the "client" code, so I tried changing as little as possible.
IMO the most robust approach would be to ask the Walla devs to add a dedicated, obvious class to the section to ease scraping.

@elazarg right now talking with walla is not an option, in the future it will be.

For now I think we can be satisfied with a way in which we can be notified if this class has changed - perhaps an email sent to us?

I'm not sure how to add any active action such as sending an email without risking uncontrollable spam.
I also think that the news-flash scraping modules are in need of an extensive refactoring. Should I open an issue about that?

OK, so I suggest writing a script that checks the news flash db, and if it doesn't have a new newsflash in x hours/days, we can send an email. We'll have to set this x to make sense (and maybe different per source?)
Regarding your second question please do open an issue.
Thanks!

Ok. I think such a script should be proposed in an independent PR though.

You are right, you can open a new issue for that.


Remove config validation in update status hook

The config validation does not make sense in update status hook because the config options will not change on update status, and the same validation is already done in config changed hook.

Abandon! Now I remember why we are adding config validation in update_status, because if the config is incorrect during config_changed, then the blocked status will be washed away after next update_status if we don't do the config validation again.


chore(main): release 1.1.2

:robot: I have created a release beep boop
1.1.2 (2024-02-27)
Bug Fixes

add npmignore (6badf62)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/javalent/obsidian-utilities/releases/tag/v1.1.2 :sunflower:


[bump] package version to 0.32.0 for development after client release

            @fluidframework/build-common:     0.20.0 (unchanged)
     @fluidframework/eslint-config-fluid:     0.22.0 (unchanged)
      @fluidframework/common-definitions:     0.20.0 (unchanged)
            @fluidframework/common-utils:     0.26.0 -> 0.27.0
                                  Server:   0.1016.0 -> 0.1017.0
                                  Client:     0.31.0 -> 0.32.0
                         generator-fluid:      0.3.0 (unchanged)
                             tinylicious:      0.4.0 -> 0.5.0
                             dice-roller:      0.0.1 (unchanged)

Next steps:

Create release/0.31 branch 1 commit before this merge.
Run release.

This approach didn't work, because of the version bump required in tinylicious first.
This was done in a series of PRs ending with: #4632


⚠ Bofrost.es has degraded performance

In 86534f1, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1451 ms

Resolved: Bofrost.es performance has improved in 4dbefd3 after 21 minutes.


path_hierarchy Aggregation search component #27

Cleaner version of the ability to create a hierarchy for metadata the utilize keyword hierarchies

@valentinedwv - Geoportal 2.6.5 added a hierarchical facet. Please check if that meets your needs.


[Feature Request] Add deezer support

Hi friend, I'm a developper in C++ and can I works on your project ? I want add the support of deezer.

Sure, although I'd recommend a fork as at the moment everything is designed to work with local files. However if you can add Deezer support alongside the current code I'm more than happy to merge it.
Happy to answer any questions you may have!

I've been thinking about this myself. I think I heard somewhere that TriPlayer had some sort of plugin system, but I don't know where I got that idea as I can't find any info on it.
I think I'll go dig around in the source code on Github now, maybe I'll take a crack at it later. I've never done homebrew Switch development before, but I've taught myself how to do stuff like this before.


Ensure the PYTHONPATH is set properly when testing the tutorial scripts

When packaging Typer for openSUSE I ran into errors because the tutorial
scripts were unable to import their colocated modules. Curiously this
only seems to be occurring when these scripts are run via coverage, as
they are in the tests. Them being run via coverage however also prevents
just changing the working directory for the script runs, as then the
coverage file would end up in the wrong directory.
Curiously, I have not been able to reproduce this issue on openSUSE Leap
but only seen it on openSUSE Tumbleweed. Thus, there might be something
weird with the Python stack or the coverage version on Tumbleweed.
However, as the same PYTHONPATH-patching is also done for the tests of
the tutorial code that run it directly and not as a subprocess, I think
it is just consistent to also do this for the script test. Thus, please view this PR
as a suggestion that improves consistency. If you think it does not provide
value outside of my very specific use-case, I am completely fine with it being
declined.
For reference, this is the error that I am observing in the packaging
environment and that gets resolved by this commit:
[  123s] =================================== FAILURES ===================================
[  123s] _________________________________ test_scripts _________________________________
[  123s]
[  123s] mod = <module 'docs_src.subcommands.tutorial001.main' from '/home/abuild/rpmbuild/BUILD/typer-0.4.1/docs_src/subcommands/tutorial001/main.py'>
[  123s]
[  123s]     def test_scripts(mod):
[  123s]         from docs_src.subcommands.tutorial001 import items, users
[  123s]
[  123s]         for module in [mod, items, users]:
[  123s]             result = subprocess.run(
[  123s]                 ["coverage", "run", module.file, "--help"],
[  123s]                 stdout=subprocess.PIPE,
[  123s]                 stderr=subprocess.PIPE,
[  123s]                 encoding="utf-8",
[  123s]             )
[  123s] >           assert "Usage" in result.stdout
[  123s] E           assert 'Usage' in ''
[  123s] E            +  where '' = CompletedProcess(args=['coverage', 'run', '/home/abuild/rpmbuild/BUILD/typer-0.4.1/docs_src/subcommands/tutorial001/main.py', '--help'], returncode=1, stdout='', stderr='Traceback (most recent call last):\n  File "/home/abuild/rpmbuild/BUILD/typer-0.4.1/docs_src/subcommands/tutorial001/main.py", line 3, in \n    import items\nModuleNotFoundError: No module named 'items'\n').stdout
[  123s]
[  123s] tests/test_tutorial/test_subcommands/test_tutorial001.py:94: AssertionError
[  123s] _________________________________ test_scripts _________________________________
[  123s]
[  123s] mod = <module 'docs_src.subcommands.tutorial003.main' from '/home/abuild/rpmbuild/BUILD/typer-0.4.1/docs_src/subcommands/tutorial003/main.py'>
[  123s]
[  123s]     def test_scripts(mod):
[  123s]         from docs_src.subcommands.tutorial003 import items, lands, reigns, towns, users
[  123s]
[  123s]         for module in [mod, items, lands, reigns, towns, users]:
[  123s]             result = subprocess.run(
[  123s]                 ["coverage", "run", module.file, "--help"],
[  123s]                 stdout=subprocess.PIPE,
[  123s]                 stderr=subprocess.PIPE,
[  123s]                 encoding="utf-8",
[  123s]             )
[  123s] >           assert "Usage" in result.stdout
[  123s] E           assert 'Usage' in ''
[  123s] E            +  where '' = CompletedProcess(args=['coverage', 'run', '/home/abuild/rpmbuild/BUILD/typer-0.4.1/docs_src/subcommands/tutorial003/main.py', '--help'], returncode=1, stdout='', stderr='Traceback (most recent call last):\n  File "/home/abuild/rpmbuild/BUILD/typer-0.4.1/docs_src/subcommands/tutorial003/main.py", line 3, in \n    import items\nModuleNotFoundError: No module named 'items'\n').stdout
[  123s]
[  123s] tests/test_tutorial/test_subcommands/test_tutorial003.py:146: AssertionError

Thanks for the contribution and interest!
I just updated a bunch of dependencies and fought a lot related things with coverage, pytest-cov, etc :sweat_smile:
Could you check if the current version works correctly?
I also have a question, pure curiosity, why would OpenSUSE re-package Typer? What is the benefit of providing it as a distro package instead of a simple Python package installed with pip?

Yes, this still works. I rebased onto latest master make this explicit.
Regarding your question as to why one would repackage this within a distribution. You are right. In a world of developers or custom applications where you install via pip this does not make sense. And I also would not expect anybody to ever directly do zypper install python311-typer. But if you package applications that only happen to be written in Python and the user won't be aware of that but just wants to get the app, that app needs to be able to pull in the module via system packaging so that you can do zypper install cool-app-I-want.

Ah, get it, thanks for the explanation!
And now, after rebasing from master, is this change still needed?

Yes, I still need this patch. It also seems I now triggered this on CI in #408 so it's no longer exclusive to openSUSE Tumblweed.

Cool, let's do it then, thanks! 🚀


Add brightness filter

Hello! Tested the post processing Brightness Filter and it works well. This PR adds the Brightness Filter options for the ros driver.

Hi, thanks for the PR, I'm going to merge it as soon as v2.21.2 of depthai is available on official apt.

@doisyg Will be available in the next release


Alerts and Timeline not showing

Question
What would you like to ask?
Im new on using Cactbot and ACT for a while. My Cactbot just like the title I said does not show up in anywhere, whenever i bow to dummy or countdown its still not working. but shirk and provoke work successfully. What can I do to fix it?

I would recommend asking the ffxiv act discord.
Here's a few quick possibilities.  The likely answer here is that you something is wrong with your chat.  Do you have the system log messages show up in your chat log? if no, turn them on so you see the countdown/engage there.  Do you have the "hide chat log for privacy" clicked in your ffxiv settings? If so, turn that off.  Are you using dalamud plugins?  Try without quicklauncher.  If that works, then it's some chat-related plugin you need to disable.

This is a pretty old bug with no response to any questions.  If this is still not working for you, I would recommend asking in the ffxiv act discord.  I hope that you have gotten this sorted.

@quisquous was running into the same issue, and your pointer to a chat mod was on the money.  In my case, TidyChat, which filters, among other things, system message.  However, the odd part is that the Alerts portion work even with TidyChat, while the timers do not.  I ended up fixing it by specifically whitelisting the <Area> will be sealed off in 15 seconds! message under Custom Filters in TidyChat.  So if anyone else has this issue and runs across (or is pointed at) this issue, if you have TidyChat or any other mod that filters chat messages, ensure that the area seal-off messages are whitelisted.


Q: bookmarklet?

Is it possible to make a bookmarklet that injects this lib into the page, and a Save button that exports the modified DOM/HTML?

Yes absolutely, the editor has a paste method which can be monkey patched for example:
# Monkey patch editors paste method
editorCls = ContentTools.EditorApp.getCls()
editorCls.paste = (element, clipboardData) ->
    ... add you custom handler here ...

element is the element being pasted into, and clipboardData is the content pasted by the user (https://developer.mozilla.org/en-US/docs/Web/API/ClipboardEvent/clipboardData).
You will need to use the develop branch as I modified the source to ensure the clipboardData is sent to the paste event, previously only a plain text version of the content was sent.

It seems, that @anthonyjb answered #7 issue?

@MalyshevDmitry thanks - yep I got that wrong, that's the answer for #7 will move it now.

@crazy4groovy I don't see any reason why not - however you need some way of marking the content that your wanted to be able to change; you couldn't just start the editor, make the entire page editable and then save the HTML out. Either the page's HTML would need to have marked editable regions in a way that your bookmarklet is set to look for (e.g data-editable="region-name"), or you'd need to provide a mechanism by which the user could configure the bookmarklet before applying it (for example perhaps a way in which the user could click on the elements they want to be able to edit).

@anthonyjb Yup that's true, I was thinking something very basic like a prompt box from the bookmarklet that asks for node(s) the used wants to edit.
eg. which nodes: p,div

test


Package TFA 0.5.2 pinned to TF2-RC2

RC2 is released so we can do a minor release pinned to this. For now this is blocked until #539 is merged.

As #532 has been fixed by #539, we'll plan to release 0.5.2.


Ability to clear errors?

Because eachError is backed by a ConflatedBroadcastChannel, it'll replay errors, but I might not be concerned with any previous error values. It would be nice if we could either clear the error, or have this work like a flow and "drop" the first error. Also, this seems related to issue https://github.com/asarazan/bismarck/issues/13, and could be solved by adding the ability to manually update the error value

From a philosophical standpoint, a system should be considered "in error" for the entire time that it cannot acquire a new value. Therefore the only "true" way to clear an error is to execute a successful fetch.
Right now I'm leaning toward restricting any external control over errors because it might confuse or dilute their meaning.


Support arm64 on macOS

Fixes #674
Add support for arm64 on macOS
Signed-off-by: The MathWorks, Inc<EMAIL_ADDRESS>

Codecov Report

Merging #778 (d238b9a) into development (e084394) will decrease coverage by 0.99%.
The diff coverage is n/a.

Additional details and impacted files

@@               Coverage Diff               @@
##           development     #778      +/-   ##
===============================================
- Coverage        88.55%   87.56%   -1.00%
===============================================
  Files              220       92     -128
  Lines            12039     6432    -5607
===============================================
- Hits             10661     5632    -5029
+ Misses            1378      800     -578

Impacted Files
Coverage Δ

...rk/include/cppmicroservices/detail/WaitCondition.h
71.42% <0.00%> (-21.91%)
:arrow_down:

framework/src/bundle/BundleRegistry.cpp
79.79% <0.00%> (-10.43%)
:arrow_down:

framework/src/util/FrameworkPrivate.cpp
70.77% <0.00%> (-4.35%)
:arrow_down:

framework/src/service/ServiceEvent.cpp
71.05% <0.00%> (-3.95%)
:arrow_down:

framework/src/util/FrameworkEvent.cpp
61.11% <0.00%> (-3.51%)
:arrow_down:

framework/src/bundle/BundleResourceBuffer.cpp
68.31% <0.00%> (-3.00%)
:arrow_down:

framework/src/util/Properties.cpp
53.93% <0.00%> (-2.46%)
:arrow_down:

framework/src/bundle/BundleEvent.cpp
84.21% <0.00%> (-2.36%)
:arrow_down:

framework/src/bundle/BundleUtils.cpp
78.12% <0.00%> (-1.88%)
:arrow_down:

framework/src/util/LDAPProp.cpp
90.69% <0.00%> (-1.76%)
:arrow_down:

... and 189 more


How do you get your current location

Hi @libaipeng1314 ,
When opening an issue, please try to provide all or some of the information requested in the issue template. Also note that if you need help with your code please try using one of the forums at https://flutter.dev/community

Issue Template
If you are looking for support, please check out our documentation
or consider asking a question on Stack Overflow:
- https://flutter.dev/
- https://api.flutter.dev/
- https://stackoverflow.com/questions/tagged/flutter?sort=frequent

If you have found a bug or if our documentation doesn't have an answer
to what you're looking for, then fill our the template below. Please read
our guide to filing a bug first: https://flutter.dev/docs/resources/bug-reports

Steps to Reproduce

Please tell us exactly how to reproduce the problem you are running into.

Please attach a small application (ideally just one main.dart file) that
reproduces the problem. You could use https://gist.github.com/ for this.

If the problem is with your application's rendering, then please attach
a screenshot and explain what the problem is.

1. ...
2. ...
3. ...

Please tell us which target platform(s) the problem occurs (Android / iOS / Web / macOS / Linux / Windows)
Which target OS version, for Web, browser, is the test system running?
Does the problem occur on emulator/simulator as well as on physical devices?

Target Platform:
Target OS version/browser:
Devices:

Run your application with `flutter run --verbose` and attach all the
log output below between the lines with the backticks. If there is an
exception, please see if the error message includes enough information
to explain how to solve the issue.

Run `flutter analyze` and attach any output of that command below.
If there are any analysis errors, try resolving them before filing this issue.

Finally, paste the output of running `flutter doctor -v` here.

Closing this issue, since no information has been provided.
Thank you.


Package with Unicode characters is not searched correctly using the gallery

Although the package was indexed correctly to Lucene, gallery keeps showing this message:

Link to package: https://dev.nugettest.org/packages/Z̡̜͍̈̍̐̃̊͋́A̜̣͍̬̞̝͉̽ͧ͗L̸̖͕̤̠̹̘͖̃̌ͤG͓̝͓̰̀ͪO͈͌/

Looks like an encoding mismatch between gallery and search service.
Gallery encodes the package id as: Z%CC%88%CC%8D%CC%90%CC%83%CC%8A%CD%8B%CC%81%CC%A1%CC%9C%CD%8DA%CC%BD%CD%A7%CD%97%CC%9C%CC%A3%CD%8D%CC%AC%CC%9E%CC%9D%CD%89L%CC%83%CC%8C%CD%A4%CC%B8%CC%96%CD%95%CC%A4%CC%A0%CC%B9%CC%98%CD%96G%CC%80%CD%AA%CD%93%CC%9D%CD%93%CC%B0O%CD%8C%CD%88
Search service encodes the package id as: Z%CC%A1%CC%9C%CD%8D%CC%88%CC%8D%CC%90%CC%83%CC%8A%CD%8B%CC%81A%CC%9C%CC%A3%CD%8D%CC%AC%CC%9E%CC%9D%CD%89%CC%BD%CD%A7%CD%97L%CC%B8%CC%96%CD%95%CC%A4%CC%A0%CC%B9%CC%98%CD%96%CC%83%CC%8C%CD%A4G%CD%93%CC%9D%CD%93%CC%B0%CC%80%CD%AAO%CD%88%CD%8C
I checked the database and the search service. Database
The bigger question here is: do we want to allow non-ASCII/non-UTF-8 encodings for package id's? @yishaigalatzer
If so, this will require some work to ensure all systems use the same encoding (and thus search results return a correct value here)

Looked a bit deeper into this one, seems normalizing the string works. Here's a PR: https://github.com/NuGet/NuGetGallery/pull/3188


Edition UI fixes

I'm ready to merge this even though its failing.  I've got a separate branch with the TcpListener fix.


Specs not working

I thought I followed all the instructions.

require guard/compat/plugin here: https://github.com/socketry/guard-falcon/blob/master/lib/guard/falcon/controller.rb#L21
setup testing here: https://github.com/socketry/guard-falcon/blob/master/spec/spec_helper.rb#L21-L22

But specs still failing
% rspec
/home/samuel/Documents/Programming/socketry/guard-falcon/lib/guard/falcon/controller.rb:56: warning: assigned but unused variable - options
/home/samuel/.rvm/gems/ruby-2.4.0/gems/async-io-0.5.0/lib/async/io/socket.rb:59: warning: shadowing outer local variable - task
/home/samuel/.rvm/gems/ruby-2.4.0/gems/async-http-0.2.0/lib/async/http/protocol/http1x.rb:51: warning: assigned but unused variable - method
/home/samuel/.rvm/gems/ruby-2.4.0/gems/async-http-0.2.0/lib/async/http/protocol/http1x.rb:51: warning: assigned but unused variable - path

Guard::Falcon::Controller
  should start server (FAILED - 1)

Failures:

  1) Guard::Falcon::Controller should start server
     Failure/Error: logger.info("Starting Falcon HTTP server on #{@options[:bind]}.")

     NoMethodError:
       undefined method `info' for Guard::UI:Module
     # /home/samuel/.rvm/gems/ruby-2.4.0/gems/guard-compat-1.2.1/lib/guard/compat/plugin.rb:73:in `info'
     # ./lib/guard/falcon/controller.rb:62:in `run_server'
     # ./lib/guard/falcon/controller.rb:76:in `start'
     # ./spec/guard/falcon/controller_spec.rb:30:in `block (2 levels) in <top (required)>'

Finished in 0.00069 seconds (files took 0.08173 seconds to load)
1 example, 1 failure

Failed examples:

rspec ./spec/guard/falcon/controller_spec.rb:29 # Guard::Falcon::Controller should start server

So, I added require 'guard' to spec_helper.rb and it worked. Because, guard/compat/plugin/helper define Guard::UI but empty, and this breaks Guard::Compat::UI.


Do not make assumptions about files in HMRC PAYE task

The documented list of files does not match the reality.
This also makes it much easier to load all the current files into the asset manager:
for file in *; do
    sudo -udeploy govuk_setenv asset-manager bundle exec rake govuk_assets:create_hmrc_paye_asset[$file]
done

Trello card

I've only made it do all zip files, as the xml file needs its file name changing when uploaded.

create_or_replace_whitehall_asset already prints out on-disk filenames and legacy URL paths as it uploads things, so I don't think there's a need for extra logging


General cleanup

I combined a few general cleanup commits into this PR. There are more that can be done, in particular removing now-unused functions in datatype/sigfig.rs.
Do you have plans for DecimalSplitsList? As is, it's not used at all, and as a result a lot of DecimalSplits methods are now unused as well, resulting in some warnings from the compiler. Can we remove those entirely? If not, we can add #![allow(unused)] in the module to silence the warnings for now.
Feedback welcome if I'm stepping on your toes with any of these, but they seem like good changes to me.
Thanks!

Also, I think you are right. DecimalSplitsList can be removed.


Schematics: Add markdown navigator component

Do you want to request a feature or report a bug?
Feature Request
Now that help is going to be it's own package as part of #1485, we need to add it to Schematics
What is the expected behavior?
Help package would be part of schematics.

Addressed in #1504


No Pointer style cursor for table head when no hint or sorting

With no sorting or a hint, the cursor still shows a pointer. I'm not saying this is the best solution, but I think the header shouldn't shouldn't show a pointer style of cursor when there is nothing to click on or additional behavior from hovering.

Coverage increased (+0.03%) to 76.442% when pulling 4ada7311ae320fbf862a6ed301f0c40d168ea057 on lalong13:patch-1 into 3032a7c22cdbbafbc53409d797a179215a40f6ad on gregnb:master.

@gabrielliwerant Would you be looking to get this change in soon? Is there anything I need to do with this? I imagine you may want to do this change in less of a hacky type fix, but I can make changes to the PR to help get it in faster.

@lalong13 Just about to look at a few PRs right now. Will keep you updated.


Extracts user info collection to a universal helper function.

What's this PR do?
This PR updates the dosomething_helpers_get_current_user() function and adds a dosomething_helpers_get_user_info($drupal_id) function. Now we can call the former function and if there is a current user logged in, then it will pass the drupal id to the latter function and actually retrieve all the user information needed from our Northstar user API, process the info a bit to only show what is needed and then return it as an array.
This is used in the new dsUser object that is located in the Drupal.settings JS object on the front-end, but can also be used for DRYing up the current code in the Phoenix API when the user object is created here, here, and here.
The dsUser object currently has 2 properties, one for info with the processed user info from NS, and activity, which is currently null but could include activity info for the user in the future as suggested by @sbsmith86. This could be done with a separate helper function call.
How should this be reviewed?
Try it out. Load a campaign, open the dev console and type Drupal.settings and hit return. Do you see the new dsUser object with info if signed in? Also, did Kudos break?! Hopefully not!
Any background context you want to provide?
We should consolidate this and the code here and only have one user object in the Drupal.settings.
Relevant tickets
Refs #6682
Checklist

[ ] Documentation added for new features/changed endpoints.
[ ] Tested on staging.
[ ] Pinged a PM if this is a larger PR that would benefit from some additional testing love.

@DFurnes @deadlybutter @angaither
cc: @sbsmith86 @katiecrane

Two subjective suggestions. The rest looks good to me! 👍

🚢!


Text nodes are cut

On one of html files I requested to a website, some html text nodes were cut.
I don't understand why is this happening, mostly because requests is not only for HTML requests right? It doesn't parse HTML. And I didn't have the problem earlier.
Expected Result
when requested with Chrome I got

<img src="http://<IP_ADDRESS>/view/image/icons/chatattached_read.png" class="icon" /> [Arch][Beta] Eriah 73

<img src="http://<IP_ADDRESS>/view/image/icons/chatattached_read.png" class="icon" /> Modyfikacje do Sargonii

<img src="http://<IP_ADDRESS>/view/image/icons/chatattached_read.png" class="icon" /> [Beta] Profile Borumus

Note the text nodes: [Arch][Beta] Eriah 73, Modyfikacje do Sargonii, [Beta] Profile Borumus
Actual Result
Requests (both .text and .content) returned

<img src="http://<IP_ADDRESS>/view/image/icons/chatattached.png" class="icon" /> [Arch][Beta] Eria...

<img src="http://<IP_ADDRESS>/view/image/icons/chatattached.png" class="icon" /> Modyfikacje do Sa...

<img src="http://<IP_ADDRESS>/view/image/icons/chatattached.png" class="icon" /> [Beta] Profile Bo...

Note the text nodes: [Arch][Beta] Eria..., Modyfikacje do Sa..., [Beta] Profile Bo...
Reproduction Steps
This script works, just note that you need first to create an account ("start game" button), otherwise game won't let you access any content. If you login in Chrome and request the url, you'll get the expected, uncut result. If you run the code below, printed result will have cut text nodes.
import requests

session = requests.Session()
session.post('http://mrush.pl/login', {"name":"NICKNAME", "password":"PASSWORD"})
r = session.get("http://mrush.pl/threads?id=8993&page=1")
print(r.text)

System Information
PyCharm, Windows 10, Python 3.6 with

Package
Version
Latest

certifi
2018.1.18
2018.1.18

chardet
3.0.4
3.0.4

colorama
0.3.9
0.3.9

cssselect
1.0.3
1.0.3

idna
2.6
2.6

lxml
4.1.1
4.1.1

pip
9.0.1
9.0.1

pyquery
1.4.0
1.4.0

requests
2.18.4
2.18.4

setuptools
28.8.0
38.5.2

urllib3
1.22
1.22

Hi there! Thanks for opening this issue. Unfortunately, it seems this is a request for help instead of a report of a defect in the project. Please use StackOverflow for general usage questions instead and only report defects here.
It's entirely likely that you need to use a different User-Agent header to get back the information you need. The server may be trying to stop scrapers.

@sigmavirus24  This is not a request for help and this has nothing to do with me. All my debugs and checks shown that it's an issue of requests.

@Soaku as @Sigmavirus24 has said, this has to be something on the server side. Requests does no modification of inline content, so the fact you’re still receiving all results but the values are changing means the server is choosing to do something different.
The suggested User-Agent change will likely work and if you need further assistance, please open a question on StackOverflow. Thanks!

@nateprewitt Your reply was a bit late, because I've already figured out the solution, as said in my previous post


Notifications command

Adds new command SET_NOTIFICATION that displays or clears notifications in apps dock

Main branch dependencies

Neodora Analysis ⚠
 ⚠ Missing cache in Github Workflow

Message: Github workflow should be using caching when step uses actions/setup-node

Workflow file
Job name
Step index
Actual cache value
Expected cache value

on-commit.yml
lint
2
Not found
npm

🔍 How to fix it

 ⚠ Incorrect runner in Github Workflow

Message: Github workflow must run on a self-hosted runner

Workflow file
Job name
Actual runs-on value
Expected runs-on value

on-commit.yml
lint
ubuntu-latest
Any of [eks-runner, eks-runner-php, eks-runner-nano, eks-runner-debug, eks-runner-compute]

🔍 How to fix it

 ⚠ Unknown tribe name

Message: The property owner in repository.json must be a valid tribe name present in mission-tracking
Repository file: repository.json

Owner status

'GRAVITY' is an unknown tribe name

🔍 How to fix it

Commit 815f27e  Job 110744


arch-install-scripts: 26 -> 27

Description of changes

This update introduces -K option to pacstrap, which initializes an empty pacman keyring in the target root instead of copying the existing keyring from the host. This option won't work without https://github.com/NixOS/nixpkgs/pull/192896 or manually adding pacman package to PATH.
However, even with the above change applied, I found pacstrap -K not automatically populating archlinux-keyring after installing it to the new root, and I had to manually run pacman-key --populate before installing packages in the chroot. :confused:
UPDATE: Using pacstrap without -K doesn't trigger pacman-key --populate either, but it'll copy the gpgdir from the host to the target, so I didn't notice this problem before. Maybe I'll open an issue to track it later.
My bad! That was because I added etc/pacman.d/mirrorlist to NoExtract in my pacman.conf.
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1323


Add lapfoxtrax.com to supported sites.

That's pretty the same with bandcamp but has no "bandcamp" in url.

This is a bandcamp premium account using a custom domain. See #334 for more information of a workaround.

Understood. Thank you.


Convert route params demo to use ES5

Fixes #112

This change is

Not sure if related, but we have two more files with new syntax (used in lazy loading):
https://github.com/vaadin/vaadin-router/blob/master/demo/demo-elements/component-definition.html
https://github.com/vaadin/vaadin-router/blob/master/demo/demo-elements/component-definition.js

Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable


Fix NullPointerException on swapCursor for IterableCursorAdapter

Fixes #10

Hey guys, any chance to promote it to Central?

Ah, yes, I thought this was included in 0.3. I'll push 0.4 now

0.4 is pushed (#18), should be synced with Central soon.

Thanks!


Add list of certificate secrets to containerjfr CRD

Fixes #142

Its working with tls certificates. If the certificate is not supported by container jfr (or improperly formatted) then cjfr fails to import it from the truststore.

Its working with tls certificates. If the certificate is not supported by container jfr (or improperly formatted) then cjfr fails to import it from the truststore.

Something else I am not sure about is wether the certificateKey field should be optional. If we do want it to be optional then I probably need to add a check to see if its null, in which case there'd be no subpath in the volumemount.

Something else I am not sure about is wether the certificateKey field should be optional. If we do want it to be optional then I probably need to add a check to see if its null, in which case there'd be no subpath in the volumemount.

Something else I am not sure about is wether the certificateKey field should be optional. If we do want it to be optional then I probably need to add a check to see if its null, in which case there'd be no subpath in the volumemount.

If not given, we should use the default value specified in #142.

Something else I am not sure about is wether the certificateKey field should be optional. If we do want it to be optional then I probably need to add a check to see if its null, in which case there'd be no subpath in the volumemount.

If not given, we should use the default value specified in #142.

So I just fixes those issues, and I added two tests, I can't seem to figure out why the second one is not working (it's the one that actually tests if the reconciler works). Im thinking it might be an issue with changing the runtime objects mid test? If you think of anything let me know!

Looking at the tests again, I think it would be more valuable if we specify exactly what we expect the VolumeMounts to be. As it is now, we have no way of knowing that NewDeploymentForCR is creating them properly and handling the default key as expected. You could add a NewVolumeMountsWithSecrets (maybe in test/resources.go) that have the exact VolumeMounts you expect to see produced for test.NewContainerJFRWithSecrets. Then use NewVolumeMountsWithSecrets as the expected value to compare against the VolumeMounts created by the reconciler.

Yeah that makes sense!

Yes, my bad, for some reason I lost it in the midst of the other temporary files I didn't want to commit!


fix: remove node-fetch from postinstall script

What kind of change does this PR introduce?
Bug fix https://github.com/supabase/cli/issues/1769
What is the current behavior?
Please link any relevant issues here.
What is the new behavior?
Feel free to include screenshots if it includes visual changes.
Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 58.295%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6005

Relevant Lines:
10301

💛  - Coveralls


Added content type footnotes+xml so mail merge fields are supported

Added a 1 line change on CONTENT_TYPES, that allows the replacing of a mail-merge field on the footnotes of a docx.

Description

Added a 1 line change on CONTENT_TYPES, that allows the replacing of a mail-merge field on the footnotes of a docx.
"application/vnd.openxmlformats-officedocument.wordprocessingml.footnotes+xml",

Motivation and Context
Thes change allows for the merge of fields inside footnotes.

How Has This Been Tested?

Created a small testing script:

from io import BytesIO
from mailmerge import MailMerge

def mergeDocumentToLocalFile(templatePath, outputPath, mergeFields):
    print(f"Using template - {templatePath}")
    with open(templatePath, "rb") as file:
        template = file.read()
    buffer = BytesIO(template)
    with MailMerge(buffer) as document:
        document.merge(**mergeFields)
        output_filelike = BytesIO()
        document.write(output_filelike)
        # need to reset the stream before writing to file
        output_filelike.seek(0)
        with open(outputPath, "wb") as output_file:
            output_file.write(output_filelike.read())
        print("File saved to", outputPath)

Used multiple documents to test with and without footnotes.
Run the script with multiple documents

Screenshots (if appropriate):
Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Thanks for the fix. I will add it as it is, but I would have liked a test file and test script in the tests/ folder so it could be tested.

well, did you test it ? I get 10 unittest errors.

Post it as an issue with an example docx and I will try to fix it in a way that doesn't break other files.


Disable Search in Tables

Hey,
is it possible to disable the search field in tables, too? I just found something about disabling th buttons, but the search field stays.
Thanks

Kind of obvious -DisableSearch :-)
        New-HTMLTable -ArrayOfObjects $Processes -HideFooter { #-ImmediatelyShowHiddenDetails
            New-TableRowGrouping -Name 'PriorityClass' -Color Red -BackgroundColor GreenYellow
        } -DisableSearch

lol.. thanks a lot!
I tried stuff like "-Search @()", "-Search $False" and "-HideSearch" -.-

https://github.com/EvotecIT/PSWriteHTML/blob/master/Docs/New-HTMLTable.md - all options (i think it's up to date).


boundary-policy provided in all the sample configurations doesn't prevent break-glass-user to access other services without MFA

Describe the bug
The boundary-policy provided in the sample configuration doesn't prevent IAM users to access other AWS services without MFA.
I created the break-glass-users with the default boundary-policy in the Management Account, and I logon with the break-glass-user and was able to access other AWS services and assumed the role in other member accounts without MFA.
To Reproduce
Steps to reproduce the behavior.
Expected behavior
break-glass-user with boundary-policy should not be able to access other services except for those listed in the NotAction.
Please complete the following information about the solution:

[ ] Version: [e.g. v1.1.0]
v1.3

To get the version of the solution, you can look at the description of the created CloudFormation stack. For example, "(SO0021) - Video On Demand workflow with AWS Step Functions, MediaConvert, MediaPackage, S3, CloudFront and DynamoDB. Version v5.0.0". If the description does not contain the version information, you can look at the mappings section of the template:
Mappings:
  SourceCode:
    General:
      S3Bucket: "solutions"
      KeyPrefix: "video-on-demand-on-aws/v5.0.0"

[ ] Region: [e.g. us-east-1]
[ ] Was the solution modified from the version published on this repository?
[ ] If the answer to the previous question was yes, are the changes available on GitHub?
[ ] Have you checked your service quotas for the sevices this solution uses?
[ ] Were there any errors in the CloudWatch Logs?

Screenshots
If applicable, add screenshots to help explain your problem (please DO NOT include sensitive information).
Additional context
I updated the boundary-policy with following changes:
(1) Removed "iam:DeactivateMFADevice"
(Users with this policy might attempt to assign themselves a virtual MFA device and receive an error that they are not authorized to perform "DeactivateMFADevice".)
(2) Updated "Resource": "arn:aws:iam:::user/${aws:username}" to "Resource": ""
(With the original Resource scope, user can access to other serivces without MFA)
(3) Updated "Bool" to "BoolIfExists"
(According to AWS Document, "Bool" is not recommended to use with "MultiFactorAuthPresent:false" & "Deny" --> "This applies only to temporary credentials that support using MFA. This statement does not deny access to requests that are made using long-term credentials, or to requests that are authenticated using MFA. Use this example with caution because its logic is complicated and it does not test whether MFA-authentication was actually used." And "BoolIfExisits" is recommended --> "it denies requests from temporary credentials that do not include MFA. It also denies requests that are made using long-term credentials, such as AWS CLI or AWS API operations made using access keys.")
{
"Version": "2012-10-17",
"Statement": [
{
"Effect": "Allow",
"Action": "",
"Resource": ""
},
{
"Sid": "DenyAllExceptListedIfNoMFA",
"Effect": "Deny",
"NotAction": [
"iam:ChangePassword",
"iam:GetUser",
"iam:GetLoginProfile",
"iam:ListMFADevices",
"iam:CreateVirtualMFADevice",
"iam:EnableMFADevice",
"iam:ResyncMFADevice"
],
"Resource": "*",
"Condition": {
"BoolIfExists": {
"aws:MultiFactorAuthPresent": "false"
}
}
}
]
}

Hi @LawLaw443,
I wanted to see if you could point me to which specific best-practices configuration file set you are seeing this behavior in?
Cheers,
Ryan

Hi, here's the configuration file: https://github.com/awslabs/landing-zone-accelerator-on-aws/tree/main/reference/sample-configurations/aws-best-practices-cccs-medium/iam-policies)/iam-user-boundary-policy.json


Replace fresnel with react-responsive

Description
This PR replaces fresnel with react-responsive for media queries.
Related issue
N/A
Motivation and Context
There are a number of issues that have arisen recently because of the intermediate components that fresnel creates. This PR attempts to begin the process of fixing them.
How Has This Been Tested?
N/A
Screenshots (if appropriate):

On mobile, the /dashboard/markets page Search bar is shown in full screen size
Also weird behaviour on market page dropdown with the loading spinner (same happens on Exchange page when selecting token list or leaderboard when searching):

https://github.com/Kwenta/kwenta/assets/548702/950f0f4d-c8d7-4152-9699-6bd9d2d6774a


why the code is so complex?

I have a idea and want to complete it based on code of dense caption.I have read the paper and the code.I find it very complex and hard to understand.Who can help me?

Did you find any help?


Move documentation to docs repo

:+1:

Oh. I've made a mistake last night. I should not have merged this until the https://github.com/php-http/documentation/pull/93 (and https://github.com/php-http/documentation/pull/94) had been merged.
Sorry about that.

Np, this is to smaller problem. The other way around would have been bigger. :wink:


Add jet-live

NOTE: Each discrete (stand-alone) request should be in its own issue.
Put "Add ____" as Title
[x] Done
Provide a link to the proposed addition.
https://github.com/ddovod/jet-live
Explain why it should be added
The library allows to reload c++ (and probably obj-c, I didn't test it) code on the fly without app restart.
Additional context
Here's a demo video, was created under linux, but the library provides same experience on mac.
Issue Checklist
Put an x in the boxes once you've completed each step. You can also fill these out after creating the issue. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before responding to the issue.

[x] I have checked for other similar issues
[x] I have explained why this change is important
[x] I have added necessary documentation (if appropriate)

@ddovod This may not be suitable for this list.

Okay, no problem


Generating skipping some NFTs

When generating wether it’s 100 1000 10000 i the program will produce images perfectly however some of them get skipped and end up looking like this

Any ideas as to why it could be skipping some nfts and hoe to fix it would be greatly appreciated all my exports are framed and transparent as well as png so it must be of coding error or trying to produce images too fast.

every layer is shown on multiple nfts but then on some slides they come out like this in build file


Do not rely on autoincremental ids in tests

Subject

I am targeting this branch, because {reason}.

Closes #{put_issue_number_here}.
Changelog

### Added
- Some `Class::newMethod()` to do great stuff

### Changed

### Deprecated

### Removed

### Fixed

### Security

Codecov Report
Patch and project coverage have no change.

Comparison is base (7b9e77a) 86.45% compared to head (4f7c449) 86.45%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff            @@
##                1.x       #4   +/-   ##
=========================================
  Coverage     86.45%   86.45%
  Complexity      480      480
=========================================
  Files            31       31
  Lines          1691     1691
=========================================
  Hits           1462     1462
  Misses          229      229

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


Code workflow

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


How can I expand PVC? Please help me

information:
[root@lu csi]# kubectl get sc,pvc,pv
NAME                                     PROVISIONER        RECLAIMPOLICY   VOLUMEBINDINGMODE   ALLOWVOLUMEEXPANSION   AGE
storageclass.storage.k8s.io/csi-rbd-sc rbd.csi.ceph.com   Delete          Immediate           true                   20m
NAME                            STATUS   VOLUME                                     CAPACITY   ACCESS MODES   STORAGECLASS   AGE
persistentvolumeclaim/rbd-pvc   Bound    pvc-aa59999c-bd55-44d5-ba19-80d6f8bec547   1Gi        RWO            csi-rbd-sc     20m
NAME                                                        CAPACITY   ACCESS MODES   RECLAIM POLICY   STATUS   CLAIM             STORAGECLASS   REASON   AGE
persistentvolume/pvc-aa59999c-bd55-44d5-ba19-80d6f8bec547   1Gi        RWO            Delete           Bound    default/rbd-pvc   csi-rbd-sc              20m
when i kubectl edit pvc do change 1G to 2G,but no success,I found it by checking kubectl describe pvc:
Normal   ExternalProvisioning   17m                  persistentvolume-controller                                                                      waiting for a volume to be created, either by external provisioner "rbd.csi.ceph.com" or manually created by system administrator
Normal   Provisioning           17m                  rbd.csi.ceph.com_csi-rbdplugin-provisioner-59957d9fc-2xkkh_73fd7fd5-4001-4ab7-a6d3-38f619b79886  External provisioner is provisioning volume for claim "default/rbd-pvc"
Normal   ProvisioningSucceeded  17m                  rbd.csi.ceph.com_csi-rbdplugin-provisioner-59957d9fc-2xkkh_73fd7fd5-4001-4ab7-a6d3-38f619b79886  Successfully provisioned volume pvc-aa59999c-bd55-44d5-ba19-80d6f8bec547
Warning  ExternalExpanding      16m                  volume_expand                                                                                Ignoring the PVC: didn't find a plugin capable of expanding the volume; waiting for an external controller to process this PVC.
Normal   Resizing               3m5s (x12 over 16m)  external-resizer rbd.csi.ceph.com                                                                External resizer is resizing volume pvc-aa59999c-bd55-44d5-ba19-80d6f8bec547
Warning  VolumeResizeFailed     3m5s (x12 over 16m)  external-resizer rbd.csi.ceph.com                                                                resize volume pvc-aa59999c-bd55-44d5-ba19-80d6f8bec547 failed: rpc error: code = Internal desc = provided secret is empty

do you have controller expand secrets in your storageclass? if no please add it to your storage class and recreate it and create a new PVC and try to resize it

closing this as not a issue

@Madhu-1  thank you very much for this issue. and i also have a problem,How to migrate PV，PVC created by different storageclasses，Details are as follows：
source storageclass
NAME   PROVISIONER    RECLAIMPOLICY   VOLUMEBINDINGMODE   ALLOWVOLUMEEXPANSION   AGE
slow   ceph.com/rbd   Retain          Immediate           true                   27d
destination storageclass
NAME   PROVISIONER  RECLAIMPOLICY   VOLUMEBINDINGMODE   ALLOWVOLUMEEXPANSION   AGE
csi-rbd-sc   rbd.csi.ceph.com   Delete          Immediate           true                   3d17h


Build with CMake gives multiple errors.

Hi team
I have been trying to build this library for several hours. I am a bit new to CMake. I cant find the correct flags to set, and keep getting errors.
CMake Error at CMakeLists.txt:15 (message):
  Prevented in-tree built.  Please create a build directory outside of the
  SDL_rtf source code and call cmake from there

-- Configuring incomplete, errors occurred!

This is why I thought I had to sh ./configure

I am setting paths manually with SET from the CMD promp and pointing cmake at the gcc g++ compilers
SET PATH=%PATH%;C:\Dev-Cpp-Emb_Winlibs\TDM-GCC-64\bin;C:\Dev-Cpp-Emb_Winlibs\SDL2\SDL_rtf-SDL2
cmake -DCMAKE_C_COMPILER=C:/Dev-Cpp-Emb_Winlibs/TDM-GCC-64/bin/gcc.exe -DCMAKE_CXX_COMPILER=C:/Dev-Cpp-Emb_Winlibs/TDM-GCC-64/bin/g++.exe C:\Dev-Cpp-Emb_Winlibs\SDL2\SDL_rtf-SDL2

I am getting the CMakeCache.txt file being generated but no further.
Regard
Axle

Sorry if I am asking in the wrong place. I will move my questions to the forum.
My bad...

This is what I have just tried :(
From both in the source dir as well as from ..\build\build.cmd
I also set the path to the build directory and the source directory
SET PATH=%PATH%;C:\Dev-Cpp-Emb_Winlibs\TDM-GCC-64\bin;C:\Dev-Cpp-Emb_Winlibs\SDL2\Build
as well as trying
SET PATH=%PATH%;C:\Dev-Cpp-Emb_Winlibs\TDM-GCC-64\bin;C:\Dev-Cpp-Emb_Winlibs\SDL2\SDL_rtf-SDL2

Maybe the compiler paths are incorrect.
I assumed -DCMAKE_C_COMPILER= to be a generic VALUE for CMake. I will remove and try again, but previously CMake defaults to the MS compiler instead of GCC

OK, had to remove the previous cache files.
Still missing something though...
tree:
SDL2
--SDL2\build\build.cmd
--SDL2\SDL_rtf-SDL2*.c;*.h
C:\Dev-Cpp-Emb_Winlibs\SDL2\Build>cmake  -G "MinGW Makefiles" -DCMAKE_C_COMPILER=C:/Dev-Cpp-Emb
cc.exe -DCMAKE_CXX_COMPILER=C:/Dev-Cpp-Emb_Winlibs/TDM-GCC-64/bin/g++.exe C:\Dev-Cpp-Emb_Winlib
-- The C compiler identification is GNU 13.2.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: C:/Dev-Cpp-Emb_Winlibs/TDM-GCC-64/bin/gcc.exe - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Configuring SDL2_rtf 2.0.0
-- Using private SDL2 find module
CMake Error at cmake/PrivateSdlFunctions.cmake:63 (find_package):
  By not providing "FindPrivateSDL2.cmake" in CMAKE_MODULE_PATH this project
  has asked CMake to find a package configuration file provided by
  "PrivateSDL2", but CMake did not find one.

  Could not find a package configuration file provided by "PrivateSDL2"
  (requested version 2.0.0) with any of the following names:

    PrivateSDL2Config.cmake
    privatesdl2-config.cmake

  Add the installation prefix of "PrivateSDL2" to CMAKE_PREFIX_PATH or set
  "PrivateSDL2_DIR" to a directory containing one of the above files.  If
  "PrivateSDL2" provides a separate development package or SDK, be sure it
  has been installed.
Call Stack (most recent call first):
  CMakeLists.txt:69 (sdl_find_sdl2)

-- Configuring incomplete, errors occurred!

Depends on SDL2.dll :/

You need SDL2 and SDL2_ttf for SDL2_rtf.
So download/build those before.
Use -DCMAKE_PREFIX_PATH=... to inform CMake about the location of the dependencies.

OK, got a successful build. So all good :)
Yes, I just realized I didn't put the 2 above depends paths into the batch file.
showrtf.exe runs and asks for a *.ttf so All good.
Thank you very much for the assist, even if it was a bit of a rubber duck moment lol
I am still a bit green with this cmake stuff lol
kind regards
Axle

P.S. in my case it ended up being build.cmd:
SET PATH=%PATH%;C:\Dev-Cpp-Emb_Winlibs\TDM-GCC-64\bin;C:\Dev-Cpp-Emb_Winlibs\SDL2\SDL2-2.30.0\x86_64-w64-mingw32\include\SDL2;C:\Dev-Cpp-Emb_Winlibs\SDL2\SDL2-2.30.0\x86_64-w64-mingw32\bin;C:\Dev-Cpp-Emb_Winlibs\SDL2\SDL_rtf-SDL2
cmake  -G "MinGW Makefiles" -DCMAKE_C_COMPILER=C:/Dev-Cpp-Emb_Winlibs/TDM-GCC-64/bin/gcc.exe -DCMAKE_CXX_COMPILER=C:/Dev-Cpp-Emb_Winlibs/TDM-GCC-64/bin/g++.exe C:\Dev-Cpp-Emb_Winlibs\SDL2\SDL_rtf-SDL2
mingw32-make
PAUSE


Development

I forgot to mention that it's recommended to add the schedule:run on the crontab (cron will run each minute, but the sync will be called each 30 minutes)
https://laravel.com/docs/5.8/scheduling


[Feature Request] Common开发模式支持

Checklist

[X] I searched exising issues and no one else requests similar feature.
[X] I think that 25%+ users are positive towards this feature.

Describe the feature
支持开发环境下common模块独立启动，并且不会影响应用模块
为game模块开发作为前置任务
Additional context
No response

开发环境下SPI都会加载
需要测试打包后SPI加载情况

#1020


Add normalizeConnectArgs from Node v6

The latest versions of Node don't include the net._normalizeConnectArgs method. I've included it exactly as it was in that Node.js version so that node-serf is now compatible with up to Node v8.7.0, as far as I can tell.
Awesome project by the way, thank you for your hard work! :)

Thanks, looks like it was renamed to _normalizeArgs in this commit. Either way it's better to not use an undocumented API, so +1.
Might want to add a comment attributing that function to Node.js/Node.js contributors.
Not sure what happened to circleci. It's not even queued.

Added the attribution and rebased. Didn't realise they had just changed the name, but yeah good call on that 👍
Weird with CircleCI, doesn't seem to actually be running it at all.

Thanks. Can you push to your branch again please (e.g. git commit --amend --no-edit; git push -f) to try to trigger a build? I think I fixed the CI setting, but I can't trigger the build.

Woo, CI ran 🎉 but some linter failures.
After that's fixed this looks good to me. Will leave open 'till tomorrow to given @hden a chance to look.

The function isPipeName is not defined, so we probably need to port that too.

Fixed the linter errors and added the two missing functions isPipeName and the subsequent toNumber that it uses. Should hopefully be good to go :)

LGTM


Lab BOM based on lab spending guidelines (first draft)

a stab at creating a BOM based on https://github.com/opiproject/opi-poc/pull/515
looking for feedback on:

BOM columns
budget
list of DPU/IPU that will support OPI
should each _PU have it's own server or put as many as possible in a server (cost per _PU)
1, 3 or 5 year warranty / support ?
any other feedback

On the 3-5 year support issue. In previous life managing Labs, I would always opt to not get 24X7 4-hour replacement support, and get the lowest level of support the vendor offered which was generally a 12x5 (business hours) and Next Business day part replacement. The cost savings was significant and Lab equipment almost never needs critical business system level of support. and with this being in a shared lab, we won't likely have 24x7 4 hours response time available from the Lab administrator team.

On the 1 DPU per server vs 1 Server and 8 DPUs. I would say we go with 1:1 to begin with. and have a test case that also has a single host with multiple dpus to cover the complexities of provisioning that brings. But let's start with the easiest case of deploying one dpi per server.

Has anyone looked at https://opendcim.org/ to document BOM + topology configuration details?   If all the information required by opendcim is captured in the *.md, that gives you confidence there are no gaps as far as information content is concerned.  Just a thought.

Has anyone looked at https://opendcim.org/ to document BOM + topology configuration details? If all the information required by opendcim is captured in the *.md, that gives you confidence there are no gaps as far as information content is concerned. Just a thought.

I will have a look and revise the pr if i missed anything

fyi, I created a PR to check in spending_guidelines.md in the official Policies folder of the opi repo: https://github.com/opiproject/opi/pull/45

@mgheorghe #558 was merged which fixes the YAMLlint issue.  If you push a change or rebase, that should fix it.

@mgheorghe seems like 43 commits, can you please git rebase instead of merge ?

@mgheorghe seems like 43 commits, can you please git rebase instead of merge ?
yes last night i tried to do the dco signoff and i messed something badly, working now on fixing it

@mgheorghe you can squash all commits to one now will be easier to fix DCO issues

i set the diagram to be 100% with, but the way to do it triggers the linter failure
unable to find a github compatible md way of setting image width to 100% except by using html tags

@opiproject/opi-maintainers we reviewed this PR on lab call can this be merged and future changes will be made as needed.

@opiproject/opi-maintainers we reviewed this PR on lab call can this be merged and future changes will be made as needed.

No need to merge this one, since the official one already merged into the main opi repo, here: https://github.com/opiproject/opi/blob/main/Policies/spending_guidelines.md
Am assuming there is no need to merge to opi-poc repo.  Please confirm, so I can close.

@opiproject/opi-maintainers we reviewed this PR on lab call can this be merged and future changes will be made as needed.

No need to merge this one, since the official one already merged into the main opi repo, here: https://github.com/opiproject/opi/blob/main/Policies/spending_guidelines.md
Am assuming there is no need to merge to opi-poc repo. Please confirm, so I can close.

will be good to merge it in.

@opiproject/opi-maintainers we reviewed this PR on lab call can this be merged and future changes will be made as needed.

No need to merge this one, since the official one already merged into the main opi repo, here: https://github.com/opiproject/opi/blob/main/Policies/spending_guidelines.md
Am assuming there is no need to merge to opi-poc repo. Please confirm, so I can close.

will be good to merge it in.

Ugh! Brain freeze.  Somehow I thought this PR was for spending_guidlines.md, which it is not.  Yeah, I agree this PR, which speaks to the BOM, needs to be merged in at least the opi-poc repo.

@opiproject/opi-maintainers we reviewed this PR on lab call can this be merged and future changes will be made as needed.

this PR fails DCO. please squash all commits and signoff to fix DCO issue

@opiproject/opi-maintainers we reviewed this PR on lab call can this be merged and future changes will be made as needed.

this PR fails DCO. please squash all commits and signoff to fix DCO issue

In case no one actually googled it, here are the detailed steps 8-)
Squashing a commit
In GitHub Desktop, click Current Branch.
In the list of branches, select the branch that has the commits that you want to squash.
Click History.
Select the commits to squash and drop them on the commit you want to combine them with. ...
Modify the commit message of your new commit. ...
Click Squash Commits.

and don't forget to click signoff :-)


Remove experimental from some fairly stable APIs.

WHAT
This stops callers from opting in, so remove experimental from
APIs that have been used enough to have come confidence.
WHY
Let callers opt-in to APIs like Accompanist Pager, but also our own, but only when actually changing.
HOW
Checklist :clipboard:

[x] Add explicit visibility modifier and explicit return types for public declarations
[x] Run spotless check
[x] Run tests
[x] Update metalava's signature text files

@shumelchyk will land for a release, let me know if you have any concerns.


Replace or improve blake2b-py dependency

py-evm Version: 0.3.0a19
OS: linux
Python Version: Python 3.9.0rc1

What is wrong?
py-evm depends on blake2b-py, which looks quite unmaintained. The PyPI package for blake2b-py comes with some prebuilt binaries, however for e.g. Python 3.9 or support on ARM architectures (Raspberry Pi) the only chance is to have it compiled. This requires to install the whole Rust toolchain to get it compiled (including: CI/CD stuff, there we would theoretically have to compile it for each commit!).
How it could be fixed?
For this reason I want to open the discussion, if there might be the chance to rely on another library for the required functionalities. Currently, some people including me are trying to reach the maintainer to get access to update the project, but for now there is no reaction from the maintainer.
I'm looking forward to your opinions.

Blake2 was recently added to hashlib (3.6+), and should be considered for updating this library moving forwards.

After doing some digging, I noticed that this was indeed the way this was attempted before (see https://github.com/ethereum/py-evm/issues/1818), but it proved too slow (see https://github.com/ethereum/py-evm/issues/1836) to use in practice for the Trinity client. In either case, my suggestion to use the hashlib implementation is indeed not viable.
However, since the Trinity client became obsolesced rather recently (reducing the necessity for sync-viable speed of this algorithm), and the numerous reported issues users have had setting up the Rust library on different platforms (see https://github.com/ethereum/blake2b-py/issues), as well as the problem that this issue notes about lack of support for blake2b-py... perhaps it would be time to reconsider this decision and move py-evm back to the slower native python implementation in order to reduce dependency and maintenance issues.

What about making blake2b-py an optional dependency? So, if you install py-evm via pip install py-evm it will use the (slow, but struggle-free) hashlib implementation, if you use pip install py-evm[blake2b-py] it will additionally install the (faster, but maybe annoying setup required) rust-based variant. And the code has then to detect if blake2b-py is installed, if yes it will use it, if no it will fall back to hashlib.
Any thoughts on that?

What about making blake2b-py an optional dependency

Yes, I think that's the way to go. We've used that pattern before and it works well.

What about making blake2b-py an optional dependency

Yes, I think that's the way to go. We've used that pattern before and it works well.

One thing to note is that the problem arises from eth-tester using py-evm when installed via web3[tester], so the optional has to bubble up 2 levels to web3 for it to work. e.g. py-evm[blake2b-py], eth-tester[py-evm,blake2-py] and web3[tester,blake2b-py]


concatenate large amount of arrays

added recursive concatentation in case of large amount of arrays. Awkward seems to breakdown when number of concatenated arrays exceeds 2 ** 7

cool


Why is Chokidar called Chokidar?

I'm just curious.

Found the origin! It's a variant of chowkidar, itself a transliteration of a Hindi word meaning 'watchman, gatekeeper', चौकीदार.
https://en.wiktionary.org/wiki/chowkidar

It was pretty random. That's mostly it!

As an Indian, I found the name funny :smile:

Found the origin! It's a variant of chowkidar, itself a transliteration of a Hindi word meaning 'watchman, gatekeeper', चौकीदार. This ultimately comes from Sanskrit _ चतुष्क_ (crossway, quadrangle, consisting-of-four).
That comes from चतुर्, loosely pronounced 'catur', which is cognate to Proto-Indo-European *kʷetwóres, root of our word quarter (via Latin quartus). Basically the idea is that you might 'guard' (chowkidar) an enclosed space, usually contained within four walls.
https://en.wiktionary.org/wiki/chowkidar

When I found this library, I thought the name definitely came from its Hindi meaning LOL. It was so funny. Just wanted to let you know that चतुर् is pronounced "chatur".

@paulmillr, I think the reason behind the name should be mentioned in the README.md. :)

[@xeoneux]
As an Indian, I found the name funny :smile:

@xeoneux, may I know why do you find the name funny?

@tukusejssirs When I hear chokidar, it reminds of an actual watchman. Chokidar is such a common word that finding it in tech was funny for me, especially having a piece of software in my system which I can refer to as my own chokidar/watchman.

That's mostly it!

Mostly? What's the rest of it? And what inspired you to call it that?

That's mostly it!

Mostly? What's the rest of it? And what inspired you to call it that?


Update Style Values and Style Examples for consistency

What I Thought I Understood
Looking at Style Values what I grasp is this:

simple, spaceDelimited and pipeDelimited MUST be an array
deepObject MUST be an object
every other style can be a primitive, array or object

style
type
in
Comments

matrix
primitive, array, object
path
Path-style parameters defined by RFC6570

label
primitive, array, object
path
Label style parameters defined by RFC6570

form
primitive, array, object
query, cookie
Form style parameters defined by RFC6570. This option replaces collectionFormat with a csv (when explode is false) or multi (when explode is true) value from OpenAPI 2.0.

simple
array
path, header
Simple style parameters defined by RFC6570.  This option replaces collectionFormat with a csv value from OpenAPI 2.0.

spaceDelimited
array
query
Space separated array values. This option replaces collectionFormat equal to ssv from OpenAPI 2.0.

pipeDelimited
array
query
Pipe separated array values. This option replaces collectionFormat equal to pipes from OpenAPI 2.0.

deepObject
object
query
Provides a simple way of rendering nested objects using form parameters.

What I Expect
What I expect to see when I look at Style Examples is this:

style
explode
empty
string
array
object

matrix
false
;color
;color=blue
;color=blue,black,brown
;color=R,100,G,200,B,150

matrix
true
;color
;color=blue
;color=blue;color=black;color=brown
;R=100;G=200;B=150

label
false
.
.blue
.blue.black.brown
.R.100.G.200.B.150

label
true
.
.blue
.blue.black.brown
.R=100.G=200.B=150

form
false
color=
color=blue
color=blue,black,brown
color=R,100,G,200,B,150

form
true
color=
color=blue
color=blue&color=black&color=brown
R=100&G=200&B=150

simple
false
n/a
n/a
blue,black,brown
n/a

simple
true
n/a
n/a
blue,black,brown
n/a

spaceDelimited
false
n/a
n/a
blue%20black%20brown
n/a

pipeDelimited
false
n/a
n/a
blue|black|brown
n/a

deepObject
true
n/a
n/a
n/a
color[R]=100&color[G]=200&color[B]=150

What I See
What I actually find on Style Examples is this:

simple can be primitive, array or object (which conflicts with what I read earlier)
spaceDelimited and pipeDelimited can be array or object (which conflicts with what I read earlier)
deepObject MUST be an object
everything else can be primitive, array or object

style
explode
empty
string
array
object

matrix
false
;color
;color=blue
;color=blue,black,brown
;color=R,100,G,200,B,150

matrix
true
;color
;color=blue
;color=blue;color=black;color=brown
;R=100;G=200;B=150

label
false
.
.blue
.blue.black.brown
.R.100.G.200.B.150

label
true
.
.blue
.blue.black.brown
.R=100.G=200.B=150

form
false
color=
color=blue
color=blue,black,brown
color=R,100,G,200,B,150

form
true
color=
color=blue
color=blue&color=black&color=brown
R=100&G=200&B=150

simple
false
n/a
blue
blue,black,brown
R,100,G,200,B,150

simple
true
n/a
blue
blue,black,brown
R=100,G=200,B=150

spaceDelimited
false
n/a
n/a
blue%20black%20brown
R%20100%20G%20200%20B%20150

pipeDelimited
false
n/a
n/a
blue|black|brown
R|100|G|200|B|150

deepObject
true
n/a
n/a
n/a
color[R]=100&color[G]=200&color[B]=150

What I Think Needs Correcting
Please correct me if I'm wrong, but my expectation is this:

style
type
in
Comments

matrix
primitive, array, object
path
Path-style parameters defined by RFC6570

label
primitive, array, object
path
Label style parameters defined by RFC6570

form
primitive, array, object
query, cookie
Form style parameters defined by RFC6570. This option replaces collectionFormat with a csv (when explode is false) or multi (when explode is true) value from OpenAPI 2.0.

simple
primitive, array, object
path, header
Simple style parameters defined by RFC6570.  This option replaces collectionFormat with a csv value from OpenAPI 2.0.

spaceDelimited
array, object
query
Space separated array values. This option replaces collectionFormat equal to ssv from OpenAPI 2.0.

pipeDelimited
array, object
query
Pipe separated array values. This option replaces collectionFormat equal to pipes from OpenAPI 2.0.

deepObject
object
query
Provides a simple way of rendering nested objects using form parameters.

If this is the case, please let me know and I will make a PR to close it.

Please refer to the 3.1.1 specification in the v3.1.1-dev branch, as there have already been some corrections to this area of the specification.

I'm coming at it from the perspective of a request/response validation author, where we're taking data in a request or response and deserializing it into a data structure. From this angle, allowing more types means fewer places where the validation would error out because of the wrong type being specified in the schema, but indeed it doesn't add any value to allow it (the application might as well accept an array with a single value).

Update: we discussed this PR at the TSC meeting today (https://github.com/OAI/OpenAPI-Specification/issues/3672) and decided:

the changes for simple are sensible - all of primitives, arrays and objects are possible and make sense, so the spec should reflect that
regarding spaceDelimited and pipeDelimited, a primitive type is not useful here (when writing an api spec, if you're expecting a string why would you specify style: spaceDelimited? just use simple).

Therefore -- @charjr: could you possibly revert the changes to spaceDelimited and pipeDelimited in your PRs? I think then we should be able to review and merge them quickly (e.g. before or by the next TSC meeting). thanks!

@karenetheridge I've updated each PR to remove the primitive type from spaceDelimited and pipeDelimited.
Hopefully it's ready to go, if not let me know.

looks great, thank you!

I've added comments to all 3 PRs following the discussion at the meeting today.

@karenetheridge updated, let me know if it needs any further changes.

All of these PRs were merged so I'm assuming this is resolved.  Closing.


Questions regarding reproduction of the glasses dataset

I would like to reproduce the glasses dataset experiment as reported in your publication. The paper reports the number of wraps to be N=5. The code has the setting of wraps to be N=1.
Is it correct that this warpN setting should be set to 5 to reproduce? In this case, CUDA runs out of memory on the NVIDIA GTX 1070 (8 GB) I am using. Related to this, both the paper and the supplementary material do not report any hardware specifications. Could you share the details of the setup that was used during your experiments?

@chenhsuanlin
Hi, I tried to run the glasses/train.sh. But to my surprise, even when warpN=5, the whole model still only occupied 2137 MB of GPU memory. All I did was modifying the data path in glasses/data.py. Did I miss something or is this also the case when you ran the glasses experiment.

@ipod825 -- Yes, this is because the checkpoint size is relatively small (~50MB), so even 5 copies of the generator may not affect the GPU memory size allocated by Tensorflow.
@sbrugman -- I hadn't been observing the memory growth much because I never really had issues, but as pointed out, ST-GAN should be able to fit in an 8GB graphics card if you're running our provided training script. Please feel free to reopen if you still have further issues.


Export PieceTreeTextBufferFactory

there is a huge log file (about 600mb) I want to show in monaco-editor, but I cannot transfer file to string because it exceeds the string limit(v8 limit). And I find vscode use the instance of PieceTreeTextBufferFactory to accept buffer. However, monaco-editor has no exprt PieceTreeTextBufferFactory and vscode-textbuffer is old.
so I hope to export the PieceTreeTextBufferFactory class in monaco-editor. or have someone tell me how to build the monaco-editor-core? I can export it and publish the package to my private npm repository.

monaco-editor has no exprt PieceTreeTextBufferFactory

The recommendation is

either use AMD loader which allows you to load an internal module
or bundle monaco editor esm modules in your project by webpack

We have a great extension developer community over on GitHub discussions and Slack where extension authors help each other. This is a great place for you to ask questions and find support.
Happy Coding!


Modeless colorpicker

The "for working with a modeless color selector window" is related to the thonny-py5mode changes that I will propose at the other repo (thonny-py5mode-tkcolorpicker)

Sorry @tabreturn, I'm terrible with git.
The "adding quotes" commits where not supposed to be here...

@villares -- you'll have to add a pull-request for your changes to the sub-module (https://github.com/tabreturn/thonny-py5mode-tkcolorpicker) so that the code here can import it

@villares -- you'll have to add a pull-request for your changes to the sub-module (https://github.com/tabreturn/thonny-py5mode-tkcolorpicker) so that the code here can import it

here: https://github.com/tabreturn/thonny-py5mode-tkcolorpicker/pull/2

Cheers @tabreturn! Will the colorpicker be updated in the next release?
On my fork I get this, and I quite like it:


Not available for Symfony 5.0.*

Restricting packages listed in "symfony/symfony" to "5.0.*"
Your requirements could not be resolved to an installable set of packages.
Problem 1
- endroid/qr-code-bundle 3.3.4 requires symfony/framework-bundle ^3.4|^4.0 -> satisfiable by symfony/framework-bundle[v4.4.0] but these conflict with your requirements or minimum-stability.
- endroid/qr-code-bundle 3.3.3 requires symfony/framework-bundle ^3.4|^4.0 -> satisfiable by symfony/framework-bundle[v4.4.0] but these conflict with your requirements or minimum-stability.
- endroid/qr-code-bundle 3.3.2 requires symfony/framework-bundle ^3.4|^4.0 -> satisfiable by symfony/framework-bundle[v4.4.0] but these conflict with your requirements or minimum-stability.
- endroid/qr-code-bundle 3.3.1 requires symfony/framework-bundle ^3.4|^4.0 -> satisfiable by symfony/framework-bundle[v4.4.0] but these conflict with your requirements or minimum-stability.
- endroid/qr-code-bundle 3.3.0 requires symfony/framework-bundle ^3.4|^4.0 -> satisfiable by symfony/framework-bundle[v4.4.0] but these conflict with your requirements or minimum-stability.
- endroid/qr-code-bundle 3.3.4 requires symfony/framework-bundle ^3.4|^4.0 -> satisfiable by symfony/framework-bundle[v4.4.0] but these conflict with your requirements or minimum-stability.
- Installation request for endroid/qr-code-bundle ^3.3 -> satisfiable by endroid/qr-code-bundle[3.3.0, 3.3.1, 3.3.2, 3.3.3, 3.3.4].

Hi @Unlik I just released an updated version of the qr-code and qr-code-bundle libraries. I hope this fixes your issue :)