Volunteers Page: Gift Redemption: After “Distribute Items” button is selected: Return Item

Bug: Does not appear to be a way to cancel a gift redemption during the "check out" phase.

Correct! I will work to add this ability

accepted


Data generation repeatability feature enhancements

feat: Added capability of setting and acquiring the seed used by a given Faker instance.
refactor: Removed Faker and RandomService instantiation by Random instance in favor of providing a seed.

@jpeffer thanks for the PR. This breaks the API, and I'm usually not a big fan of that. I'm not sure how many people are using the faker with Random constructor.
@snuyanzin  what do you think?

aagree and there are still some tests commented out... May be we could think in a direction of having two separate RandomService like classes. One is the old one and the second is with more seed control implementing same interface e.g.... This is just thoughts out loud...

Yea I was fully aware it would break the API, though that was somewhat purposeful as the original design was inherently problematic. That was a carry over from the original library you forked from I believe.
There are a few potential approaches:

This becomes a breaking change and therefore a major version update
We create a new Faker and RandomService at different package coordinates, then deprecate and modify the existing Faker and RandomService as a shim to the new ones
We simply deprecate the old constructors

The tests were purposefully commented out with a TODO section that contained additional questions pertaining to those tests. The main question I had was whether they covered a use case that is actually necessary or even desirable. It's encouraging a pass by reference design, which I would typically discourage.
I left this as a draft PR simply to start the discussion. I'm happy to make further changes that meet your goals for the project. I think I favor option 2 above if you don't want to break the API now.

@snuyanzin, I did run across the stackoverflow mentioning this option, but it seemed somewhat brittle. I think not relying on this workaround with be the wisest thing to do. I'll be opening another PR with an approach that aims to not break the existing API.


Vite 5 Support

Is there any work on Vite 5 support? Otherwise I can open a pull request.

Hi! I was planning starting this soon, but any PR will be welcome, thanks!

When can we get support for Vite 5 ?

Created a PR now @fabio-ivona .. hope it helps :))

Released :heart:


issue with running on Azure

On my lab many students have problem with running Jupyter workbook having an issue:
"/home/mmlspark/lib/conda/envs/azureml_fe7155b29abe02ff488ec064c24fe4ae/lib/python3.5/urllib/request.py", line 1324, in unknown_open raise URLError('unknown url type: %s' % type) urllib.error.URLError: <urlopen error unknown url type: http\>
Anyone knows how to solve it or fix it? I tried to change mmlspark, change version, add repplacements to a code, but nothing happens. Even I made myself whole environment from the beginning. Nothing :(
Case: Neural Networks - CNTK (Use the MNIST database  to train a neural network built with the Microsoft Cognitive Toolkit  and Azure Machine Learning Workbench to recognize handwritten digits.  Then deploy the model and write a Node.js app to visualize the output.)

I will look into it and provide a fix. Thanks!

Thank you for quick reply and info! I will be thankful and waiting :) Cheers!

This lab is being migrated to docs.microsoft.com/learn and won't be updated on this repo anymore, so I'm closing this issue.


Fix illegal tag character replacement, add cURL to container, and add EBS example to readme.

While trying to setup YACE, we found that one of our EBS volumes had a tag which contained an equals sign, "=", which YACE was reporting as illegal and crashing on. I've updated this replacement command to now handle equals signs as expected.
I've also added a bit to the README.md file that shows some basic EBS configuration to get "BurstBalance" stats from those EBS volumes, since EBS isn't covered by the default example configuration.
I've also added the installation of cURL to the container's Dockerfile to assist with troubleshooting.

@cdchris12 - Nice thanks a lot for your contribution! - Will release it today.


Add How-To instructions for landing-page redirects

Pertinent section of the Wagtail docs
https://docs.wagtail.io/en/stable/advanced_topics/index.html
https://docs.wagtail.io/en/stable/reference/contrib/forms/customisation.html
Suggestion type

x
Type of change
Fix/enhancement

[ ]
Copy edit
Fix spelling or grammar errors, improve clarity

[ ]
Content correction
Correct inaccurate explanations, instructions, or code

[x]
New content
Add a new topic or provide more detailed instructions or examples

[ ]
Reorganization
Move topic(s)

[ ]
Other suggestion
Anything else

Details
This uses both the forms and routable_page contrib modules to redirect
without using a separate landing page.
There is a similar guide in the Form builder documentation at https://github.com/wagtail/wagtail/blame/master/docs/reference/contrib/forms/customisation.rst#L552, but it requires adding a separate landing page. I have long thought that the non-redirecting behaviour of the standard Form builder pages was counter to the fairly standard practice of redirecting after a POST request, to prevent issues with refreshing the pages. This was raised in #2834, but closed because the use of the GET parameter to provide the form submission was implementation dependent. It looks unlikely to become part of Wagtail core this way, but the recipe ought to be documented.
Because this example spans two modules, I have not added it to the form builder documentation directly. I have also used the new 'How To' label from the January 2021 documentation sprint.

Just tested the code out as is (copy/paste) and works great.
Note network tab - 302 redirect to new 200 for the new URL under sub-route.

Pushed some refinements (whitespace change, changelog, added page to index.rst) to the branch, rebased and merged in.


feat: add location info to session table

Hi @Transmigration-zhou. Thanks for your PR.
PRs from untrusted users cannot be marked as trusted with /ok-to-test in this repo meaning untrusted PR authors can never trigger tests themselves. Collaborators can still trigger tests on the PR using /test all.
I understand the commands that are listed here.

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


make_zn_scores: add max_obs argument

:param  max_obs: the mxaimum number of observations allowed to calculate
zn_scores. This limits the maximum number of periods that should be used
for zn-scoring. The purpose of this argument is to prevent observations 	from entering that are too old.

Implementation complete. Unit Test required.


Set project type in metadata of uploaded objects

The gRPC server will set the project type (routeviews, RIPE, ...) in the metadata when it writes to GCS; the converter will extract collector name based on the project type in metadata and the object path.
Right now, if the project type is not supported/recognized, the collector field will be left empty.

seems safe to merge at this point.


react sanely to errors

When we show an error that bubbles up from the OS, we often stop dead.  Instead, we should:

wait a few seconds, and retry in case the error could be transient (eg: ECONNRESET or ECONNREFUSED)

provide some explanation if the error is something we understand (eg: ENFILE should provide a link to how ulimit works so people can try increasing their # of file handles - something like http://unix.stackexchange.com/questions/108174/how-to-persist-ulimit-settings-in-osx-mavericks, or at least explain that the deploy has too many files for the current OS configuration)

I agree with this in general, but there isn't a single switch we can throw to solve it.  Closing in favor of when this comes up specifically, and we can commit to reacting more sanely in those cases.


Improve opcode reference detection robustness.

The current opcode regexp fails to detect opcode reference in
the following cases.

Texts extracted using non-Poppler (Glyph & Cog) pdftotext in
Windows, which sometimes omit the headnote and begin directly with
the opcode itself.
Instructions that have a different headnote pattern, e.g., VMX
instructions.

This changeset updates the opcode regexp to address the
above issues.

Poppler pdftotext takes priority over non-Poppler since 1) Poppler is the backend of the original pdftotext and 2) I personally use the Poppler version. I'd like to support both, but it's vital that the Poppler version works at least as well as it already does.
Unfortunately your change causes the Poppler database to lose 69 mnemonics (see pdftotext.txt). It also changes the page for the ambiguous movsd mnemonic, though I don't mind that. I produced that diff by printing each database with pp and running diff -u on the results.

Poppler pdftotext takes priority over non-Poppler since 1) Poppler is the backend of the original pdftotext and 2) I personally use the Poppler version. I'd like to support both, but it's vital that the Poppler version works at least as well as it already does.

I would love to have an easy-to-deploy (e.g., via Chocolatey) Windows port of Poppler. For now, I am stuck with xpdf-utils, which is Glyph & Cog version of it.

Unfortunately your change causes the Poppler database to lose 69 mnemonics (see pdftotext.txt). It also changes the page for the ambiguous movsd mnemonic, though I don't mind that. I produced that diff by printing each database with pp and running diff -u on the results.

Thanks for the list; it is helpful narrowing down the issue.
Interestingly, there are some instructions, such as prefetchwtl, invept, vmcall, that the current regexp misses while the patch's regexp captures.

I will dig into this and get back to you when I get time.

I have root-caused the regression to long opcode summary (after --) got split into multiple lines by pdftotext.
I have updated the regexp accordingly to address this. Now it finds all previously missing opcodes, plus the VMX ones in Volume 3 that are missing in current implementation.

Merged, thanks! It still misses a few mnemonics in my two-year-old copy
of the manual, but it works exactly as advertised on the latest version.
I suspect the old manual had formatting irregularities that have since
been corrected by Intel and so no longer need to be accounted for in the
regexp.

Thanks for the quick turnaround!

I suspect the old manual had formatting irregularities

The SDM is manually formatted using Word rather *TeX; so irregularity is no surprise. It is better to have lax/forgiving regexp to capture the essence.


[#18] Delete category code added

Predefined categories are prevented from being deleted. Categories in use cannot be deleted. This closes [#18]

Code Looks good to me


Add type safety to GlobalMessageModule, RoutingModule and AuthModule

Created from #274. See also #7 .
Use the models from #301.

@znikola Not seeing any facade/store that can be used with the models created in #301

I'm not seeing an interface that models token types in #301, does it exist elsewhere in the project?

Yes, it's in auth/models.

Looks good.
I have a small question: I just did a few spot checks in the modules in scope of the ticket.
I wondered if types could be added to those files:
projects/core/src/routing/facade/routing.service.ts```


⚠ PrivateHD has degraded performance

In d723270, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 1820 ms

Resolved: PrivateHD performance has improved in 175fbd8.


NavBar covers part of the View

Version

react-native-router-flux v3.2.6
react-native v0.22.2

Expected behaviour
The content of a view should be below the NavBar
Actual behaviour
The content is partly hidden by the NavBar if the container is full enough.
Steps to reproduce

Open Example project
On the <Scene key="root"> remove the hideNavBar property
Inject a lot of text in the home view
Navigate to the home view, see that the text is behind the navbar

for the example the first few textlines were numbers, starting from 1. That way it is easy to see that 1 and 2 are below the navbar

I am willing to investigate myself but I have no idea where to start

It is not bug, it is known feature of React Native as well as native iOS nav bar (so semi-transparent navbars could be shown nicely). You need to adjust your padding by yourself.
https://github.com/facebook/react-native/issues/4264

Perfect, thanks!


Allow to use DOCKER_HOST to override configuration

If the DOCKER_HOST environment variable is set, it will be used to
override the values defined in .mochapodrc.yml. This is to allow
mocha-pod projects to be used in different contexts without the need to
modify the configuration file for each context.
Change-type: minor

@balena-ci I self certify!


"IP allow list" checkbox options are affecting the JIRA and GitHub App Configuration

Hi, we recently found out that the GitHub App configuration in JIRA has error. Our IT team tried to debug and un-checked "IP allow list" options, which helped to make the configuration successfully. But if we enable the "IP allow list" options, the error occurred again. (Please see the attached screenshot).

Info provided by Atlassian support:
"I just found out that there was a recent change in our IP ranges that need to allow Atlassian products to be able to communicate with third-party integrated apps. So, if you want to continue to use the IP allowlist on the Github site, you will need to ensure our IP ranges are included in that list.
Please have them modify the IP allowlist to included all of our updated IP ranges provided here: https://ip-ranges.atlassian.com/"
Could you please help to modify the IP allowlist?
Thanks!

Hey @sherylxiyueliu,
This issue is already tracked here: https://github.com/atlassian/github-for-jira/issues/411. Please follow that issue, I'm going to close this as a duplicate. But here's an explanation:
The app currently doesn't support IP allowlists, because it's running in an environment that doesn't guarantee a fixed range of IP addresses. The good news is that this is going to change, hopefully during the next weekend (August 22nd), on which we're moving the app into a new environment.
Once that move is done, it should work with IP allowlist enabled. You won't even have to add the IPs manually as we'll configure them into the GitHub app and when you click the "Enable IP allowlist configuration for installed GitHub apps" checkbox, it should automatically configure those IP addresses for you:

Thanks @thombergs


Error: spawn E2BIG

Using<EMAIL_ADDRESS>and<EMAIL_ADDRESS>~/src/yw/finfin master*  7s
❯ npm t

><EMAIL_ADDRESS>test /Users/yoshuawuyts/src/yw/finfin
> standard && NODE_ENV=test node test | colortape

child_process.js:1132
    throw errnoException(err, 'spawn');
          ^
Error: spawn E2BIG
    at exports._errnoException (util.js:734:11)
    at ChildProcess.spawn (child_process.js:1132:11)
    at Object.exports.spawn (child_process.js:964:9)
    at spawn (/Users/yoshuawuyts/src/yw/finfin/node_modules/standard/index.js:
162:20)
    at /Users/yoshuawuyts/src/yw/finfin/node_modules/standard/node_modules/run
-parallel/index.js:32:7
    at Array.forEach (native)
    at module.exports (/Users/yoshuawuyts/src/yw/finfin/node_modules/standard/
node_modules/run-parallel/index.js:31:11)
    at /Users/yoshuawuyts/src/yw/finfin/node_modules/standard/index.js:136:7
    at done (/Users/yoshuawuyts/src/yw/finfin/node_modules/standard/node_modul
es/run-parallel/index.js:15:13)
    at /Users/yoshuawuyts/src/yw/finfin/node_modules/standard/index.js:157:7
npm ERR! Test failed.  See above for more details.

~/src/yw/finfin master*

Google doesn't seem to be familiar with these popping up in node. Maybe anyone's had this happen before? The project can be found here in case you want to replicate. Thanks!

E2BIG means that the argument list is too long. See errors.c where it's defined.
I cloned your project and it worked fine for me, on iojs 1.3.0, so this must be an operating system difference. What OS are you using?
Please run getconf ARG_MAX to check the max argument size (in bytes) of your OS.
Also, please install the latest version of standard (2.7.1). Verify by running standard --version. Then run DEBUG=standard npm t and paste the output. It should show us the exact command that standard is trying to run under-the-hood (and we can see if there's actually a huge argument list being passed, or not).

Hah, I ran standard with DEBUG=standard and it appeared node_modules weren't being ignored in non-root directories. Adding a standard.ignore property to package.json solved it for me. I believe there was a discussion about ignoring node_modules in an issue, but I can't seem to find it (would be useful to link to, I think.) Anyway I'm all good now; thanks! :tada:

Thanks for sharing the outcome. This settles it. I'll merge the issue to ignore all node_modules folders.

Hey @yoshuawuyts @feross, maybe this isn't the right place to ask this since it's not related to this repo at all, but I'd appreciate if you could tell something about this question that I asked on stack overflow. I'm not able to debug the E2BIG error source in my project's grunt build process. If you could help (if that's possible, that is), I'd really need some help. Thanks


Rename unimported-packages flag

cc @naoina
Since, in nsf/gocode, the unimported packages thing was not a flag (it was a key in configuration), we can use this opportunity to give the new flag a more descriptive name and usage message.

I don't think the original name is all that bad.  One could envision in the world of modules that this approach eventually extends outside of the standard library.

@zmb3 That's a good point. It wouldn't be too hard to actually add this feature by using the source importer. Let's leave it as is, and that will create an incentive to actually add this feature :)

@frou: the usage string you added is better though, so let's make that that change.


Hide Chevron Buttons (Camera Switching)

[REQUIRED] Describe the feature you'd like:
We should be able to control visibility of the left/right chevron buttons that switch between cameras (when there are multiple camera entities on the card). They block a considerate portion of the view on smaller screens and are unnecessary for those who have drag/swipe enabled for the live cameras, or use the menu dropdown.
[OPTIONAL] Describe alternatives you've considered:
Visibility could even be controlled by the menu style setting (no menu/hover menu), or moved to the menu itself
[OPTIONAL] Additional context:
I can block the HTML elements on my browser, but I am unaware of a method to block them in the app.

To disable the next/previous controls for both media viewing and live:
media_viewer:
  controls:
    next_previous:
      style: none
live:
  controls:
    next_previous:
      style: none

thank you very much! I thought I disabled every toggle in the card's UI and it was still there, but this works.


79: Fix Some Code Smells

Fix Some Code Smells
Renamed some variables and removed some unused variables as indicated by SonarCloud.
Issue
#79.

I don't know why Sonar is failing.  There's some talk about a missing class since Java 14, but why would we have a regression?  Did SonarCloud do something?  Seems like the server-side stuff can make us run into this error (if the Sonar Source link above is to be believed).


List dataview refactor

Lots of ES6's syntax sugar goodies applied:

Template strings to build aggregation query.
New class syntax
Renamed base class
Remove every reference to widgets.

Beware! https://github.com/CartoDB/cartodb/issues/12826

I am going to merge this to master (it is necessary to close the #748 ). And then, I will work in merge #763


Is there a way to limit the allocation of GPU memory that mxnet uses?

Hello.
I'm curious if there is a way to limit the allocation of GPU memory that mxnet uses.
For example in theano, setting cnmem in .theanorc limits the maximum amount of memory used by the system.
Thank you!

I too would be interested in this.  I have a system using caffe, mxnet, and my own memory allocator and 12GB isn't enough!
It would be really nice to able to install my own memory allocator so everyone can share nicely.

This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this is still an active issue. Thanks!


Switch to using build-essentials cookbook

This is better than maintaining code in this cookbook. Let the guys maintaining build-essential deal with distribution-specific issues and such.
Fixes #277

I've also been having issues with missing headers that was resolved by applying this patch.

Makes sense.

Updated from master, so it'll merge cleanly.


add validation for min and max on select field

Pre-Flight checklist

[x] Testing

[x] Code is tested
[x] Changes have been manually tested

What are the relevant tickets?
closes #185
What's this PR do?
this adds validation for the min and max property on a Select field (this is only enforced when the multiple prop is also set). I also added some general testing for the select validation, as there weren't yet any tests for it.
How should this be manually tested?
I made a little change to the localdev/configs/ocw-course-site-config.yml file, so you should be able to just run the override_site_config management command to add a multiple-select 'tag' field to the page content type. Then you can check in the UI that it's correctly enforcing the min and max (we may want to remove this again from localdev/configs/ocw-course-site-config.yml before we merge so it's not too annoying).
Screenshots (if appropriate)

If I start off with an empty tags list and click Save without modifying the field, it will save without a validation error. This probably won't happen with new pages but it could happen with existing pages that get a new field

ok I fixed the issue, it related to not having a multiple-select specific default value, so an empty string was being put there instead of an empty list. I fixed it, so editing a page which does not have a value saved yet for tags should work (i.e. if you open the editor you should see a validation message saying 'tags must have at least one entry'

Codecov Report

Merging #211 (b33c275) into master (3dc38a5) will increase coverage by 6.24%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #211      +/-   ##
==========================================
+ Coverage   86.12%   92.37%   +6.24%
==========================================
  Files         109       54      -55
  Lines        2609     1167    -1442
  Branches      397      254     -143
==========================================
- Hits         2247     1078    -1169
+ Misses        343       89     -254
+ Partials       19        0      -19

Impacted Files
Coverage Δ

static/js/types/websites.ts
100.00% <ø> (ø)

static/js/components/forms/validation.ts
100.00% <100.00%> (+12.50%)
:arrow_up:

static/js/lib/site_content.ts
89.71% <100.00%> (+0.83%)
:arrow_up:

static/js/util/factories/websites.ts
100.00% <100.00%> (+1.75%)
:arrow_up:

main/celery.py

websites/__init__.py

websites/admin.py

news/util.py

websites/permissions.py

ocw_import/tasks.py

... and 49 more

Continue to review full report at Codecov.

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


How to implement in Nix Flakes?

Hey I'm trying to add scalpel into my dotfiles config but I'm unsure how to implement the extendModules hook into my particular flake, which is based on Misterio77's start config. This is what my flake.nix looks like below
{
  # BASED ON https://github.com/Misterio77/nix-config/
  description = "My NixOS Configurations for multiple machines";

  inputs = {
    nixpkgs.url = "github:nixos/nixpkgs/nixos-unstable";

    hardware.url = "github:nixos/nixos-hardware";
    impermanence.url = "github:nix-community/impermanence";
    nix-colors.url = "github:misterio77/nix-colors";

    home-manager = {
      url = "github:nix-community/home-manager";
      inputs.nixpkgs.follows = "nixpkgs";
    };

    nh = {
      url = "github:viperml/nh";
      inputs.nixpkgs.follows = "nixpkgs";
    };

    firefox-addons = {
      url = "gitlab:rycee/nur-expressions?dir=pkgs/firefox-addons";
      inputs.nixpkgs.follows = "nixpkgs";
    };

    nixpkgs-howdy.url = "github:fufexan/nixpkgs/howdy";

  };

  outputs = { self, nixpkgs, home-manager, ... }@inputs:
    let
      inherit (self) outputs;
      lib = nixpkgs.lib // home-manager.lib;
      # Supported systems for your flake packages, shell, etc.
      systems = [
        "aarch64-linux"
        "i686-linux"
        "x86_64-linux"
        "aarch64-darwin"
        "x86_64-darwin"
      ];
      # This is a function that generates an attribute by calling a function you pass to it, with each system as an argument
      forEachSystem = f: lib.genAttrs systems (sys: f pkgsFor.${sys});
      pkgsFor = nixpkgs.legacyPackages;
    in {
      inherit lib;
      # Reusable nixos modules you might want to export
      # These are usually stuff you would upstream into nixpkgs
      nixosModules = import ./modules/nixos;
      # Reusable home-manager modules you might want to export
      # These are usually stuff you would upstream into home-manager
      homeManagerModules = import ./modules/home-manager;
      #templates = import ./templates;

      # Your custom packages and modifications, exported as overlays
      overlays = import ./overlays { inherit inputs outputs; };

      # Your custom packages
      # Acessible through 'nix build', 'nix shell', etc
      packages = forEachSystem (pkgs: import ./pkgs { inherit pkgs; });
      # Devshell for bootstrapping
      # Acessible through 'nix develop' or 'nix-shell' (legacy)
      devShells = forEachSystem (pkgs: import ./shell.nix { inherit pkgs; });
      # Formatter for your nix files, available through 'nix fmt'
      # Other options beside 'alejandra' include 'nixpkgs-fmt'
      formatter = forEachSystem (pkgs: pkgs.nixpkgs-fmt);

      #wallpapers = import ./home/misterio/wallpapers;

      # NixOS configuration entrypoint
      # Available through 'nixos-rebuild --flake .#your-hostname'
      nixosConfigurations = {
        # Main desktop
        ryzennova = lib.nixosSystem {
          modules = [ ./hosts/ryzennova ];
          specialArgs = { inherit inputs outputs; };
        };
        # Personal laptop
        yoganova = lib.nixosSystem {
          modules = [ ./hosts/yoganova ];
          specialArgs = { inherit inputs outputs; };
        };
      };

      # Standalone home-manager configuration entrypoint
      # Available through 'home-manager --flake .#your-username@your-hostname'
      homeConfigurations = {
        # Desktops
        "novaviper@ryzennova" = lib.homeManagerConfiguration {
          modules = [ ./home/novaviper/ryzennova.nix ];
          pkgs = pkgsFor.x86_64-linux;
          extraSpecialArgs = { inherit inputs outputs; };
        };
        # Laptops
        "novaviper@yoganova" = lib.homeManagerConfiguration {
          modules = [ ./home/novaviper/yoganova.nix ];
          pkgs = pkgsFor.x86_64-linux;
          extraSpecialArgs = { inherit inputs outputs; };
        };
      };
    };
}

Probably something like
    nixosConfigurations = {
        # Main desktop
        ryzennova =
            let base = lib.nixosSystem {
              modules = [ ./hosts/ryzennova ];
              specialArgs = { inherit inputs outputs; };
            };
            in base.extendModules {
              modules = [
                scalpel.nixosModules.scalpel
                <yourscalpelconfigfiles>
              ];
              specialArgs = { prev = base; };
            };
   };


Updated java fdk version, java sdk version and policies

Updated java fdk version, java sdk version and policies

I left the python versions as is. For Python the requirements.txt has
fdk oci>=2.2.18
So it fetches the latest versions. I verified one example by checking the output of fn -v deploy ...


Render internally with a higher font size to reduce blurred text due to UI scaling

This is still wip because unfortunately there is still a bug inside the measurement methods. So far I was not able to figure it out yet.
In the rendered output of the test project one can see that the bounds are still slightly calculated wrong:

If someone wants to help me out fixing this, I would be very happy.
PS:
I'm aware that the changes which add another glyph map within the DynamicSpriteFont are not necessary anymore. I just did not find time yet to remove it.

I understand that this PR is follow-up to the discussion at the discord, however not everyone who watches the project is present at discord.
Thus can I ask you to create an issue with detailed description of the problemunderstandable for someone who isnt present at discord? And then link the issue to this PR.

You're right @rds1983, I'll open a ticket for this and link it with this PR!

OK, the DynamicSpriteFont is now working correctly again.
Output of the test application.

Need to fix the StaticSpriteBatch now.

So there is one slight bug in there which I don't really understand. With increasing internal rendering font size multiplicator for some positions the measurement height increases slightly.
2x

4x

8x

Besides that, one also sees that with increasing internal resolution we get quite the Alias, so some anti aliasing would be necessary there :|
However I think 2x is sufficient for most cases.

Forgot to note here, for the StaticSpriteFont this is kind of disabled. I did set the rendering multiplicator to 1.

With the last commit the bug is fixed. Now everything works correctly.

stbtruetype contains helpers for prefiltering/oversampling textures so they look better when scaled by the GPU.
It is what I do in my font renderer: https://github.com/TechnologicalPizza/MonoGame/blob/0a5b4c226d61cabdb21373d00ed8d3ba453e977b/MonoGame.Testing/Font.cs#L145-L148
This produces way better results than naively providing larger font resolutions, and my solution works between arbitrary source resolutions too.

Many thanks for the reference @TechnologicalPizza, however that requires a more invasive change so far I see :(

Well prefiltering is now included too. Overall things are looking pretty good now.

Old version:

Version with  fontResolutionFactor=2f, kernelWidth=1, kernelHeight=1

@realvictorprm, thank you very much!


Autofocus to search-input

It will be much easier if it autofocus search-input so user can directly write something to search when the page loaded
instead of clicking search area then write something

@captn3m0

Might be something worth directly proposing to the theme that we use: https://github.com/pmarsceill/just-the-docs/

If you'd like to work on this issue, setup the project locally as per HACKING.md. Then understand the Jekyll file structure and add the new script accordingly.
Only trigger this if the user starts typing alphanumeric characters.

Hey @captn3m0, I'd be interested in taking a look at this, but was wondering whether you'd prefer I do it here or in the theme? Seems like it could be appropriate in either place.


Don't emit debug messages in non-debug unittest build

They are not particularly interesting to know.

std/math.d(7501): Error: declaration `std.math.__unittest_L7472_C26.BUG5628` is already defined

Auto-merge toggled on

Seems to be failing in 64bit

Auto-merge toggled off


Provide a Statement configuration API for scripting

he YAML represents a set of data that is a template for statements which can then be used to create executable operations. Sometimes, users need to configure statements that are generated from templates at a higher-level than YAML. For example, a user may need to create many variations of a statement which might appear in an application without having to enumerate them all, precisely by using the same type of data-dependent logic that a client application may use.
It should be possible to provide an object directly from the scripting layer to modify the statement template configuration. This would be an API, and may take the form of a scripting extension which is built on top of the standard statement configuration API.
detailed design: TBD
originally filed as https://github.com/engineblock/engineblock/issues/67

superseded by 5.21+ command API


White text on "Your top genres" and "Browse all" cards for better readability

Is your feature request related to a problem? Please describe.
The text on the cards on the search page is often very hard to read with the colors the theme chooses (see screenshot 1).
Describe the solution you'd like
Make the text on the cards on the search page white, or a light gray, so that the text is readable on the different colored backgrounds.
Describe alternatives you've considered
I could just update the user.css for myself, but I think this might be something that should be standard for the theme.
.x-categoryCard-CategoryCard and .a2ruVaZt_DdrdHz3GqgU (?) would need color: white, or some other lighter color.
(e.g. #eaeaea like most other "white" text)
Additional context
Hard to read text (Indie):

Good to read text (white text):

We can't use .a2ruVaZt_DdrdHz3GqgU because these classes change with each spotify update, can you find another selector?

#searchPage h3 {
    color: #eaeaea;
}

Could be used for "Your top genres" and for "Browse all", as both use h3.
And under "Your Library" the liked songs text is also on a colored background and sometimes hard to read.

So this could also be implemented to make that text white as well:
.main-heroCard-draggable,
.main-heroCard-draggable a,
.main-heroCard-draggable div {
    color: #eaeaea;
}

Thanks a lot


Unclear code example

There's no indication of where to initialize the DailyTemperature record via the private static example given, for a console application created for this tutorial.

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

ID: 294a082e-5584-008e-b83a-7f090d898cda
Version Independent ID: d4a11dfa-2836-b423-63cd-c356e72e16a7
Content: Use record types - C# tutorial
Content Source: docs/csharp/whats-new/tutorials/records.md
Product: dotnet-csharp
Technology: csharp-whats-new
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for noting this @ion-storm-chaser
I'll update this tutorial in the next month.


Update local module and theme folder links to comply with default folder structure.

Module and theme folder links in local setup are not in sync with default folder structure of modules and themes under /drupal directory.

This looks like it could be in the master too.


Support for generating TypeScript union types for non-algebraic enums

In a lot of situations, plain union types are better suited than enums. Ideally there would be some way to make the output of this:
#[derive(Serialize, Deserialize)]
#[typeshare]
pub enum FooBar {
    Foo,
    Bar,
}

be this:
export type FooBar = "Foo" | "Bar"

instead of
export enum FooBar {
    Foo = "Foo",
    Bar = "Bar",
}

I had hoped I'd be able to coerce the generator into doing as desired by using the serde tag attributes:
#[serde(untagged)] is unsupported
#[serde(tag = "type", content = "content")] doesn't work due to
thread '<unnamed>' panicked at 'the serde tag attribute is not supported for non-algebraic enums: FooBar', /Users/foo/.cargo/registry/src/github.com-1ecc6299db9ec823/typeshare-cli-1.0.1/src/main.rs:234:17
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

If I pollute my Rust code with meaningless newtypes:
#[derive(Serialize, Deserialize)]
#[typeshare]
#[serde(tag = "type", content = "content")]
pub enum FooBar {
    Foo(()),
    Bar(()),
}

Then I do get a TypeScript type that I can then manipulate, but this Rust pollution is not ideal.
export type FooBar =
	| { type: "Foo", content: undefined }
	| { type: "Bar", content: undefined };

where I can then
import { FooBar as FooBarGen } from "./foobar.gen";

export type FooBar = FooBarGen["type"] // resolves to "Foo" | "Bar"

Hello, and thank you for the suggestion! I'll bring this up with the other team members to discuss how we'd like to implement this.

Great, I appreciate it! For what it's worth, I think a lot of issues like this could have their impact mitigated heavily by some solution to #22, most type generation libraries I've seen (although they are focused on TypeScript) all provide escape hatches for the weird edge-cases you'll inevitably run into with what serde can let you do, I'll comment directly on that issue though.


_TZ3000_fccpjz5z / TS130F does not get paired

I try to add this curtain module which is supposed to be supported by the tuya app. It gets paired only as simple device.
Does anyone have an idea to get this paired and working with Homey pro?
Thx!

Hi @ruben375, it's added to the test version of the app. Will be in next version of the live.
https://homey.app/en-gb/app/com.tuya.zigbee/Tuya-Zigbee/test/

@JohanBendz
Will you add the add the end point setting feature as well? Actually in the test app it works only for up/down switching.


switch from group name to group id

I'd like to get other maintainers' opinions here, but this doesn't feel right to me because it would break any existing configuration that's using name, since that field would now be computed, and id would be required. A non-breaking approach could be to have a new data source, called something like data_user_group_by_id but that also feels a little bit weird because if we have a _by_id data source, there should be a _by_name data source for consistency.
Overall this doesn't feel to me like this is necessary because if you know the id you can look up the name. I'm flexible on this, but I'm going to review this as "Request changes" anyway until we can get an agreement.

The main idea behind this change was: initially to find ID of the resources in CloudConnexa UI user needed to open Swagger and it was too much work, so we did data sources to work by name field, as in most cases it was unique value.
After that approx 14 October 2024 we've introduced button in CloudConnexa UI near almost (if not almost) all entities - when user press on icon ID of the resources is getting copied.
So now users can get the ID of the resources easily and it kind of makes sense to now require "id" field in data sources as required.
From another side i've checked our code and found out that we are our self is heavily dependent on "name" field in two places:
data "cloudconnexa_network" "this" {
  for_each = local.networks
  name     = each.key
}

data "cloudconnexa_network" "this" { # passed into module
  for_each = var.networks
  name     = each.key
}

If this data source "cloudconnexa_network" were to change to only require ID field that would be inconvenient as we've built data structure that is dependent on:
locals {
networks = {
    azure_test = {
      default_subnet             = "xxx.xxx.xxx/xxx"
      default_subnet_description = "azure-test_full"
      extended_easy_subnets      = {}
      extended_complex_subnets   = {}
      applications = {
        "xxxxx_app_ports" = {
          route         = [{ domain = "XXXXX.local" }]
          service_types = ["HTTP", "HTTPS"]
        custom_service_types = [{ protocol = "ICMP" }] }
		}
      ipservices = {
        "azure-lab_full" = {
          route         = ["xxx.xxx.xxx/xxx"]
          service_types = ["ANY"]
        }
        "azure-prod_full" = {
          route         = ["xxx.xxx.xxx/xxx"]
          service_types = ["ANY"]
        }
        "azure-xxx-xxx" = {
          route         = ["xxx.xxx.xxx/xxx"]
          service_types = ["ANY"]
        }
      }
    }
    aws_lab = {
      default_subnet             = "xxx.xxx.xxx/xxx"
      default_subnet_description = "Labs subnet"
      extended_easy_subnets      = {}
      extended_complex_subnets   = {}
      applications = {
        "xxxxxxx" = {
          description = "xxxxx"
          route       = [{ domain = "XXXXXXXXXXX.local" }]
        custom_service_types = [{ protocol = "TCP", from_port = 8443, to_port = 8443 }] }
      }
      ipservices = {
        "AWS_full" = {
          description   = "AWS ALLOW ALL"
          route         = ["xxx.xxx.xxx/xxx]
          service_types = ["ANY"]
        }
      }
    }
}
}

"azure_test", "aws_lab" are the "keys" in data sources above ^^
From yet another side Terraform provider still in beta and some minor changes even breakable somewhat accepted/expected

Mmhh I'm kind of rusty on the Terraform provider SDK, but is there a way to make neither field required, and have logic that would enforce that one of the two are present?

Current status: performed testing of next resources:
+ cloudconnexa/data_source_application.go
+ cloudconnexa/data_source_network.go
+ cloudconnexa/data_source_user_group.go

"+" sign means that all good (checked all combination of id, name, only id, only name, both id and name provided, none of id and name provided)
As for "cloudconnexa/data_source_connector.go" it is harder. Unlike all the rest of entities where API enforce having unique name with connectors we can actually have next situation:

What i'm trying to say that it was broken before, broken now and as part of this PR i believe for this resource we should use only "id" field and make it required

@patoarvizu could you please review this PR again?


feature Add tablet rebalance run in cloud

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run fe_ut

run buildall


struct to Variable question

Thanks for the awesome library! I have started using it in one of my projects (https://github.com/forensicmatt/RustyUsn#query-records) to help filter/format output and plan to add it to many more.
Currently I am having to convert a record to JSON string, then to a Variable via Variable::from_json(). But, it would be awesome if I could just serialize my struct to a Variable and save time by skipping the JSON middle man. It seems like this might be possible using the ToJmespath trait. However, I do not understand traits very well. If this is the case and I can go from struct to Variable, could you help me with an example? Maybe something simple like turning the following structures into a Variable:
extern crate jmespath;
use jmespath::*;

#[derive(Debug)]
struct TestStruct1{
    test_field1: u32,
    test_field2: u16,
    test_field3: u16,
    test_field4: String
}

impl ToJmespath for TestStruct1 {
    #[inline]
    fn to_jmespath(self) -> Rcvar {
        Rcvar::new(Variable::from(self))
    }
}

#[derive(Debug)]
struct TestStruct2{
    test2_field1: u32,
    test2_field2: u16,
    test2_field3: String,
    test2_field4: TestStruct1,
}

fn main() {
    let test_struct = TestStruct2 {
        test2_field1: 32,
        test2_field2: 16,
        test2_field3: String::from("TestStruct2!"),
        test2_field4: TestStruct1 {
            test_field1: 32,
            test_field2: 16,
            test_field3: 16,
            test_field4: String::from("TestStruct1!")
        }
    };

    println!("{:#?}",test_struct);
}

It looks like ToJmespath is implemented for any type that is T: Serialize. Therefore, if you import the trait ToJmespath into your scope and also implement (i.e. derive) Serialize for your struct, then you would be able to do test_struct.to_jmespath() or ToJmespath::to_jmespath(test_struct).


Add log info when emitting lineage from spark (following 2650)

(I accidentally closed my PR 2650, I am resubmitting the same PR here.
Overview
This pull request introduces enhancements to logging within the Spark integration of OpenLineage. The goal is to enable clear and accessible logs that indicate whether an event has been successfully sent to the backend API.
Importance
Accurate logging is crucial for monitoring and verifying the reliability of event dispatching.
Changes Proposed
Implement additional logging statements that confirm the dispatch of events.
These logs will provide immediate feedback in the Spark execution logs, aiding in quick diagnostics and status checks by administrators and developers.
Your feedback and suggestions on this enhancement are welcome.
One-line summary:
Checklist
[x] You've signed-off your work
[x] Your pull request title follows our guidelines
[] Your changes are accompanied by tests (if relevant)
[x] Your change contains a small diff and is self-contained
[] You've updated any relevant documentation (if relevant)
[x] Your comment includes a one-liner for the changelog about the specific purpose of the change (if necessary)
[] You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)
[] You've added a header to source files (if relevant)

It looks like spotlessApply haven't been run on this commit, so CI if failing in main branch


price history chart has incorrect x axis labels

market?id=0xeac824bea3cf45f34fc4bec7485eef41c79e3cc4
for markets older than 3 months go to showing one day a week. That will clean up the x axis.

We should fix this one before release

this has been addressed, we show xaxis time labels based on length of market,
less than 3 days old, label every 3 hours
less than 1 month old, label are daily
over 1 month old, labels are weekly
@Chwy5 can you confirm on price history charts

X axis looks fixed....Did find one small issue of the dots (for the trade in price history) don't line up with the dates, for daily....will close this ticket and create a new one for this new issue


[Bug] No go file present in the directory

I am unable to execute sail.exe with go build command as mentioned in the document.
It is saying no go file present in the directory. I also tried go mod and go init but nothing helps.

Hey @Swapnasarit-Panda,
I see you closed this Issue, I hope you were able to get it sorted out.
If you have any other issues please let us know!


Introduce AbstractNote

The Roland TD-50 (newest e-drum flagship) supports notes with Velocity values up to 127+32 (allows for 32 more values somehow).
The current Note cannot support that, and in my eyes it should not. However, by defining AbstractNote and making current Note a subtype, this allows us to define some other note in our scipts that can take advantage of all the functions in the library (and in another library that we are developing) that dispatch on Note/Notes.
For this Notes is now a parameteric type.

@JoelHobson do you agree? I haven't finished it yet but I am opening a PR to get the ok from you as well.

@Datseris Sounds good. I agree that the existing Note type shouldn't do that, but I've got no problem with making sure things are extensible in case someone really needed 127+32 velocity levels.


mummer-4.0.0beta2 install problem: no executable "configure" file in the uncompressed directroy

According to INSTALL.md , the install command is
./configure --prefix=/path/to/installation
But  I have not found the "configure" file,  and the version I downloaded is mummer-4.0.0beta2.zip.
Here is the list of   the uncompressed directroy:
1 ACKNOWLEDGEMENTS
2 ChangeLog
3 configs
4 configure.ac
5 COPYRIGHT
6 docs
7 examples
8 include
9 INSTALL.md
10 LICENSE.md
11 m4
12 Makefile.am
13 Makefile.orig
14 MANUAL.md
15 mummer.pc.in
16 OBSOLETE.md
17 README.md
18 scripts
19 src
20 swig
21 tests
22 tt
23 unittests
24 yaggo.mk
Any  ideas would be greatly appreciated. Thanks.

The question has been solved. Thanks.

I'm having the same issue. I downloaded and untarred the .tar.gz file.  No configure file was there.

Dear @liruilin7086 ,
you said, the question has been solved. Please share the solution.
I also cannot find the configure file, and the version of automake running in the CentOS 7.2 (GNU automake 1.13.4) does not satisfy the requirements of this mummer version.


Unable to use helm chart with postgresql database

Hi,
the helm chart is focusing on cockroachdb deployments, which may be fine for cloud based setup.
We want to use zitadel on-premise with a baremetal postgresql cluster. Unfortunately the chart requires a number of cockroachdb related secret, certificates etc.
Do you have instructions how to use the chart with postgresql? Is it sufficient to fake the content of the required secrets?

Do I understand you correctly that you would like to have the secret data keys configurable for these secrets?
Other connection secrets like username, password, host etc are configurable in the ZITADEL yaml as described here. For secret ZITADEL config, you can use the secretConfig property in the Helm values.

Hi,
the helm chart requires certain secrets to be present. These are generated by cockroachdb. In a postgresql setup there is no cockroackdb, thus no secrets. This prevents the pods from starting since the secret cannot be mapped.
If I use empty fake secrets, the init container fails since it tries to copy //* to some destination. This fails if the secret is empty and no files are present.

An addition:
I can use fake secret content since the it is not used in the actual database setup in case of postgresql. Deployment fails nonetheless since the zitadel init code requires administrative access to the postgresql database. Since we have a shared postgreql cluster, we cannot use administrative credentials in third party applications. I've already created an issue in the zitadel core project, since code change will be required to solve this problem.

@Be-El, @vavsab I created a user story in #91 that should address your issues. Please track and participate in #91.


--buildpack using / changing?

Could I migrate an app using --buildpack without creating a new one?  It's already on Cedar and heroku says I can't migrate to the same stack...  I basically want to use the latest stable node - already did heroku create and it works, but I'd like to keep all the configuration that comes with the existing deployment.  Alternatively, is there a tool to copy the plugins / settings / env vars from one app to another?

Behind the scenes --buildpack is basically an alias to setting a config variable.  So you should be able to change the config var, something like heroku config:add BUILDPACK_URL=new_url.  That should do the right thing, but let us know if it doesn't fix things for you.  Thanks!

Thanks man, it works beautifully.

My pleasure, some of the stuff like this is still a little mysterious even to me (I had to ask somebody to find out you could do this).  Hopefully we can get better interfaces and docs moving forward to make it all easier though.  Thanks for the feedback!


Execução do PowerToys

Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.18362.0
IntPtr Length: 8
x64: True
Date: 09/12/2020 09:26:13
Exception:
System.Reflection.ReflectionTypeLoadException: Unable to load one or more of the requested types.
não é uma aplicação de Win32 válida. (0x800700C1)
at System.Reflection.RuntimeModule.GetTypes(RuntimeModule module)
at System.Reflection.RuntimeModule.GetTypes()
at System.Reflection.Assembly.GetTypes()
at PowerLauncher.Plugin.PluginsLoader.<>c__DisplayClass2_1.b__1()
at Wox.Infrastructure.Stopwatch.Debug(String message, Action action)
at PowerLauncher.Plugin.PluginsLoader.CSharpPlugins(List1 source)    at PowerLauncher.Plugin.PluginsLoader.Plugins(List1 metadatas)
at PowerLauncher.Plugin.PluginManager.LoadPlugins(PluginSettings settings)
at PowerLauncher.App.<>c__DisplayClass14_0.b__0()
at Wox.Infrastructure.Stopwatch.Normal(String message, Action action)
at PowerLauncher.App.OnStartup(Object sender, StartupEventArgs e)
at System.Windows.Application.OnStartup(StartupEventArgs e)
at System.Windows.Application.<.ctor>b__1_0(Object unused)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
System.BadImageFormatException:  não é uma aplicação de Win32 válida. (0x800700C1)
Quando eu baixo o App PowerToys e clico em "Finish" aparece essa mensagem. Alguém pode me ajudar?

error is saying: it is not a valid Win32 application. (0x800700C1) which is pretty vague.  You are on Win10 1903 so this should work.
@deboralco would it be possible for us to debug this on your computer.  One of the plugins is failing to load.
zipping up %localappdata%\Microsoft\PowerToys\PowerToys Run\ and putting in here could also be helpful.


AnalyzeDocumentChain

You can omit the base class implementation. Just one file where this works is enough, we'll highlight the interfaces a bit later.
You can use GPT-4 for initial implementation
Tests are encouraged but not required. It's better to do something minimal than nothing.

References:

https://api.python.langchain.com/en/latest/chains/langchain.chains.combine_documents.base.AnalyzeDocumentChain.html

missed this one, was done in https://github.com/tryAGI/LangChain/pull/43/files

missed this one, was done in https://github.com/tryAGI/LangChain/pull/43/files


how to disallow the user to rate ?

I want to show the current rating that is saved in the database but don't want the user to have the ability to change it

Hi @hvlong
Please take a look at #67
Thanks


comprehensive tests for throttle

This probably relates to #496, so be aware of that if the work happens to be split.

I can do this too.


CHANGES.md missing 0.5.0 release

Hi!
The 0.5.0 release is missing from:
https://github.com/ngReact/ngReact/blob/master/CHANGES.md

Thanks should be fixed now

Many thanks :-)


Non Tradable ERC20 Token

Added NonTradableToken.sol
Non tradable ERC20 tokens, those are non transferable tokens.
Why is this required ?

When a company ABC goes to private investors before even presale/ICO. Company wants to collect ETH using the Mintable Smart Contract, but company do not wants to make these tokens tradable.
When ABC company launch their main ICO, then company will pre-fill the tokens for the private investors in main ICO Smart Contract.

I believe this can be implemented using LimitedTransferToken as a base

@adklempner If contract creator uses  has LimitedTransferToken as a base then he has to do the following to make it Non Tradable

Override transferableTokens() method

What's the point of using the ERC20 standard for a token that is not tradeable?  Just create a contract that's doesn't have transfer functions

When a company looking from some private investors to invest in the ICO even before the actual ICO. In that case company would not want the private investors to trade the tokens they minted by giving ETHs.

You can do that with a PausableToken

With PausableToken, I see a security issue.
What if the owner's private key is compromised in future. In that case hacker will be able to UnPause the tokens anytime.
We must create a contract full proof that no possibility of hacking is there.

What if the owner's private key is compromised in future

Thats why you use a multisig address for ownership

Or a hardware wallet.
But I will ask again, if you can't unpause the transfer at some point. Why would you need functions used for transfer?
You could also put the tokens for the pre-investor inside a TokenTimeLock

Thats why you use a multisig address for ownership
@reiven Can we make a multisig wallet the owner of a contract called PausableToken ?

@SylTi When I will create my token related contract I will use the ERC20.sol and import it.
Are you suggesting that I should not import those files as it is and make changes into those files and remove the transfer() function ?

@jchittoda sure! you can make a multisig address owner of any contract

What I'm saying is, this pull request, in my opinion, is not useful. It's essentially the same thing as PausableToken without the possibility to turn it on.  And inheriting from an ERC20 token to disable everything is a bad approach, if you don't want the functions, just don't inherit from it.
If you want help with your own code, this is not the place. You can join us on the slack channel we will be happy to help.

Agree with @SylTi and with others.


Update sbt-scala-native to 0.4.9

Updates org.scala-native:sbt-scala-native from 0.4.4 to 0.4.9.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-native", artifactId = "sbt-scala-native" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.scala-native", artifactId = "sbt-scala-native" }
}]

labels: sbt-plugin-update, early-semver-minor, semver-spec-patch, commit-count:1

Superseded by #70.


Problem with delete

Hello
I  am trying to use the mapDB for a small test. Therefore I need to add some data to the db and delete it. Adding Data works just fine.  I can do this multiple times without errors.
Delete the Hashmap works fine too, but only one time.
If I then try again to add data everything is fine. But the second call of my delete Method gives then the following Error:
FATAL EXCEPTION: main
Process: com.example.michael.dbmanager_debug_application, PID: 4902
java.lang.InternalError: data were not fully read, check your serializer
at org.mapdb.Store.deserialize(Store.java:265)
at org.mapdb.StoreDirect.get2(StoreDirect.java:431)
at org.mapdb.StoreWAL.get2(StoreWAL.java:318)
at org.mapdb.StoreWAL.get(StoreWAL.java:302)
at org.mapdb.EngineWrapper.get(EngineWrapper.java:60)
at org.mapdb.AsyncWriteEngine.get(AsyncWriteEngine.java:361)
at org.mapdb.Caches$HashTable.get(Caches.java:213)
at org.mapdb.HTreeMap.recursiveDirClear(HTreeMap.java:734)
at org.mapdb.HTreeMap.recursiveDirClear(HTreeMap.java:728)
at org.mapdb.HTreeMap.recursiveDirClear(HTreeMap.java:728)
at org.mapdb.HTreeMap.recursiveDirClear(HTreeMap.java:728)
at org.mapdb.HTreeMap.clear(HTreeMap.java:704)
at org.mapdb.DB.delete(DB.java:1135)
For add i Use the following
map = db.createHashMap("NAME").make();
        for(int i = 1; i <= dataarray.getSize(); i++){
            map.put("timestamp" + (oldSize + i), "" + dataarray.getZeitstempel(i-1));
            map.put("header" + (oldSize + i), "" + dataarray.getHeader(i-1));
            map.put("data" + (oldSize + i), "" + dataarray.getData(i-1));
            fahrtxmap.put("size", "" + (oldSize + dataarray.getSize()));

For delete I use  db.delete("NAME").
Can you help me to find my mistake?

Sorry 1.0 and 2.0 is not supported anymore


Time Series support: viewing a report in Analyze should show the time range the data comes from

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1409
Originally created by @dmor on 2019-12-09 19:35:23

When in Analyze, the start date of the data (which was set in the pipeline configuration) should be visible to the user so they know the context for what they are looking at.
Something simple like this would help:

A next step iteration would be to allow the user to filter their data by date, so they don't have to include everything the ELT has pulled. I'm capturing this in a separate issue

View 26 previous comments from the original issue on GitLab


webpack doesn't run on Windows

Seems like the current implementation of webpack/lib/MemoryOutputFileSystem.js doesn't run on windows machines. It utilizes unix-style separators.
I've replaced all occurences with path.sep and uncommented the checks like this:
path = path.split(pathUtil.sep);
//if(path[0] != "") return callback(new Error("Invalid path " + path.join("/")));

and now it's running.

ridiculous. tried to use it. not working with windows? lol.
hopefully this framework will be not succesful.

@chrisiDC webpack is working on Windows. Check your configuration and stop flaming.

with 20 years of experience i must be a complete fool not to be able to set it up within 4 hours.. have to quit my job.
but sorry if i flame u. anyways, i dont think it works in windows out of the box...
cheers

I'm sorry. I used to have a Windows machine and I had no problems. Even @sokra, the owner of the project, has a Windows machine, so I bet webpack runs on Windows. Without any error or hint, we can't help you.

i will provide an error hint asap. currently i am in the office :)
thx for your feedback


[Question]: how to make the result more controllable

请提出你的问题
Background: We'd like to do some generation based on the items we owned, which is seems like we draw some pictures base on our own items.
Process:

I did the "screenshot" for tens of photos of the items based on our 3D model in different distance and angle
I add the special token by tokenizer.add_tokens(['SPECIAL_TOKEN'], True) as symbol/pin for the item I added
After 1k~3k steps of fine tuning on ether dreambooth (sks SPECIAL_TOKEN) or textual_inversion (<textual_inversion>), i got 2 customized model
I input the prompt of '$SPECIAL_TOKEN' + other odds and ends to try to draw the pictures of my item
The result is only 50~60% similar with the item, e.g. input a table, white, half height, output a table in brown/only several hight .etc

I know i can force the result is 100% same with the original one as it would get influenced by the base model, but how can i raise the similar rate at least reach 80~90%? e.g. I input a white table, output a white table unless i high light the table should be brown one in my prompt?

Is it able to ensure the result is quite similar with the original ones I injected? or it's totally a false proposition to do this attempt?
If yes, what I can do next in professional perspective?

to pick other model as base?
prepare much more feed images?
more steps/epoch with less learn rate?
Or no, try other technology e.g. face swap .etc ?

当前仓库的微调（与Hugginface一致）代码存在已知问题，训练效果没有原来仓库的那么好，首先需要排除代码的问题。

你可以先使用 https://github.com/victorchall/EveryDream-trainer 这里的dream代码进行微调，因为有微调这个 https://huggingface.co/ringhyacinth/nail-set-diffuser 权重的用户就反馈了使用原版代码比HF或PPDiffusers的代码要好。


Competition for 3 possible material design themes

Feature Request
Desired behaviour
As a part of this migration, come up with two or three demo-able themes. Convert one or two representative pages to use material widgets (my tools, accounts) as a demonstration.
As a part of this, create one theme based on Dockstore's current colours and two can be more creative.

Seems cool, only on Mac OSX for some reason.
https://material.io/tools/theme-editor/

https://material.io/tools/color/#!/?view.left=0&view.right=0&primary.color=21335b&secondary.color=487980

I think this is done for now. I'll create another issue to create a style guide for Dockstore developers for the Wiki.

Complete, looks good


BHM Update for v0.5

Fixes to YAML configuration file formatting.
Rename opt_prov to optical_prov.
Several changes to queries to optimise Q3C queries.

Hi José
many thanks for working through the changes needed to get these queries working properly. For target_selection v1 I'll attempt to copy your the materialised CTE trick into the other BHM cartons, where this will speed up processing significantly.
I have copied some minor updates and fixes from the bhm_v05_3 branch into the albireox/bhm-update branch. These changes were limited to the SPIDERS AGN and clusters cartons, and were mainly adjustments to the target priority ladder. There are also some fixes to avoid nonsensical magnitudes being emitted by the bhm_spiders_agn_supercosmos carton.
The bhm_rm_base carton does not provide targets - only shared parameters for the other bhm_rm_* cartons, therefore I have removed it from the list of 'live' cartons in the target_selection.yml file.
I'm happy to approve your pull request, but some conflicts with the main branch seem to be preventing this. I'll not attempt to fix these in case I screw something up.
Thanks again
Tom

I merged back main and fixed a couple linting issues, but otherwise it looks fine, so I'll merge.


loggerd on PC

issue #2277

Thanks for the PR, but the raw logger isn't really used. Ideally, we want all the logic in loggerd.cc and encoder_thread to run on a similar encoder on PC as to what runs on device, meaning no ifdefs for different encoder types. Let me know if you want further clarification.

Thanks for the PR, but the raw logger isn't really used. Ideally, we want all the logic in loggerd.cc and encoder_thread to run on a similar encoder on PC as to what runs on device, meaning no ifdefs for different encoder types. Let me know if you want further clarification.

some generalization similar to camerad would be nice!

Actually, I think I have a nice way to do this in a few lines. Let me try that first.


FIX: missing case in GetMethodByInstructionPointer()

Same implementation as in ClrMD 1.1

I still have the unnecessary commit list even though the modified file is fine.
I'm changing the way to create the PR and resubmitting it ASAP.


arch4u-recommended-third-party-ruleset.xml:83: category/java/documentation.xml - add rules details in table defined in readme.md -->

The puzzle 27-723e5b53 from #27 has to be resolved:
https://github.com/dgroup/arch4u-pmd/blob/4c6fe8e750435a5c5f1fff36516a93e0f315eae2/src/main/resources/io/github/dgroup/arch4u/pmd/thirdparty/arch4u-recommended-third-party-ruleset.xml#L83-L83
The puzzle was created by Yurii Dubinka on 08-Feb-22.
role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 27-723e5b53 has disappeared from the source code, that's why I closed this issue.


Key remaps not working on Google Chrome

ℹ Computer information

PowerToys version: 0.23.2
PowerToy Utility: Keyboard Manager
Running PowerToys as Admin: Yes
Windows build number: [run "winver"] 20236

📝 Provide detailed reproduction steps (if any)

Remap Keys on Powertoys
Try to use them while operating on Google Chrome

✔ Expected result
If I remap my search key, for example, to be Ctrl + F instead of the default behavior, I expect that to also apply when using Google Chrome, but Chrome overrides it no matter which settings I use on its advanced settings.
❌ Actual result
Keys behave as if there was no remap.
📷 Screenshots

@TheSimArchitect
Could you please provide the exact keys that do not work for you? Screen from Keyboard Manager would be great.

Browser Search Keys, Browser Next, Browser Previous

The Menu Key (besides right Alt) remapping works normally, though.
My keyboard is the one in the image (MS Natural Ergonomic Keyboard 4000)
Sometimes Browser Previous and Browser Next behave as expected (remapped), but the Browser Find key always opens Windows Search instead of sending a Ctrl F, sadly.
The key binding works normally on Notepad ++, for example (Ctrl F as expected).

@TheSimArchitect when you press the Search key in Remap Shortcuts -> Type button what does it show? I don't have the exact keyboard you have, but I have the Ergonomic Desktop keyboard and when I click the Search key on that it shows Win+Shift+F21, which means it is not a Browser Search key, its a Windows Search key which is internally a shortcut and not a real key (like the Office key). Can you check what does it show for you?

The key bindings above were produced by using the key.
Power toys shows "Browser Search" when I use the key and the functionality works as expected on other apps like Notepad ++, but Chrome is bypassing PowerToys and sending the regular "Windows Search" command to the operating system as you mention instead, when it has the focus.
Ironically enough, Chrome is the very place where I need to use the key binding more often, since I have some repetitive tasks that require me to use Ctrl + F and using that key would be much faster.

On my keyboard search key is the Browser Search key as well.
Remapping Browser Search -> Ctrl F works for me.

@TheSimArchitect
Just to clarify you get Browser Search key by clicking on the 'Type' button and then type your search key, right?
Could you try remapping Browser Search -> Ctrl Q and verify if it works for Chrome?

Yes. And it does work everywhere except on Google Chrome (from what I tried this far).
It's totally ok, though, I wanted you to know about the bug. I am partially stuck with Chrome because most of my audience uses it, otherwise I'd probably have moved on to another browser...
Same thing with Ctrl Q
Same thing with either left or right control
Same thing with Alt F
No matter what I try, if I am trying to use it on Chrome Windows Search Windows will show up instead.

@mykhailopylyp were you able to check if the remapping works for you on Chrome? I don't have a Browser Search key so I can't test this out. The only thing I was able to do was remap a key to Browser Search and validate that works on both Edge and Chrome.
@TheSimArchitect so on Chrome if you just press Browser Search (unremapped) it works fine? Can you send the list of all your remappings? Since you are saying it happens if you remap Browser Search to pretty much anything Win Search shows up, it's possible that maybe Chrome internally sends a key event for one of the other keys that you have remapped which in turn shows Chrome. Could you try removing all your remappings except the Browser Search one and see if it still doesn't work?

@arjunbalgovind
I can not reproduce it.

@TheSimArchitect
are you running Chrome in dev mode or do you have any special extension that might be interfering?

I use this version of Chrome.
I don't have any special extension that should interfere.
I will uninstall Chrome Dev and install Chrome Beta Instead.

So... Chrome Beta Works. I guess only a few applications override remaps even though I am running PowerToys in Admin mode.
Now, did you fix something around the issue on the latest update of Powertoys? It could be that too, since I did not try Chrome Dev with the new version of PowerToys or the old version of PowerToys with Chrome Beta.

@TheSimArchitect
Could you please try the following scenario?

Delete all remappings except Browser Search -> Ctrl + F
Start chrome(or restart it) and verify if remapping works. Verify it on the blank tab as well.
Turn off the Keyboard manager. It can be done in settings.
Return to chrome and press the Browser Search key. What is the behavior? From what I know it should focus Search google or url box.
Turn on the Keyboard manager
Return to chrome and verify if remappings work. Verify it on a blank tab as well.
Send us files from C:\Users\{UserName}\AppData\Local\Microsoft\PowerToys\Keyboard Manager folder. Replace {UserName} by your system name.

I think I found the cause!
It's a conflict between Microsoft Mouse and Keyboard Center overriding the key stroke.
I just formatted my computer and before I installed it, the key binding for the search key was working properly, then it became broken after I installed it.
A workaround is to set up the macro for the key direcly on Microsoft Mouse and Keyboard Center instead.
Can't avoid having Microsoft Mouse and Keyboard Center installed because the = ( and ) above the numpad won't work anywhere without it.

After installing Microsoft Mouse and Keyboard Center my browser search key started triggering windows search.  It is a compatibility issue.

@TheSimArchitect
thanks for the information, added to https://github.com/microsoft/PowerToys/wiki/Application-Compatibility#keyboard-manager

Thank you very much! I am very glad that I could find the cause, so other users facing the same problem know what to do or at least what's happening. 😇


Using downwardApi without apiVersion leads to never-ending pod delete/recreate loop

metacontroller version 2.2.0
Using the downwardAPI in a pod spec seems to lead to a never ending cycle of destroying/recreating the pod. Example:
downwardAPI:
 items:
   - path: pod_name
     fieldRef:
       fieldPath: metadata.name

A workaround is to add apiVersion: v1 under the fieldRef from our side.

hi @gavin-nia , can you post logs or create a small code which recreates the behavior ? It would speed up debugging. I am a bit unfamiliar with those parts of PodSpec.

Thanks for the quick response @grzesuav . Will try to get that to you today - will the default log level tell you enough information?
To reproduce, it should be possible to see this behavior by modifying one of the existing examples (e.g. catset or bgd) so I will see if I can create a patch for that also.

Hi @grzesuav , Not sure if this makes sense but you can apply the following patch to the 'catset' example then try and run it. You will see that the pods are continually being destroyed and recreated.
diff --git a/examples/catset/my-catset.yaml b/examples/catset/my-catset.yaml
index a6cfc02..585370d 100644
--- a/examples/catset/my-catset.yaml
+++ b/examples/catset/my-catset.yaml
@@ -36,6 +36,13 @@ spec:
         volumeMounts:
         - name: www
           mountPath: /usr/share/nginx/html
+      volumes:
+      - name: "pod-info"
+        downwardAPI:
+          items:
+          - path: "pod_name"
+            fieldRef:
+              fieldPath: metadata.name
   volumeClaimTemplates:
   - metadata:
       name: www

Thanks for the quick response @grzesuav . Will try to get that to you today - will the default log level tell you enough information?
To reproduce, it should be possible to see this behavior by modifying one of the existing examples (e.g. catset or bgd) so I will see if I can create a patch for that also.

I believe with 7 you will see patch requests logged, so at least see whats is being constantly  updated

Hi @grzesuav , Not sure if this makes sense but you can apply the following patch to the 'catset' example then try and run it. You will see that the pods are continually being destroyed and recreated.
diff --git a/examples/catset/my-catset.yaml b/examples/catset/my-catset.yaml
index a6cfc02..585370d 100644
--- a/examples/catset/my-catset.yaml
+++ b/examples/catset/my-catset.yaml
@@ -36,6 +36,13 @@ spec:
         volumeMounts:
         - name: www
           mountPath: /usr/share/nginx/html
+      volumes:
+      - name: "pod-info"
+        downwardAPI:
+          items:
+          - path: "pod_name"
+            fieldRef:
+              fieldPath: metadata.name
   volumeClaimTemplates:
   - metadata:
       name: www

could you also post the version which fixes the issue, to avoid  ambiguity?

Hi @grzesuav
This is the version that works OK (added apiVersion).
diff --git a/examples/catset/my-catset.yaml b/examples/catset/my-catset.yaml
index a6cfc02..f479ab2 100644
--- a/examples/catset/my-catset.yaml
+++ b/examples/catset/my-catset.yaml
@@ -36,6 +36,14 @@ spec:
         volumeMounts:
         - name: www
           mountPath: /usr/share/nginx/html
+      volumes:
+      - name: "pod-info"
+        downwardAPI:
+          items:
+          - path: "pod_name"
+            fieldRef:
+              apiVersion: v1
+              fieldPath: metadata.name
   volumeClaimTemplates:
   - metadata:
       name: www

@gavin-nia I debug it, it seems kubernetes is defaulting on server side - it adds apiVersion: v1 so when metacontrollers is doing diff it see that object need to be updated. So I would say it behaves as it should. Metacontroller is detecting that actual state is different than expected.
I am unsure if it can be handled on metacontroller side. I will dig into https://metacontroller.github.io/metacontroller/api/apply.html#dynamic-apply implementation, however cannot promise anything.

I implemented a fix in metacontroller implementation, however IMO switching to SSA (https://github.com/metacontroller/metacontroller/issues/501) would be closing point for this

:tada: This issue has been resolved in version 3.0.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[Feature Request] DisabledTime mechanism for Date Time Picker

Existing Component
Yes
Component Name
DateTimePicker
Description
There is a method on the date time picker (disabledDate) which will disable a date, however it doesn't seem to disable the time as well.

use selectableRange in pickerOptions https://codepen.io/anon/pen/xNyrBR?&editable=true

@wacky6 Hi, it's really a useful feature for me, but it seems that datepicker with the type datetimerange has not support selectableRange in pickerOptions yet.
How could I use it in datetimerange datepicker ?


FATAL EXCEPTION: p.f0 Dispatcher

Before you create a new issue, please search for similar issues. It's possible somebody has encountered this bug already. PLEASE REMOVE THIS LINE TO ACKNOWLEDGE THAT AN ISSUE DOESN'T ALREADY EXIST FOR THIS BUG
Describe the bug
There is no problem in the test environment, the formal environment payment will report errors
java.lang.IllegalAccessError: Method 'void com.paypal.pyplcheckout.home.viewmodel.MainPaysheetViewModel.h(com.paypal.pyplcheckout.home.viewmodel.MainPaysheetViewModel)' is inaccessible to class 'b.h0.f.m.b.c' (declaration of 'b.h0.f.m.b.c' appears in /data/app/~~a0Lqr52CIf93otvM-REm3A==/
com.paypal.checkout:android-sdk:0.8.7
I guess it was a mistake caused by confusion?
I hope to solve this problem as soon as possible，thanks!

I have confirmed that the formal environment can be paid normally after I remove the confusion

@wangdabin2333
It looks like you are trying to access something that's either internal or private to com.paypal.pyplcheckout.home.viewmodel.MainPaysheetViewModel. You shouldn't really be accessing anything in there.
Could you please make me understand what your use-case is and what you are trying to do?

@RequiresApi(api = Build.VERSION_CODES.M)
public void invokePayPalPay(AliPayBean.PaypalAndroidVO paypalAndroidVO, BaseActivity context, final DoResult doResult) {
PayPalCheckout.startCheckout(new CreateOrder() {
@Override
public void create(@NonNull CreateOrderActions createOrderActions) {
ArrayList purchaseUnits = new ArrayList<>();
PurchaseUnit purchaseUnit =
new PurchaseUnit.Builder()
.amount(new Amount.Builder().currencyCode(CurrencyCode.USD)
.value(String.valueOf(paypalAndroidVO.getTotal()))
.build())
.description(paypalAndroidVO.getOrderDescription())
.customId(paypalAndroidVO.getOrderId())
.build();
purchaseUnits.add(purchaseUnit);
Order order = new Order(OrderIntent.CAPTURE,
new AppContext.Builder().userAction(UserAction.PAY_NOW).build(), purchaseUnits, ProcessingInstruction.NO_INSTRUCTION);
createOrderActions.create(order, (CreateOrderActions.OnOrderCreated) null);
}
});
PayPalCheckout.registerCallbacks(new OnApprove() {
@Override
public void onApprove(@NonNull Approval approval) {
approval.getOrderActions().capture(new OnCaptureComplete() {
@Override
public void onCaptureComplete(CaptureOrderResult captureOrderResult) {
Log.e(TAG, String.format("CaptureOrderResult: %s", captureOrderResult));
if (captureOrderResult instanceof CaptureOrderResult.Success) {
OrderResponse orderResponse = ((CaptureOrderResult.Success) captureOrderResult).getOrderResponse();
if (orderResponse != null) {
PurchaseUnit purchaseUnits = orderResponse.getPurchaseUnits().get(0);
Payments payments = purchaseUnits.getPayments();
Capture capture = payments.getCaptures().get(0);
doResult.confirmSuccess(paypalAndroidVO.getOrderId(), capture.getId());
} else {
doResult.confirmError();
}
} else if (captureOrderResult instanceof CaptureOrderResult.Error) {
doResult.confirmError();
}
}
});
}
}, new OnCancel() {
@Override
public void onCancel() {
doResult.customerCanceled();
}
}, new OnError() {
@Override
public void onError(@NonNull ErrorInfo errorInfo) {
doResult.confirmError();
}
});

@wangdabin2333 - This looks to be an issue with proguard being enabled. When you're running in the test environment, do you have proguard disabled?


First version of camera efficiency module and application

@orelgueta,
I implemented the camera_efficiency module which is responsible to simulate and analyze the results of testeff. It's still a simple version. It calculates the NSB and cherenkov efficiencies, export a csv file and plot them. It still doesn't calculate the total NSB rate. You can show me how to do it and I implement afterwards.
I added a Camera attribute to the TelescopeModel.
I added few functions to the Camera class to get the funnel shape, lightguide file and diameter.
The application is similar to the camera fov. The plotting is simple, no spetial cosmetics were used. I tried to find an example using your vizualise module but I didn't find it. We can make the plotting fancier afterwards as well.
In case you are not able to test it, I send the plots below.
Please, have a look whenever you have time for it.

I skimmed it quickly, all looks OK at first glance. I will take a more detailed look next week probably (too many meetings today).
However, I can already say that the efficiency calculations are a bit more involved due to some exact definitions of CTA. I think I will take advantage and implement the calculations in this branch before we merge to trunk, what do you think?

I added few functions to the Camera class to get the funnel shape, lightguide file and diameter.

I did this myself yesterday actually as well. I will use your code, so to avoid conflict, but I have to make some changes to it, some minor and one a little bigger. Will work on it next week.

I skimmed it quickly, all looks OK at first glance. I will take a more detailed look next week probably (too many meetings today).
However, I can already say that the efficiency calculations are a bit more involved due to some exact definitions of CTA. I think I will take advantage and implement the calculations in this branch before we merge to trunk, what do you think?

I think it's a good idea. I just wanted to do this pull request to show you the code.

Great, I will try to do it as quickly as possible so not to delay this PR too long.

I just noticed this was done on your develop branch. I assume you will still work on it in the next few days? I would like to work on it as well before we merge and it would be good to avoid conflicts. May I ask you to work on a different branch until this PR is done?
We should actually be a bit more organised in general with branches I think. We both created "general" branches to work on and simply do everything on those branches. That is probably not very good practices. I suggest we create small branches for every small project and delete them once the PR is accepted. Do you agree?

I just noticed this was done on your develop branch. I assume you will still work on it in the next few days? I would like to work on it as well before we merge and it would be good to avoid conflicts. May I ask you to work on a different branch until this PR is done?

Yes. I have other stuff to do meanwhile, don't worry.

We should actually be a bit more organised in general with branches I think. We both created "general" branches to work on and simply do everything on those branches. That is probably not very good practices. I suggest we create small branches for every small project and delete them once the PR is accepted. Do you agree?

For only two developers, I still think that makes sense to have our personal branches to implement small features, spatially framework stuff.
But you are right that for this kind of project it's better to create specific branches. We just need to be careful to really finish the pull requests and delete the branches to avoid accumulating old ones.


Guard against optional, but required by us, axom headers

It is confusing when these fail to be present.

LGTM


Training data format

Hi Paul-krug,
Thanks for your nicely written TCN repo. I am quite new to this time-series prediction area, and I'm trying to use TCN as a robot manipulation controller, which would usually use this: robot_action_at_time_t = TCN(environment_observation_at_time_{t-1},..., at_time{t-m+1}) by using the historical m steps observation to predict the current action. I tried TCN with the input (1, 153, 1000), where 1 is the batch_size, 153 is the observation dimension, and 1000 is the total time step for a whole trajectory. The action dimension is 25, so the output_projection in TCN is 25. I noticed that the output is (1, 25, 1000), where 1 and 25 are understandable, and 1000 is the total time steps of the manipulation. I have the following questions:

During training, should I use a sub-trajectory with a moving window of fixed length (e.g. 50 steps for each, slicing the whole sequence 1000 into many small pieces) or simply use the whole sequence as input is recommended?
For those actions (predictions) in the early part of the trajectory (early part of the time steps, e.g. the second steps in 1000), are their input also got zero-padding (e.g. the second steps in 1000 are zero-padded with 998 zeros) as you described in the streaming inference? You mentioned: For kernel sizes > 1, a TCN will always use zero padding to ensure that the output has the same number of time steps as the input.
During inference (when the robot starts to manipulate), should I use the same number of time steps (sequence lengths, like 50 as a moving window input) as training and I manually do data shift to discard oldest observations and include newest observations? Will different numbers of sequence lengths influence the performance?
I really appreciate your replay!
Zhaole

Sorry for the late reply, but I will try to answer your questions:

It is generally ok to use the whole sequence as input, but you may use shorter slices if your specific problem requires it.
Zero-padding is used in the convolutions in order to keep the amount of input and output time steps equal. No padding is required on your targets (unless you explicitly want to introduce some sort of past/future prediction).
If you trained a fully causal network, then it does not matter how many frames you give it at inference time, i.e. the prediction is the same regardless of whether it takes place on the entire sequence or frame by frame.

I hope that helps.
Cheers,
Paul


[BUG]sourceMapFilename Conflict multiple assets

System Info
System:
OS: macOS 13.3.1
CPU: (10) arm64 Apple M1 Pro
Memory: 194.64 MB / 32.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 16.15.0 - ~/.nvm/versions/node/v16.15.0/bin/node
Yarn: 1.22.18 - ~/.nvm/versions/node/v16.15.0/bin/yarn
npm: 8.6.0 - /opt/homebrew/bin/npm
Browsers:
Chrome: 113.0.5672.126
Firefox: 103.0.2
Safari: 16.4
Details

Reproduce link
https://github.com/B1anker/multiple-assets
Reproduce Steps

yarn install
yarn build

Seems webpack also has this problem

@faga295 Sorry, can you provide an example of the problem? I tried to reproduce, but no luck

https://github.com/faga295/rspack-reproduce/tree/sourcemap_multiple_emit reproduce step: pnpm build:webpack

@faga295 I see, that is the intresting issue, when you have child compilation and use default entry (i.e. just pass entry) you will have two files:

main.css
main.js

Using sourceMapFilename: "sourcemap/[name][ext].map" is not good idea, because you will have sourcemp/main.map file for JS and CSS, that is why webpack reports about it, default value sourceMapFilename: "[file].map[query]" (file already contains an extensions).
Even more look at dist, you have main.map, it is not good idea to have just a map extension, I would recommend to always have an original extension, i.e. file.js.map/file.css.map/file.something.map.
Under the hood in plugins you can have any numbers of child compilations, we can't control it.
Perhaps we could write a warning about this, but in the real world, I have seen similar uses in various exotic situations, so it will be a bad DX for them, maybe the best solution is improve documentation and write Always use [ext] in sourceMapFilename, unless you understand why you don't need it


Use core_layout template

What
Update controllers to use core_layout template instead of wrapper.
Why
frontend is one of the few remaining apps relying on a wrapper slimmer template – the other one being licence-finder. This PR updates all controllers after the initial piece of work to move slimmer configuration at the controller level – intended to ease the transition – to use the core_layout component.
This piece of work is important as wrapper was conceived to leak styles on all basic elements violating our isolation principle.
What this means in practice is that the content presented on pages rendered by these controllers won't benefit from global styling and would require explicitly declared styles for each element using components and basic govuk- classes (e.g. govuk-body).
Controllers in this application (the ticked ones are addressed in this PR all controllers are addressed now):

[x] application_controller.rb
[x] completed_transaction_controller.rb
[x] error_controller.rb
[x] find_local_council_controller.rb
[x] help_controller.rb
[x] licence_controller.rb
[x] local_transaction_controller.rb
[x] place_controller.rb
[x] simple_smart_answers_controller.rb
[x] transaction_controller.rb
[x] travel_advice_controller.rb

Visual changes
While this change shouldn't affect the application visually, it addresses some layout and styling issues along the way as shown in the examples below for completed_transactions, licence and transactions.

BeforeAfter

Trello card

This looks great @alex-ju! It looks like it needs a rebase to fix a conflict on app/views/simple_smart_answers/_current_question.html.erb though.
I'll check the rest of it out in the meantime

This looks great @alex-ju! It looks like it needs a rebase to fix a conflict on app/views/simple_smart_answers/_current_question.html.erb though.
I'll check the rest of it out in the meantime

Thanks, @sihugh. I rebased now.

Code changes all look sensible. I can't find all the pages these changes impact, would it be possible to provide URLs? Either in the PR description or individual commit messages.

It's been almost 6 months now – I did have them in my history, but it's hard to find them at this point. Since most of them are not documented in the main readme file I'm usually using relying on routes to deduce them, especially because they sometimes change over time. I know it's not easy, it wasn't for me either.

I'm redeploying the preview app and will update this comment with at least one link per controller (although I tested the views exhaustively back then).

Rebased to master. Will merge once the checks pass and will test in integration.

@alex-ju I think this PR might have broken the rendering of tables in simple smart answers.
For example on this page: https://www.gov.uk/student-finance-forms/y/english-student-part-time/apply-for-student-loans-and-grants/2018-to-2019/no/no/yes
In this PR:

Before this PR:

I'm not sure why this would have broken the styles though - do you have any ideas?

@alex-ju I think this PR might have broken the rendering of tables in simple smart answers.
For example on this page: https://www.gov.uk/student-finance-forms/y/english-student-part-time/apply-for-student-loans-and-grants/2018-to-2019/no/no/yes
In this PR:

Before this PR:

I'm not sure why this would have broken the styles though - do you have any ideas?

@thomasleese I don't think it's affected by this change, as I see the table issue in other repos too. Will investigate it and report back, thanks for flagging it.

@alex-ju Sorry I forgot to follow up with this! I deployed https://github.com/alphagov/frontend/pull/2386 which fixed the problem.


[design]: add internal package to hide our implementation details

I would like to add a internal package such as "com.coreos.jetcd.internal" and move some implementation classes like EtcdKVImpl to it.
And we should document that all the classes in package internal are our internal implementation detail and should not be used directly.
Then We can update them freely.

@skyao I already have a design proposal when I try to implement this client, you may take a look.

"com.coreos.jetcd.internal" is added.


config: make ini values optional

I'd like the INI key to be permissible without an = character. Currently, this
works:
key=

But this does not:
key

It seems not-very-intuitive to require that = sign to be present.
use case
[hosts]
es1.nyc.example.com
es2.nyc.example.com
es1.lax.example.com=protocol:https, dc:lax
es2.lax.example.com=protocol:https, dc:lax

There's also quite a few places where we shove data off to separate files to get list loading ability, and this permits storing that data in an ini [section].

I've wanted to do this before as I much prefer using .ini files over list types for index style lookups because of the obvious speed advantage in doing so and this obviously makes that possible.
I'd resisted changing it like this though as we're drifting further away from the standard.
So whilst I think this is a good change; it 1) needs documenting and 2) I think it needs @baudehlo to approve.

+1

On Jun 14, 2015, at 6:27 AM, Steve Freegard<EMAIL_ADDRESS>wrote:
I've wanted to do this before as I much prefer using .ini files over list types for index style lookups because of the obvious speed advantage in doing so and this obviously makes that possible.
I'd resisted changing it like this though as we're drifting further away from the standard.
So whilst I think this is a good change; it 1) needs documenting and 2) I think it needs @baudehlo to approve.
—
Reply to this email directly or view it on GitHub.

docs added.


can't upload files or folders into path with special characters like [ ]

Hey, when i'm trying to commit files into folders with special characters
i've got this
  mput: resources/[standalone]/[phone]/lb-phone_prop/.fxap: no files found
  mput: resources/[standalone]/[phone]/lb-phone_prop/fxmanifest.lua: no files found
  mput: resources/[standalone]/[phone]/lb-phone_prop/stream/lb_phone_prop.ydr: no files found
  mput: resources/[standalone]/[phone]/lb-phone_prop/stream/lbphone_props.ytyp: no files found

here is my workflow
name: 🚀 Développement - Star'Island
on:
  push:
    branches: [dev]

jobs:
  deploy:
    runs-on: ubuntu-latest
    steps:
      - name: Checkout code
        uses<EMAIL_ADDRESS>        with:
          ref: ${{ github.ref }}
          fetch-depth: 0

      - name: Deploy to SFTP
        uses<EMAIL_ADDRESS>        with:
          remote-protocol: sftp
          remote-host: ${{ secrets.SFTP_HOST }}
          remote-port: 2022
          remote-user: ${{ secrets.SFTP_DEV_USER }}
          remote-password: ${{ secrets.SFTP_PASSWORD }}
          local-path: ./
          remote-path: /
          sync: delta
          debug: true

Please see this pending issue with lftp https://github.com/lavv17/lftp/issues/69


Error occured when using train_imagenet.py

Hi, I met a problem when training my own image set. I have already generated the .rec format of images as follows:
sudo python im2rec.py /media/zhaoxiaoming/3C3AAF8E3AAF43AC/Work/Image_Info/test_dataset  /media/zhaoxiaoming/3C3AAF8E3AAF43AC/Work/Image_Info/ --resize=28
I want to use the /mxnet/example/image-classification/train_imagenet.py to have a try. I typed following command:
python ~/mxnet/example/image-classification/train_imagenet.py
--network inception-bn-28-small
--data-dir /media/zhaoxiaoming/3C3AAF8E3AAF43AC/Work/Image_Info/
--num-examples 331000
--num-classes 5805
--train-dataset train_dataset.rec
--val-dataset test_dataset.rec
--data-shape 28
--gpus 0

then I met the error:
2016-05-16 14:37:45,725 Node[0] start with arguments Namespace(batch_size=100, clip_gradient=5.0, data_dir='/media/zhaoxiaoming/3C3AAF8E3AAF43AC/Work/Image_Info/', data_shape=28, gpus='0', kv_store='local', load_epoch=None, log_dir='/tmp/', log_file=None, lr=0.01, lr_factor=1, lr_factor_epoch=1, model_prefix=None, network='inception-bn-28-small', num_classes=5805, num_epochs=20, num_examples=331000, save_model_prefix=None, train_dataset='train_dataset.rec', val_dataset='test_dataset.rec')
[14:37:45] src/io/iter_image_recordio.cc:211: ImageRecordIOParser: /media/zhaoxiaoming/3C3AAF8E3AAF43AC/Work/Image_Info/train_dataset.rec, use 1 threads for decoding..
[14:37:47] src/io/iter_image_recordio.cc:211: ImageRecordIOParser: /media/zhaoxiaoming/3C3AAF8E3AAF43AC/Work/Image_Info/test_dataset.rec, use 1 threads for decoding..
2016-05-16 14:37:48,421 Node[0] Start training with [gpu(0)]
[14:37:50] /home/zhaoxiaoming/mxnet/dmlc-core/include/dmlc/./logging.h:235: [14:37:50] src/storage/./gpu_device_storage.h:39: Check failed: e == cudaSuccess || e == cudaErrorCudartUnloading CUDA: out of memory
[14:37:50] /home/zhaoxiaoming/mxnet/dmlc-core/include/dmlc/logging.h:235: [14:37:50] src/engine/./threaded_engine.h:306: [14:37:50] src/storage/./gpu_device_storage.h:39: Check failed: e == cudaSuccess || e == cudaErrorCudartUnloading CUDA: out of memory
An fatal error occurred in asynchronous engine operation. If you do not know what caused this error, you can try set environment variable MXNET_ENGINE_TYPEto NaiveEngine and run with debugger (i.e. gdb). This will force all operations to be synchronous and backtrace will give you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to empty after debugging.
terminate called after throwing an instance of 'dmlc::Error'
  what():  [14:37:50] src/engine/./threaded_engine.h:306: [14:37:50] src/storage/./gpu_device_storage.h:39: Check failed: e == cudaSuccess || e == cudaErrorCudartUnloading CUDA: out of memory
An fatal error occurred in asynchronous engine operation. If you do not know what caused this error, you can try set environment variable MXNET_ENGINE_TYPEto NaiveEngine and run with debugger (i.e. gdb). This will force all operations to be synchronous and backtrace will give you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to empty after debugging.

Could anyone point out the solution to it? I would really appreciate it. Thanks a lot!

I found that I set the batch parameter in the train_imagenet.py  to 100, which is too large for my computer. After I set it to 32, it succeeds.
So this issue could be closed.


fill doesn't work

fill function doesnt work and leads to an error:
Uncaught TypeError: drawFunctions[shape] is not a function

function in canvas-utils.js

Fixed