Steam Workshop not installing/configuring into game properly (Space Engineer)

Bug Report
System Information

Ubuntu Server 22.04.4 LTS
AMP Release "Callisto" v<IP_ADDRESS> built 03/04/2024 18:20
Mainline

I confirm:

[x] that I have searched for an existing bug report for this issue.
[x] that I am using the latest available version of AMP.
[x] that my operating system is up-to-date.

Symptoms

What are you trying to do?
I and my friend are trying to install a mod from Steam Workshop using ID in SteamCMD and Updates > Steam workshop items into a game called Space Engineer.

What are you expecting to happen?
What Steam workshop items were supposed to do is have them actually install into the world and automatically configure the Sandbox_config.sbc file without having to manually configure them. (At least that's what I assume)

What is actually happening? ('Nothing' is not an acceptable answer!)
It actually does nothing, we spent a whole day figuring out until I found out we have to manually input every mod IDs into Sandbox_config.sbc file which is it makes that feature seemingly pointless to have if it does nothing at all.

Reproduction

Enter reproductions steps here.

Install AMP in Ubuntu server using bash command as shown on the AMP website
Set up AMP configurations and allow them to install rest of it for you to get AMP up and running
Log in your account in AMP
Create a new Space Engineer instance
Start server to make a new world (Can leave default configurations alone)
Stop server
Add mod IDs from steam workshop (any mod) to Steam workshop items in SteamCMD and Updates
Start server
Verify if these mods are actually installed or not (Check console, they usually tell you that they are downloading specified mods but in this situation, they doesn't tell you anything and instead simply start a new world without mods. Also verify Sandbox_config.sbc in servers > Saves > (world name) > Sandbox_config.sbc)

I believe this also don't work for many of other supported games that are integrated with Steam workshop (I tried this with gmod server and it doesn't work either. Can easily reproduce steps using same steps above except without Sandbox_config.sbc). If this is wrong repo and should be reported in AMPTemplates, please let me know because I wasn't sure which one is right to report to since this is more related to the AMP's panel. Thank you!

This won't work for Space Engineers. You need to manually add the mods to the Sandbox_config.sbc file for SE to download them directly.


Set more SSL properties

@bjorncs

ssl.authProvider : (Java system property: zookeeper.ssl.authProvider) Specifies a subclass of org.apache.zookeeper.auth.X509AuthenticationProvider to use for secure client authentication. This is useful in certificate key infrastructures that do not use JKS. It may be necessary to extend javax.net.ssl.X509KeyManager and javax.net.ssl.X509TrustManager to get the desired behavior from the SSL stack. To configure the ZooKeeper server to use the custom provider for authentication, choose a scheme name for the custom AuthenticationProvider and set the property zookeeper.authProvider.[scheme] to the fully-qualified class name of the custom implementation. This will load the provider into the ProviderRegistry. Then set this property zookeeper.ssl.authProvider=[scheme] and that provider will be used for secure authentication.


Add reflectometry related code from ess

I suppose this is all just copied & minimally adapted to make imports for tests work?

Yes exactly. This is more or less a direct copy from ess.reflectometry.


Fast version deangelis for test

Description
Adding recipe/testing/recipe_deangelis15nat_fig1_fast.yml,  short test recipe to reproduce figure 1 from recipe_deangelis15nat.yml using only one model. It should run around 1 minute on one CPU (Levante). The original recipe failed several times for new releases due to its derrived variables. This short version did run for ESMValCore v2.5 and the current release candidate v2.6.0rc3.

Contributes to close #1636
Link to documentation: https://docs.esmvaltool.org/en/latest/recipes/recipe_deangelis15nat.html
This is the documentation page to the original recipe recipe_deangelis15nat.yml, do the test recipes need an own docu?

Before you get started

[x] ☝ Create an issue to discuss what you are going to do

Checklist
It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

[x] 🛠 This pull request has a descriptive title
[x] 🛠 Code is written according to the code quality guidelines
[ ] 🛠 Documentation is available
[x] 🛠 Tests run successfully
[ ] 🛠 The list of authors is up to date
[ ] 🛠 Any changed dependencies have been added or removed correctly
[ ] 🛠 All checks below this pull request were successful

New or updated recipe/diagnostic

[x] 🧪 Recipe runs successfully
[ ] 🧪 Recipe is well documented
[x] 🧪 Figure(s) and data look as expected from literature
[ ] 🛠 Provenance information has been added

New or updated data reformatting script

[ ] 🛠 Documentation is available
[ ] 🛠 The dataset has been added to the CMOR check recipe
[ ] 🧪 Numbers and units of the data look physically meaningful

To help with the number of pull requests:

🙏 We kindly ask you to review two other open pull requests in this repository

@esmvalbot Please run recipe/testing/recipe_deangelis15nat_fig1_fast.yml

@esmvalbot Please run testing/recipe_deangelis15nat_fig1_fast.yml

Thanks! I noticed that there was no provenance for the figure, so I added it for the diagnostic used in this recipe (it looks like it's also missing for the other diagnostic scripts in the same folder though, maybe this would be something to look into).

This is the documentation page to the original recipe recipe_deangelis15nat.yml, do the test recipes need an own docu?

Yes, I think so. No need for scientific documentation, but it would be good to list the testing recipes and their options (or link to the original recipe page) and for each recipe why it is considered a good test. Could you add a documentation page for the testing recipes? A single documentation page for all recipes in the testing directory would probably be most convenient.

I added a docu page, where also the other test recipes could be added: doc/sphinx/source/recipes/recipes_testing.rst
and put it under "Others", is it ok like this? (Does the link to the docu for recipe_deangelis15nat.yml version work?)

because even such a small thing like shortening an existing diagnistic for testing purposes creates a lot of extra work

What extra work are you are talking about? 3 lines of code needed to be added to fix the provenance for this recipe and I did that for you, so where is the extra work for you?

This can easily result in 50-100 plots, if there are 6 additional provenance files that are really too many (and from the code I'd assume the provenance files in work and plot are identical?).

In the case of this diagnostic yes, but not all diagnostics create a plot and NetCDF file with the same data. Some only create plots or even other file formats. In any case, the feature that creates the 2 extra citation info files was added over 2 years ago in ESMValGroup/ESMValCore#402, so it is a bit late to complain about the number of output files now, but you're welcome to open an issue if you have ideas about how this could be improved. I do agree that it are many files and it would look a bit messy if you try to open this with a file browser. If you view the results by opening the index.html file in your browser the presentation is nicer.

Does the link to the docu for recipe_deangelis15nat.yml version work?

I fixed it for you, you needed to add backticks around the link name. You can preview the documentation by clicking on the docs/readthedocs.org:esmvaltool check below.

Thanks! This gives me a good start to implement some automated recipe testing.

because even such a small thing like shortening an existing diagnistic for testing purposes creates a lot of extra work

What extra work are you are talking about? 3 lines of code needed to be added to fix the provenance for this recipe and I did that for you, so where is the extra work for you?

This can easily result in 50-100 plots, if there are 6 additional provenance files that are really too many (and from the code I'd assume the provenance files in work and plot are identical?).

In the case of this diagnostic yes, but not all diagnostics create a plot and NetCDF file with the same data. Some only create plots or even other file formats. In any case, the feature that creates the 2 extra citation info files was added over 2 years ago in ESMValGroup/ESMValCore#402, so it is a bit late to complain about the number of output files now, but you're welcome to open an issue if you have ideas about how this could be improved. I do agree that it are many files and it would look a bit messy if you try to open this with a file browser. If you view the results by opening the index.html file in your browser the presentation is nicer.

Does the link to the docu for recipe_deangelis15nat.yml version work?

I fixed it for you, you needed to add backticks around the link name. You can preview the documentation by clicking on the docs/readthedocs.org:esmvaltool check below.

Thanks for fixing the link and merging!
With extra work I meat first understanding the issue, but also, that I now should ideally start new PRs for my more then 2 year old recipes to fix that  - leading back to the endless discussion about who has to adapt existing recipes to new technical requirements.

Adding provenance is not a new technical requirement, it has been around since January 2019, long before there was a stable release and the promise of backward compatibility. Anyway, let's not argue on GitHub, it would probably be better to discuss this in person at the next workshop.

Adding provenance is not a new technical requirement, it has been around since January 2019, long before there was a stable release and the promise of backward compatibility. Anyway, let's not argue on GitHub, it would probably be better to discuss this in person at the next workshop.

The provenace was there, only not in the correct folder (although it is not really clear, which is the correct folder follwing the doku text). I really see the point to require provanance in a standardtised way, but from the doku/ PR you linked I don't really understand, that the standard should be 6 provenance files in two different directories for every plot...

Yes, maybe we could look at the documentation together at the next workshop and make it more clear?

In principle yes, but I'm afraid any change would lead either to the request to change something in many diagnostics or there will continue to be a lot of different solutions depending on who wrote a diagnostic when.


Why wiredep exclude bootstrap by default??

Is there a reason for which wiredep exclude bootstrap by default?
I have dowloaded hello app with gulp, javascript and sass. And by defaut in conf/gulp.conf.js bootstrap is excluded for wiredep, why is that?
exports.wiredep = {   exclude: [/\/bootstrap\.js$/, /\/bootstrap-sass\/.*\.js/, /\/bootstrap\.css/],   directory: 'bower_components' };
Here is the content of .yo-rc.json :
{   "generator-fountain-angular1": {     "version": "1.0.0",     "props": {       "skipInstall": true,       "framework": "angular1",       "modules": "inject",       "js": "js",       "ci": "",       "css": "scss",       "resolved": "/Users/swiip/Workspace/FountainJS/fountain/generator-fountain-angular1/generators/app/index.js",       "namespace": "fountain-angular1:app",       "_": [],       "sample": "hello",       "router": "uirouter"     }   } }

https://github.com/FountainJS/generator-fountain-webapp/issues/126


Implements the functionality to customize the logging based on config file

Summary of Changes
Forgot to enable all the existing logger for the fileConfig.
Tests
NA
Documentation
NA
CheckList
Make sure you have checked all steps below to ensure a timely review.

[ ] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[ ] PR includes a summary of changes.
[ ] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

[ ] PR passes make test

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@8273333). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master      #95   +/-   ##
=========================================
  Coverage          ?   76.73%
=========================================
  Files             ?       20
  Lines             ?      967
  Branches          ?       91
=========================================
  Hits              ?      742
  Misses            ?      199
  Partials          ?       26

Impacted Files
Coverage Δ

metadata_service/oidc_config.py
0% <0%> (ø)

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 8273333...6bca373. Read the comment docs.


Add branch job to github action

👷 Deploy Preview for pedigreesync processing.

Name
Link

🔨 Latest commit
e634add46881549433bcce83adc5d6a75aa5d3f9

🔍 Latest deploy log
https://app.netlify.com/sites/pedigreesync/deploys/6773f6e543469300084cf2e2


CSS bundling fails without entry.client.tsx and entry.server.tsx in a monorepo

What version of Remix are you using?
1.14.0
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce

Have a monorepo that installs packages outside of the sub-package root (ex. using Yarn Workspaces)
Use create-remix to create a new project in the monorepo
Enable unstable_vanillaExtract or unstable_cssModules
Delete app/entry.client.tsx and app/entry.server.tsx
Add the following to root.tsx:

import { cssBundleHref } from "@remix-run/css-bundle";
import type { LinksFunction } from "@remix-run/node";

export const links: LinksFunction = () => [
  ...(cssBundleHref ? [{ rel: "stylesheet", href: cssBundleHref }] : []),
];

remix build

Expected Behavior
The project should build correctly, as it does when entry.client.tsx and entry.server.tsx are present (this is also the appropriate workaround)
Actual Behavior
The following error is produced:
error: Build failed with 1 error:
css-bundle-entry-module:__remix_cssBundleEntryModule__:1:29: ERROR: Could not resolve<EMAIL_ADDRESS>    at failureErrorWithLog (/workspaces/remix/node_modules/@remix-run/dev/node_modules/esbuild/lib/main.js:1591:15)
    at /workspaces/remix/node_modules/@remix-run/dev/node_modules/esbuild/lib/main.js:1047:28
    at runOnEndCallbacks (/workspaces/remix/node_modules/@remix-run/dev/node_modules/esbuild/lib/main.js:1463:61)
    at buildResponseToResult (/workspaces/remix/node_modules/@remix-run/dev/node_modules/esbuild/lib/main.js:1045:7)
    at /workspaces/remix/node_modules/@remix-run/dev/node_modules/esbuild/lib/main.js:1157:14
    at responseCallbacks.<computed> (/workspaces/remix/node_modules/@remix-run/dev/node_modules/esbuild/lib/main.js:694:9)    at handleIncomingPacket (/workspaces/remix/node_modules/@remix-run/dev/node_modules/esbuild/lib/main.js:749:9)
    at Socket.readFromStdout (/workspaces/remix/node_modules/@remix-run/dev/node_modules/esbuild/lib/main.js:670:7)
    at Socket.emit (node:events:513:28)
    at addChunk (node:internal/streams/readable:324:12)    at readableAddChunk (node:internal/streams/readable:297:9)
    at Readable.push (node:internal/streams/readable:234:10)
    at Pipe.onStreamRead (node:internal/stream_base_commons:190:23) {
  errors: [
    {
      detail: undefined,
      id: '',
      location: {
        column: 29,
        file: 'css-bundle-entry-module:__remix_cssBundleEntryModule__',
        length: 79,
        line: 1,
        lineText: 'export * as entryClient from<EMAIL_ADDRESS>        namespace: '',
        suggestion: ''
      },
      notes: [],
      pluginName: '',
      text: 'Could not resolve<EMAIL_ADDRESS>    }
  ],
  warnings: []
}

I tested with exactly the same package outside of a monorepo, and the error did not occur. So my best guess is that in a monorepo the relative import is failing due to a scoping issue of some kind.

Just confirming that at least in my instance, your PR resolves the issue. I applied this patch and it works:
diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index 61b318c..0ba16c6 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -47,7 +47,7 @@ function cssBundleEntryModulePlugin(config) {
           loader: "js",
           contents: [
           // These need to be exports to avoid tree shaking
-          `export * as entryClient from ${JSON.stringify(path__default["default"].relative(config.rootDirectory, config.entryClientFilePath))};`, ...Object.keys(config.routes).map((key, index) => {
+          `export * as entryClient from ${JSON.stringify(path__default["default"].resolve(config.rootDirectory, config.entryClientFilePath))};`, ...Object.keys(config.routes).map((key, index) => {
             let route = config.routes[key];
             return `export * as route${index} from ${JSON.stringify(`./${route.file}`)};`;
           })].join("\n")

I can confirm that this issue is resolved for my case in v0.0.0-nightly-bf9d0a9-20230314. @huw does it work for you now?

Solved for me too. Awesome.

Fixed in 1.15.0: https://github.com/remix-run/remix/blob/remix%401.15.0/packages/remix-dev/CHANGELOG.md#patch-changes


feat: poc

This PR implements MessagePack specification and back-referencing stuff.
Todo

[x] Implement back-referencing.
[x] Implement Timestamp (Date) extension.
[x] Implement Map/Set extensions.
[x] ~Replace TextEncoder/TextDecoder with something better.~ I'll leave it for now.

Also needed, to demonstate viability:

[x] Size benchmarks. Compare JSON, MessagePack, and BackPack.
[x] Perf benchmarks. Compare JSON.stringify/JSON.parse, MessagePack, and BackPack.

Bugs:

[x] Timestamp extension doesn't work with dates before 1970.

References
Hashing?
Using hashes for references apparently is a bad idea. As it turns out, there's only a handful of hashing algorithms that can produce 1-2 bytes long hashes (not talking here about reducing, for instance, 32 bit long hash ids to 8 bits via masking, i.e. hash & 0xFF). The smaller hashes, the bigger probability of collisions.
Even worse, hashing is always susceptible to collisions, and that, I think, automatically renders the whole serialization/deserialization non-deterministic. In hashmaps it leads to suboptimal lookups/inserts, but here it may lead to corrupted data.
Other options?
So I went with a simpler (and performant!) approach. Turns out, we don't need to hash strings at all, but we can use monotonically increasing ids derived from hashmap's size.
Notes
It's dead simple and works well, but there's a maximum number of refs that can be safely stored: 2^16 = 65536. There's no point in allowing to store 32 bit long refs, since the overhead makes the whole approach non-viable for most cases.
NB. Right now all strings that are 4-16 bytes long get turned into references. If I find a way to turn into refs only duplicated strings, then gains will be even more noticeable.

Serialization is way too slow and asks for rewriting.

Benchmarks
Performance
This is after replacing TextEncoder and TextDecoder with custom encoder and decoder and adding cache for strings and property names. Serializing is 5-6 times faster.
Running "Serialize" suite...
Progress: 100%

  BackPack:                       311 ops/s, ±0.66%     | slowest, 79.32% slower
  MessagePack (@msgpack/msgpack): 582 ops/s, ±2.53%     | 61.3% slower
  MessagePack (msgpackr):         984 ops/s, ±1.19%     | 34.57% slower
  JSON.stringify:                 1 504 ops/s, ±0.87%   | fastest

Running "Deserialize" suite...
Progress: 100%

  BackPack:                       163 ops/s, ±0.41%     | slowest, 86.38% slower
  MessagePack (@msgpack/msgpack): 493 ops/s, ±0.59%     | 58.81% slower
  MessagePack (msgpackr):         1 017 ops/s, ±0.88%   | 15.04% slower
  JSON.parse:                     1 197 ops/s, ±0.88%   | fastest

Performance improvements (Deserialize)

Removed ASCII "detection" to make it 2-2.5x faster.

Running "Deserialize" suite...
Progress: 100%

  BackPack:                       377 ops/s, ±0.78%     | slowest, 67.39% slower
  MessagePack (@msgpack/msgpack): 473 ops/s, ±0.52%     | 59.08% slower
  MessagePack (msgpackr):         991 ops/s, ±0.68%     | 14.27% slower
  JSON.parse:                     1 156 ops/s, ±0.93%   | fastest


More fixes for the namespaces of indices

Existing Screen functions were mistakenly indexed as Screen.Function, so index names were not valid.

Cross referencing the autocomplete file with the index revealed a few places where the script functions needed to be forced as global
Also adds override ability for where the page structures do not directly relate to scripting objects
Fixes remaining indices for 'Multimedia' functions

namespace_map maps the hierarchy of the manual pages to the one of the AGS objects available through the AGS Script API. The hierarchy of itself being the same as the manual pages, and each element is mapped to globales, unless it's a table that contains a value, and then it get's reallocated as a child of that element instead. The purpose of this is so that the genindex is mapped correctly so that searching the index in the help makes sense, the f1 works in the Editor?
I believe it's something that will be heavily edited on each new version of AGS, as the API get's changed.
I also had a hard time figuring out understanding how the function gets assigned to be global instead of under other place (example: Game), so some comments at these specific places in write_metablock.lua are welcome.  These are general comments on this file.

namespace_map is only for exceptions, for where the with the existence of the page doesn't match a script object (e.g. Multimedia) or where a global function is listed along with object functions (e.g. IntToFloat). So really the size of namespace_map is a measure of how the page structure doesn't match the script API. This could be down to inconsistencies in the Script API itself or down to poor choice in page structure, but if this list of exceptions ever grows that is a sign that something is going wrong elsewhere - it should only ever get smaller.


Artifact reorganisation creates bad path during synthesis

After updating Openlane I hit the following error during synthesis:
ERROR: Can't create file /project/openlane/user_project_wrapper/runs/user_project_wrapper/reports/synthesis/1-synthesis.log/2-synthesis_pre.stat.

synth_report_prefix and synthesis_reports are indeed bad:
set ::env(synth_report_prefix) {/project/openlane/user_project_wrapper/runs/user_project_wrapper/reports/synthesis/1-synthesis.log/2-synthesis}
set ::env(synthesis_reports) {/project/openlane/user_project_wrapper/runs/user_project_wrapper/reports/synthesis/1-synthesis.log/2-synthesis.log}

It does look like an issue if we call run_yosys twice:
        set ::env(synth_report_prefix) [index_file $::env(synthesis_reports)/synthesis]
        set ::env(synthesis_reports) [index_file $::env(synthesis_reports)/synthesis.log]

So er, wait, I'm confused. Does synthesizing twice cause this issue?

I presume it's because my design has this:
set ::env(SYNTH_USE_PG_PINS_DEFINES) "USE_POWER_PINS"

Which makes us call run_yosys a second time:
        if { [info exists ::env(SYNTH_USE_PG_PINS_DEFINES)] } {
                puts_info "Creating a netlist with power/ground pins."
                if { ! [info exists ::env(SYNTH_DEFINES)] } {
                        set ::env(SYNTH_DEFINES) [list]
                }
                lappend ::env(SYNTH_DEFINES) {*}$::env(SYNTH_USE_PG_PINS_DEFINES)
                run_yosys -output $::env(synthesis_tmpfiles).pg_pins.v -no_set_netlist
        }

Gotcha. I'll take a look.

Okay, sorry- oversight on my part. synthesis_reports should never be re-set. PR created, and it should close this issue once merged.
I'm terribly sorry for the inconvenience.


关于playground/create_prompt.py文件[Question]

Question
您好，请问在eval.sh命令中出现的playground/create_prompt.py文件在哪里可以找到呢？

Sorry, some files are ignored by the gitignore, I have uploaded these files and you can try running the scripts to create the prompt to evaluate the general knowledge.

Thank you for your reply. Where can I download TextVQA_0.5.1_val.json in 2_eval_textqa.sh?

You can download it from LLaVA or  TextVQA_0.5.1_val.json and image.


Sequenced Acquisition works in MDA gui but not with pycromanager acquisition engine?

Bug report

Bug summary

@ieivanov and I typically run experiments that use hardware triggering of a z-stage from camera pulses.  We do this by running a "sequenced acquisition" where Micromanager will send a sequence of z-positions to the stage, and it will step through that sequence when it receives a pulse from the camera.  We use an ASI piezo stage and we prepare this acquisiton by setting the device property ZStage-Use Sequence: 'Yes'.  I am not sure what happens behind the scenes on Micromanagers acquisiton engine, but after this property is set to "Yes", any z-stack acquisition will now be acquired through hardware sequencing.
Pycromanager seems to have a bug when I try this same setup, but run an acquisition with the code below.  I receive a java.lang error immediately upon trying to run the acquisition. See Below
Code for reproduction

from pycromanager import Bridge
from pycromanager import Acquisition, multi_d_acquisition_events

bridge = Bridge()
mmc = bridge.get_core()
mms = bridge.get_studio()

save_dir = 'Q:\\Cameron\\test'

# Z-Stack Settings

z_start =  -8#um
z_end = 8 #um
z_step = 0.25 #um

name='test'

pos_list_single = np.expand_dims(np.asarray([mmc.get_xy_stage_position().x, mmc.get_xy_stage_position().y, mmc.get_position('ZStage')]),0)

if __name__ == '__main__':
    with Acquisition(directory=save_dir, name=name, show_display=True) as acq:
        events = multi_d_acquisition_events(order = 'pz',
                                            z_start = z_start,
                                            z_end = z_end,
                                            z_step = z_step,
                                           xyz_positions = pos_list_single,
                                            )
        acq.acquire(events, keep_shutter_open=True)

Expected outcome

I expect this to run a triggered z stack acquisition at the current position
Actual outcome

An acquisition display window will pop up, and then the error below immediately follows in a pop-up window:
java.lang.RuntimeException: java.lang.RunTimeException: java.util.concurrent.ExecutionException: org.micromanager.acqj.internal.acqengj.HardwareControlException
Note
This was not an issue before we updated the firmware for the stage controller.  This started happening immediately after the update.  I am in conversation with ASI about the firmware and can relay any questions to them.  I want to blame the firmware, but I was surprised to see that it still works fine with the Micromanager MDA acquisition engine, so I am wondering if you have any ideas here.
Version Info

Operating system: Windows 10
pycromanager version: 0.10.13
MicroManager version: 4/28/2021
Python version: 3.7
Python environment (command line, IDE, Jupyter notebook, etc): jupyter notebook

It looks like you're doubly supply the z positions, both directly (z_start, z_end, ...) and with (xyz_positions). If you print the events generated by multi_d_acq... do they look right?

If thats not the problem, you could try calling the relevant code that the acquistion engine is calling directly, and see if you can reproduce. If so, that says error in device layer and not pycromanager
This is Java, but striaghtforward to translate and call through pycromanager:
DoubleVector zSequence = new DoubleVector() ;

// Loop and add all z positions
zSequence.add(zPos);
core_.loadStageSequence(zStage, zSequence);
core_.prepareSequenceAcquisition(core_.getCameraDevice());
core_.startStageSequence(zStage);

while (core_.isSequenceRunning()) {
                 //wait
}
core_.stopSequenceAcquisition();

//N is number of images
core_.startSequenceAcquisition(N, 0, true);

//loop and call N times to throw away the images
core_.popNextTaggedImage();

Thanks for troubleshooting with us Henry!
Cam, I do have the java env set up, if that would be useful. Let me know and I can point you to it.
Cam, can you also confirm that z sequencing used to run with this pycromanager code before the firmware upgrade? It's quite strange that we're running into this problem.
Henry, the firmware upgrade was to increase the available sequence buffer length from 50 positions to 250 positions. Do you see why that may be a problem? Here we're using an ASI MS-2000 controller (https://micro-manager.org/wiki/ASIStage). I know that 250 positions work with their Tiger controller (https://micro-manager.org/wiki/ASITiger) without a problem. Could it possibly be a device adapter issue?

Yeah that would make sense to me, except its weird that it works with MM MDA. So its possible theres a bug in AcqEngJ in dispatching the sequence to the hardware.
You could also try adding a pre_hardware_hook, and print the event, which at that point will be a sequence of multiple events, to see if it looks right


Scale size of fish marker in fishing spot with zoom level

When zoomed all the way out, I literally can't see my character. I wouldn't mind these monkfish to be a little smaller if I'm zoomed out all the way.

Personally i wouldn't like them small. If so, make it a enable/disable feature.


In IOS hmsTrackSetting back camera not working. what is the problem?

hey @vorachirag318 Do you want to set the back camera while joining the room?
We'll be releasing support for this soon.
Meanwhile, you can call switchCamera() in onJoin event to show the back camera.

hey @vorachirag318 We have released new version 1.1.0 which fixes this issue. Try it out & let us know.

hey @vorachirag318
Were you able to try out the new package version 1.1.0 ?

hey @vorachirag318
Hope you were able to resolve this issue. Ping us on Discord for any other assistance: https://100ms.live/discord


Handling annotated_types.Predicate fails with multiple conditions

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
Hi,
Initially logged an issue on annotated-types, but a contributer redirected me to here instead.
To reproduce, run the code in this https://github.com/annotated-types/annotated-types/issues/81#issuecomment-2356746225.
It seems that pydantic is unable to handle annotated_types.Predicate correctly when using it in multiple conditions.
Next to pydantic, I'm using annotated-types version 0.7.0
Example Code
from annotated_types import Ge, Predicate, Len
from pydantic import BaseModel
from typing import Annotated

class PriceBroken(BaseModel):
    value: Annotated[float, Ge(0)]
    currency: Annotated[str, Len(3, 3), Predicate(str.isupper)]

class PriceAlsoBroken(BaseModel):
    value: Annotated[float, Ge(0)]
    currency: Annotated[str, Predicate(str.isupper), Len(3, 3)]

pb = PriceBroken(value=1.0, currency='USD')
print(pb.currency == 'USD')
pab = PriceAlsoBroken(value=1.0, currency='USD')

Python, Pydantic & OS Version
pydantic version: 2.9.0
        pydantic-core version: 2.23.2
          pydantic-core build: profile=release pgo=false
                 install path: /workspace/.venv/lib/python3.12/site-packages/pydantic
               python version: 3.12.5 (main, Sep  5 2024, 00:16:34) [GCC 12.2.0]
                     platform: Linux-6.8.0-41-generic-x86_64-with-glibc2.36
             related packages: mypy-1.11.2 typing_extensions-4.12.2
                       commit: unknown

This is fixed in v2.9.2, I believe 👍


Add classname and styles to Voronoi cells

Closes #684
This change makes it possible to conditionally add classnames and styles to Voronoi cell, depending on the data points.
This PR also adds initial docs for the Voronoi diagram.

@mcnuttandrew I've updated the code and added a test for the Voronoi diagram.
Starting from scratch I found it a little hard to add docs (not my strong suit). But I added a base, that will hopefully encourage others to improve upon it. I also wasn't clear on how to test the docs. npm start only gives me the show-case items. So there might be some glitches that I didn't catch.
Let me know what you think.

@mcnuttandrew Good to hear :) Commited the last fixes.

@mcnuttandrew Any idea when this will be published on npm? Thanks!

@balthazar ?

1.8.1


queue

Do you know what is Queue, and what is Zset？

I've re-titled your issue. Please do not use profanity in issues.
Can you restate your question with more information? What are you trying to accomplish? What have you tried so far? Where are you stuck?

why It requires shaperate connection to push and pop otherwise is not shcedulling?


Add pod name to the list of things we log.

With chaos duck (and other HA tests) we kill things and we now will be able to know
which exact instance logged a message to aid with debugging.
/assign mattmoor

/approve


There is no backward compatibility of V2

The new release is not backward compatible with V1 and does not raise any warning when compiling with the new version. Functions compile without warning but simply do not work anymore.
User of my sketches complain Bluetooth is not working after an new upload because they automatically upgraded their library versions.
I can of course fork V1.4.3 and rename the library but then I have to change all my sketches I made the last years.  That will cause a lot of unexpected work.
Please rename the V2 NimBLE library to NimBLE2 or similar and replace 1.4.3 in the NimBLE library as Version 3.0 so the original NimBLE library automatically use V1 again.

This is the reason for release versions, renaming the library is not necessary or warranted, Just use the release that works for you.

Are you using Platformio for dependency management? Might be worth locking to major versions for libraries that implement semantic versioning as most do these days.
A lot of libraries implement backwards incompatible changes at major version increments. The uplift for NimBLE-Arduino 2.x was nothing compared to ArduinoJSON 7.x

A main issue is that V1 functions compile without a warning with version V2 but do not function anymore.
I recognize the problem now but many users of the sketch will not. With Arduino IDE2.0, that many people use, there is no way to stay with NimBLE V1.
But thanks for the work that is done.

@ednieuw That was surprising to hear so I had to have a look for myself. It is indeed possible to change the version of the library used in Arduino IDE 2.0:

@h2zero Unfortunately you can not lock a library version to a sketch in the Arduino IDE.
An upgrade to NimBLE V2 is possible but many of my sketches, also used by other people, are using the NimBLE library.
These old sketches will still compile without warning and with Bluetooth not functioning.
Maybe you can include a warning during compilation.
I tried to copy to 1.4.3 library to my sketch folder and "#include "NimBLE-Arduino\src\NimBLEDevice.h"  but then many header files can not be found.
C:\Users\ednie\Documents\Files\Arduino\Arduino_Nano-ESP32-WordClock\ESP32Arduino_WordClockV060\NimBLE-Arduino\src\nimble\nimble\include\nimble\ble.h:25:10: fatal error: nimble/porting/nimble/include/syscfg/syscfg.h: No such file or directory

Unfortunately you can not lock a library version to a sketch in the Arduino IDE.

That is unfortunate indeed, but you can switch back and forth depending on the project you're working on.

An upgrade to NimBLE V2 is possible but many of my sketches, also used by other people, are using the NimBLE library.

The changes required to use v2.x should not be too burdensome.

These old sketches will still compile without warning and with Bluetooth not functioning.
Maybe you can include a warning during compilation.

Not sure how I would go about including a warning, and I'm sure most users would not appreciate a compiler warning every time they build their projects.

I tried to copy to 1.4.3 library to my sketch folder and "#include "NimBLE-Arduino\src\NimBLEDevice.h" but then many header files can not be found.

You would have to copy just the src folder to your sketch folder for this to possibly build due to the include paths.

Not sure how I would go about including a warning, and I'm sure most users would not appreciate a compiler warning every time they build their projects.

The compiler warning can just be a comment to upgrade. Then there is some kind of warning.
This morning I already had a complaint from one of my sketch users bluetooth was not working. Most users are not able to dig into these large c++ libraries. Many users hardly know that they have upgraded a library in the Arduino IDE.
The IDE complains at every start there are boards and libraries to upgrade and it can do it for all libraries with one click.
Is it possible to overload the depreciated functions?
Or
Add a #define NIMBLE_VERSION 2
Then it is possible to discriminate between the two versions in the sketch.
I now apply the same approach to the Arduino Nano ESP32 version 2.0.18, supported by Arduino, and the Espressif core 3.0.7, which support new features.
#if ESP_ARDUINO_VERSION >= ESP_ARDUINO_VERSION_VAL(3, 0, 0)
#include <EdSoftLED.h>
                      #else
#include <Adafruit_NeoPixel.h>
                      #endif

I tried to copy to 1.4.3 library to my sketch folder and "#include "NimBLE-Arduino\src\NimBLEDevice.h" but then many header files can not be found.

You would have to copy just the src folder to your sketch folder for this to possibly build due to the include paths.

Ah, that works. You have to place the contents of the src folder beside the ino sketch. That is too messy for me; too many tabs in the IDE and files in my sketch folder.

This seems like a major oversight in the Arduino IDE rather than an issue with this library. Semantic versioning is pretty standard — it’s wild that there isn’t a way to lock to a specific version - major or minor - given the possibility for any of your dependencies to make incompatible changes even between revisions.
Have you considered switching to a better IDE such as Platformio?

Have you considered switching to a better IDE such as Platformio?

Yes, several times. But there is so much installed I loose control what's happening.
The people I write my software for are often not very experienced programmers. More builders of machines, soldering electronics and so on.  The Arduino IDE is because of its simplicity preferable to use and to help with.
That is the reason I want to solve the problems in the software and not in installations. If also try to get rid of libraries for obvious reasons.
I am a fan now of uploading a compiled binary file with OTA and control/setup the MCU with BLE or a HTML page. (For as long as securities do not kill these options)

I would recommend in this case that the users and the code be updated to use v2.
That said there are some new config options in v2 that you can use to check the existence of and know what that v2 is in use.
#ifdef CONFIG_NIMBLE_CPP_FREERTOS_TASK_BLOCK_BIT

Thanks.


added UI changes for API tokens page empty state

Signed-off-by: vinay033<EMAIL_ADDRESS>:nut_and_bolt: Description
Currently, when there are no api tokens on the API Tokens page we show an empty table, instead, we would like to follow the empty state pattern.
:athletic_shoe: Demo Script / Repro Steps
Existing UI

New changes

:chains: Related Resources
https://github.com/chef/automate/issues/303

LGTM but I'm no CSS expert

CSS is only added for showing button in the center if there is no token available.

Note that for the failing integration test you will need to mock the <app-authorized> component with something like this:
       MockComponent({ selector: 'app-authorized',
                        inputs: ['allOf', 'anyOf'],
                        template: '<ng-content></ng-content>' })
      ],

You can see that in use in components/automate-ui/src/app/page-components/navbar/navbar.component.spec.ts

Note that for the failing integration test you will need to mock the <app-authorized> component with something like this:
       MockComponent({ selector: 'app-authorized',
                        inputs: ['allOf', 'anyOf'],
                        template: '<ng-content></ng-content>' })
      ],

You can see that in use in components/automate-ui/src/app/page-components/navbar/navbar.component.spec.ts

Thanks for the suggestion, I will check and update the PR.

MockComponent

@msorens I just checked  component already mocked in components/automate-ui/src/app/pages/api-token/list/api-token-list.component.spec.ts.
in logs show error like
1) Admin pages IAM v1 API Token list control button when clicked, shows Delete Token
--
  | - Failed: element not interactable
  | (Session info: headless chrome=73.0.3683.86)
  | (Driver info: chromedriver=2.46.628388 (4a34a70827ac54148e092aafb70504c4ea7ae926),platform=Linux 4.14.97-90.72.amzn2.x86_64 x86_64)


Not working in crontab (Ubuntu 20.04)

I have bash and node.js scripts:
runner.sh:
some=parameter npm run myscript

crontab
* * * * * cd /path/to/project && ./runner.sh

When I'm running the same from shell (./runner.sh) - everything works well. But I want to recieve notifications from cron.
Can you help me?
May be something wrong with paths?
I've tried to find it in project, but I couldn't. :-(

I just use node for the cronjobs and it works, I am not sure if it's problem of the shell, but here is my crontab config:
*/10 * * * * /Users/username/.nvm/versions/node/v12.18.4/bin/node /Users/username/Projects/test/index.js


[FEAT] Performance optimizations to stay under the Railway free plan

It would be ideal to stay under the Railway $5 plan, if possible, which is a resource usage-based IaaS provider that TAU supports deployments for. @FiniteSingularity made a suggestion to try Uvicorn instead of Daphne to use less resources.
Additional context

original discussion: https://discord.com/channels/827201860074995742/827277974168535040/980600746607255622
https://docs.djangoproject.com/en/4.0/howto/deployment/asgi/uvicorn/

Here is the problem we're seeing, and I am open to suggestions in terms of how to reduce resource usage for Railway.

Railway's "free" tier is essentially $5 in credits given each month.  The current cost to run an instance of TAU is around $5.50-$6.00, just outside of the free tier.
Unfortunately, there is no way that I can see to have data persistence if we use the sqlite option for TAU (which would keep us from having to run a Postgres container.  The current CPU/Memory usage of the postgres container is about 50% of our overall Railway resource usage).
We also require a redis container as a message broker for Django channels.  The good news on this front- the redis container is a very small portion of the overall resource usage- basically a rounding error.
Daphne is known to have issues with memory leaks, and you have very little control over its memory/cpu usage (which is known to be quite high, especially for small projects like this).
Uvicorn is an ASGI server process, much like Daphne, but appears in all ways to be more slim/customizable.

As you might imagine, the average CPU usage of TAU is quite low- it sits almost idle most of the time, and processing incoming events/IRC, and passing them through to a few websocket subscribers is not very resource intensive.  The resource usage that is putting us over the $5 credit is memory.
Looking at my last month's usage, the average memory was a few mb for Redis (again, rounding error), ~240mb for Postgres, and 280mb for the server/worker container (running Daphne).  It looks like keeping memory usage to under 500mb total would keep us in the free tier.
I've seem some estimates that Uvicorn used for smaller projects (and only using 1 or 2 work threads) can save up to half the memory usage over Daphne.  I'd estimate that 50-60mb of our usage is the TAU worker process, and the remaining 200+ is Daphne.  So if we can cut that in half, we'd be well under the 500mb limit.  I can try to test this out later next week, but if anyone wants to give it a try themselves, you'll need to use pip in your container to install Uvicorn (or add it to the requirements.txt file and rebuild the container), then change the [program:server] command in the supervisord.conf file to run uvicorn (or even gunicorn with a uvicorn worker).  I am seeing conflicting issues on if the /tau/asgi.py file will need to be modifed to run under uvicorn.

I am going to close this, as I dont think it is possible at this point.  The usage costs have just gone up on Railway, and I think that even if we did get it down under the free tier, as railway changes their pricing, it would quickly drop into paid.  It still isn't expensive, but unfortunately free seems unlikely/a constant chase.


Prevent LimitOverrunError with large output lines

@ mention of reviewers
@ihsaan-ullah
A brief description of the purpose of the changes contained in this PR.
If a submission writes a output line larger than the stream buffer size ( default 64k ) a LimitOverrunError will be raise. Rather than using readline(...) use readutil(....) and in the case of a overrun just return the current buffer, the rest of the line will be returned with the next read.
A checklist for hand testing

Tested with local worker connected to codabench.org

Checklist

[x] Code review by me
[x] Hand tested by me
[x] I'm proud of my work
[ ] Code review by reviewer
[ ] Hand tested by reviewer
[ ] CircleCi tests are passing
[ ] Ready to merge

Any movement on this?

Merged in another branch to test it (#1404)


SW-3605 Endpoints to claim and release plots

Add two API endpoints to support the monitoring plot selection UI in the mobile
app's pre-monitoring flow.
POST /api/v1/tracking/observations/{observationId}/plots/{plotId}/claim claims
one of the monitoring plots that has been assigned to an observation. A monitoring
plot may only be claimed by one user at a time.
POST /api/v1/tracking/observations/{observationId}/plots/{plotId}/release
releases a previously-acquired claim on a monitoring plot, making it available for
other users to claim.

@sgrimm queued this pull request to merge with Graphite.

Graphite rebased this pull request as part of a merge.

@sgrimm merged this pull request with Graphite.


Operations with json logging fail when there is a dependency with outputs

Describe the bug
When I have two terragrunt modules which have a dependency defined (Module B uses outputs from Module A) using the dependency block.
To Reproduce
Steps to reproduce the behavior, code snippets and examples which can be used to reproduce the issue.
# File: module-a/terragrunt.hcl

# File: module-a/outputs.tf

output "test" {
  value = "hello-world"
}

# File: module-b/terragrunt.hcl

dependency "a" {
  config_path = "../module-a"
}

inputs = {
  test = dependency.a.outputs.test
}

# File: module-b/variables.tf

variable "test" {
}

Run in module-b:
terragrunt plan -json --terragrunt-json-log --terragrunt-tf-logs-to-json &> plan-log.json

Expected behavior
Plan Operations run through.
Nice to have
Log excerpt:
<version calls>

{"executedCommandArgs":["output","-json"],"level":"info","msg":"{","time":"2024-04-09T10:33:10+02:00","workingDir":"/workspace/tf-repro/module-a"}
{"executedCommandArgs":["output","-json"],"level":"info","msg":"  \"test\": {","time":"2024-04-09T10:33:10+02:00","workingDir":"/workspace/tf-repro/module-a"}
{"executedCommandArgs":["output","-json"],"level":"info","msg":"    \"sensitive\": false,","time":"2024-04-09T10:33:10+02:00","workingDir":"/workspace/tf-repro/module-a"}
{"executedCommandArgs":["output","-json"],"level":"info","msg":"    \"type\": \"string\",","time":"2024-04-09T10:33:10+02:00","workingDir":"/workspace/tf-repro/module-a"}
{"executedCommandArgs":["output","-json"],"level":"info","msg":"    \"value\": \"hello-world\"","time":"2024-04-09T10:33:10+02:00","workingDir":"/workspace/tf-repro/module-a"}
{"executedCommandArgs":["output","-json"],"level":"info","msg":"  }","time":"2024-04-09T10:33:10+02:00","workingDir":"/workspace/tf-repro/module-a"}
{"executedCommandArgs":["output","-json"],"level":"info","msg":"}","time":"2024-04-09T10:33:10+02:00","workingDir":"/workspace/tf-repro/module-a"}

<version calls>

{"level":"error","msg":"Could not parse output from terragrunt config /workspace/tf-repro/module-a/terragrunt.hcl. Underlying error: unexpected end of JSON input","time":"2024-04-09T10:33:11+02:00"}
{"level":"error","msg":"Unable to determine underlying exit code, so Terragrunt will exit with error code 1","time":"2024-04-09T10:33:11+02:00"}

Versions

Terragrunt version: 0.56.3
Terraform version: 1.5.7
Environment details: Ubuntu 22.04 LTS

Fix included in https://github.com/gruntwork-io/terragrunt/releases/tag/v0.57.2

@denis256 Thank you very much for the quick fix!


add hpd historical data

related to https://github.com/NYCPlanning/data-engineering/issues/30
goals

be able to archive new source data: HPD Historical Units by Project (hpd_historical_units_by_building)

changes

add template file and python script
change readme to have a more common example of library archive arguments

notes

emulates approach for similar source data hpd_hny_units_by_building
screenshot of archived data in edm-recipes S3 bucket:
successful use of the archived data in a downstream process

@damonmcc (and cc @fvankrieken ) This all looks fine, but  I suppose I don't fully understand the rationale for the inheritance pattern in the ./library/scripts folder. Looks like there's a ton of boilerplate... for example, why not just implement
    def runner(self) -> str:
        df = self.ingest()
        local_path = df_to_tempfile(df)
        return local_path

in a parent class, since those lines are reproduced all over. Seems like you could eliminate quite a few of these files.
And in the case of the script added here, what about having sheetname defined in the template, and have an ExcelScriptorInterface or something? Seems more traditionally OOP'y, but let me know if there's something I don't understand.
Maybe a bigger discussion. Otherwise LGTM!

06_20_2023_Lookback Legislation.xlsx also wasn't sure where this file comes from


con filename reserved in windows

Expected Behavior
git clone git@github.com:awslabs/aws-ai-qna-bot.git works on windows
Actual Behavior
error: unable to stat just-written file lambda/handler/lib/con.js: No such file or directory
fatal: unable to checkout working tree
warning: Clone succeeded, but checkout failed.
Steps to Reproduce the Problem

On a windows machine with git configured
git clone git@github.com:awslabs/aws-ai-qna-bot.git

It seems files named con are reserved and this code base contains 2 entries.
Thanks for the excelent QnA bot though!

fixed in v2

FY: I have this same problem with a file named aux.sh in a personal repo, unrelated to this project:

error: unable to stat just-written file hints/aux.sh: No such file or directory


No errors, but no rows created

This gem looks great, but when I tried to use it, no data got imported.
I tried everything I could think of - I used the before/after import procs to confirm that it was running.  Tried multiple options, turning validation off, setting permit_params.  I used the before_import proc to confirm that the file was getting uploaded and the file was readable on the server.
Not sure - maybe this just doesn't work with the latest master branch version of ActiveAdmin?

@shedd , please try branch 3.0.0, if you can. Now I'm refactoring and writing specs for this gem.


TypeError in /dues

Error in Apiary
TypeError in /dues
Cannot read property 'forEach' of undefined
View on Bugsnag
Stacktrace
https://my.robojackets.org/js/app.js:2 - o.duesPackageChoice

View full stacktrace
Created by RoboJackets Admin via Bugsnag

Happens after selecting a dues package for a totally new user

Happens after selecting a dues package for a totally new user


In ElasticSearch,Why segment index don't create with date

use elasticsearch to store data, every day trace  data is very large and save time is very long , skywalking just use one segment index to save trace data ,when data more and more , it will be a problem, so guys , how to resolve? Normally create index with date ,one day one index eg: segment_2019-06-21

Read master document and codes, this change has been done.

ok ，thinks！


Fix twig extension documentation

Describe the Pull Request
Fixes previous errors and documents function and test usages

[x] I read contribution guidelines
[ ] Pull request introduces a BC-Break
[ ] Pull request is covered by tests
[x] Pull request is properly documented in docs

Don't worry about checks here, we are removing all the monorepo features, and the code here isn't up to date.


Using the API on Odoo 11 returns ERROR

Hi,
I try to connect on a Odoo 11 using the API with the following method:
openERPSession = new Session(url, 8069, db, user, pass);
openERPSession.startSession();
The error message is: HTTP server returned unexpected status: INTERNAL SERVER ERROR
When I look at the log file on the Odoo server, I see the following error:
2017-11-21 08:38:34,357 32119 INFO ? werkzeug: <IP_ADDRESS> - - [21/Nov/2017 08:38:34] "POST /xmlrpc/2/common HTTP/1.1" 500 -
2017-11-21 08:38:34,358 32119 ERROR ? werkzeug: Error on request:
Traceback (most recent call last):
File "/usr/local/lib/python3.5/dist-packages/werkzeug/serving.py", line 209, in run_wsgi
execute(self.server.app)
File "/usr/local/lib/python3.5/dist-packages/werkzeug/serving.py", line 197, in execute
application_iter = app(environ, start_response)
File "/odoo/odoo-server/odoo/service/server.py", line 251, in app
return self.app(e, s)
File "/odoo/odoo-server/odoo/service/wsgi_server.py", line 166, in application
return application_unproxied(environ, start_response)
File "/odoo/odoo-server/odoo/service/wsgi_server.py", line 154, in application_unproxied
result = handler(environ, start_response)
File "/odoo/odoo-server/odoo/service/wsgi_server.py", line 122, in wsgi_xmlrpc
params, method = xmlrpclib.loads(data)
File "/usr/lib/python3.5/xmlrpc/client.py", line 999, in loads
p.feed(data)
File "/usr/lib/python3.5/xmlrpc/client.py", line 438, in feed
self._parser.Parse(data, 0)
File "../Modules/pyexpat.c", line 414, in StartElement
File "/usr/lib/python3.5/xmlrpc/client.py", line 674, in start
raise ResponseError("unknown tag %r" % tag)
xmlrpc.client.ResponseError: ResponseError("unknown tag 'ex:nil'",)
Can someone help me on this?
Regards,
Bernard

Hi, could you display before connexion the values for url, db, user and pass ?

Thanks flotho,
I don't know why but 2 of those parameters where null.
I deleted them all, save, recreate and 2 others where null too.
It looks like a pentaho problem, not linked to the API.
Thanks again


[WIP] Add toast notifications to the landing page

This requires a new release of origin-web-common if/when https://github.com/openshift/origin-web-common/pull/39 is merged.

LGTM.  There are going to be merge conflicts as landingPage and homePage html and controllers changed in the 'Recently Viewed' and 'Consolidate Catalog Items' PRs.

With the updates to https://github.com/openshift/origin-web-common/pull/39 it is only required to include the toast-notifications directive at the application level (could by page if desired).

This PR is no longer WIP. Ready for final reviews.


Upgrade to thespian 3.8.0

After https://github.com/kquick/Thespian/pull/7 is released with 3.8.0 (see release notes), we should upgrade and remove our temporary workaround.

Closed by 57dea5a.


Point Light and shader extension

Hi
Is there any effect or techniques that I can implement point light with in the library? I search whole repo source and couldn't find any.
And also, shader files ".fx"s are so confusing and I could find out where are entrypoints and even how an Effect class create shaders from source/compiled files. Can you guide me so I can extend existing shader files to implemen unavailable features?

The XNA Game Studio "BasicEffect" shaders did not implement point lights, although it's fairly easy to create a point-light shader.
For the basics of writing a custom effect, take a look at this tutorial.
The source for the BasicEffect shaders all came from the XNA Game Studio 4 StockShaders sample.


Command line arguments when launch a script.

From @GoogleCodeExporter on May 31, 2015 11:24
I want to run my script with specify arguments, and I can't do this in
current ASE.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 27 Nov 2009 at 6:47
Copied from original issue: damonkohler/android-scripting#142

From @GoogleCodeExporter on May 31, 2015 11:24
Currently, the way to do this is to have your script prompt for the values
you'd like
to specify. You can use getInput or the usual terminal input methods.

It would be nice to be able to specify them on the command line though for
things like
Locale.

Original comment by<EMAIL_ADDRESS>on 30 Nov 2009 at 8:57

Changed state: Accepted

From @GoogleCodeExporter on May 31, 2015 11:24
one thing you might try is using a YAML or JSON file intermediate to pass
structured
arguments between apps.  Most of the languages, including Python, can handle
these.

Original comment by<EMAIL_ADDRESS>on 1 Dec 2009 at 4:55

From @GoogleCodeExporter on May 31, 2015 11:24
Original comment by<EMAIL_ADDRESS>on 30 Nov 2009 at 8:57

Added labels: Type-Enhancement
Removed labels: Type-Defect


fix fix timeout to get tablet schema

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 buildall


[Feature] Allow disabling tenant menu

This PR allows the tenant menu to be entirely hidden for a panel otherwise using multitenancy. I have found this is something I'm wanting to do where I am using tenancy for automatic scoping of resources by setting the User as the tenant. In this case there is no need to have a tenant menu because the setup makes it impossible for a user to belong to multiple tenants.
Another possible (perhaps more likely) use case would be using multitenancy to share resources between multiple users, but where each user by design will only ever belong to a single tenant.

The only real benefit of our tenancy feature is so you can have switching like this and a different URL scheme... if you're using user-based tenancy you might as well just apply some global scopes and not use the feature? LMK if I'm wrong.

I'm not saying I have the best use case for this but I only need th4 scoping on on3 panel not all which is why I didn't go with a global scope. There probably is still a better way but it is working the menu is just superfluous in my case.
And I can imagine other cases where you may decide for one reason or another to limit a user to a single tenant but maybe use tenancy to share resources between users. Or maybe you want to opt to use your own tenant switcher. For example recreating the Jetstream style switcher in the user menu.
The only viable workaround I can think of right now is to echo out the opening of an html comment before and closing of an html comment after using render hooks which seems kinda silly.

Add the global scope in a middleware and only apply it to 1 panel

I never though you could configure a global query scope in Middleware but I guess that makes sense. Still dosent address the other possible use cases so I'm sure eventually someone is going to come along wanting to remove the menu for one of those other reasons.

If you have a user belonging to 1 team, for example, you still wouldn't want the team ID in the URL. Again, you would use a global scope there. As I said, the only reason you should use the Filament tenancy feature is if you have multiple different tenants that you want to switch between.

First off I'd disagree that you'd always want that.
Secondly as I also mentioned it gives more flexibility if someone prefers to use their own tenant switcher. I for example am not a fan of it or it's location and could see myself prfering to put a list of tenants in the users menu like Jetstream does.
If the work has already been done I don't see why not provide the option.

Okay, I do understand. I will hold off on merging this though until someone specifically asks for it to be removed from a many-to-many tenant setup. It can always be hidden or customized with CSS if you don't like the look of it.

@danharrin I was looking for a way to do this and stumbled across this PR. Just adding my vote :) I'd also find this useful for the reasons Ryan mentioned. I'm hiding it via CSS atm but loading unnecessary HTML isn't ideal.


[ClangImporter] Import objc_subclassing_restricted classes as 'public'

(rather than 'open')
This attribute was originally introduced for Swift classes exposed to Objective-C, but if it is used on a pure Objective-C class Swift should respect it. There might be an actual reason, such as a hot code path in ContrivedExampleKit checking against the class object's address instead of using -isKindOfClass:.
For source compatibility, only enforce this in Swift 5 mode.
rdar://problem/33971529

@swift-ci Please test


Closes: #1

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
The usernames beginning with characters other than alphabetic ones also returned True. Fixed that. Now they return False.
Fixes #1

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!


Unable to build with 128BitNeedleId

Hey.
Im currently trying out what I'd be able to do with seaweed and up until now everything's looking pretty fine.
In my projects I'm usually working with UUID's, so I tried to build with the 128BitNeedleId tag.
Sadly, the build is failing because of invalid operations ins /storage/needle_map/compact_map.go
My build, ran with:
CGO_ENABLED=0 GOOS=linux go build -x -tags 128BitNeedleId -o /home/sergio/work/git/seaweedfs/weed github.com/chrislusf/seaweedfs/weed
WORK=/tmp/go-build022534110
mkdir -p $WORK/b492/
cat >$WORK/b492/importcfg << 'EOF' # internal
# import config
packagefile github.com/chrislusf/seaweedfs/weed/storage/types=/home/sergio/.cache/go-build/bb/bbc307a316153e153aaa115c0d11a82efee3f5855f241fa22b51dc77865c2072-d
packagefile github.com/google/btree=/home/sergio/go/pkg/linux_amd64/github.com/google/btree.a
packagefile sort=/usr/lib/go-1.10/pkg/linux_amd64/sort.a
packagefile sync=/usr/lib/go-1.10/pkg/linux_amd64/sync.a
EOF
cd /home/sergio/go/src/github.com/chrislusf/seaweedfs/weed/storage/needle_map
/usr/lib/go-1.10/pkg/tool/linux_amd64/compile -o $WORK/b492/_pkg_.a -trimpath $WORK/b492 -p github.com/chrislusf/seaweedfs/weed/storage/needle_map -complete -buildid l12Sk0Lo9qUWdsK2kE8r/l12Sk0Lo9qUWdsK2kE8r -goversion go1.10.4 -D "" -importcfg $WORK/b492/importcfg -pack -c=4 ./btree_map.go ./compact_map.go ./needle_value.go ./needle_value_map.go
# github.com/chrislusf/seaweedfs/weed/storage/needle_map
storage/needle_map/compact_map.go:57:32: invalid operation: key - cs.start (operator - not defined on string)
storage/needle_map/compact_map.go:128:32: invalid operation: key - cs.start (operator - not defined on string)
storage/needle_map/compact_map.go:146:32: invalid operation: key - cs.start (operator - not defined on string)
storage/needle_map/compact_map.go:185:14: invalid operation: key - cm.list[x].start (operator - not defined on string)
storage/needle_map/compact_map.go:282:29: invalid operation: nv.Key - cs.start (operator - not defined on string)
Sadly, I'm simply not experienced enough with go, so sorry, if this is rather easy to fix. I'll keep on trying and I'll further dig into the code. If I find a solution myself, I'll comment and close.
Thanks for any help in advanced, much appreciated!

@Smarrito your build process is correct. It's just the 128bit version is not compiling any more.
The UUID as part of the file id seems good. But I think this is not really anyway better, because you will still need to store a UUID to volume id mapping somewhere.
So it is very likely the 128 bit version will be discontinued.


Upgrade H2 version to 2.0.202 to fix CVE-2021-23463

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
[x] Update the CHANGES log.

avoid false negative report from security scanning tools

What do you mean a false report? A CVE can't be produced in our case?

avoid false negative report from security scanning tools

What do you mean a false report? A CVE can't be produced in our case?

I mean the CVE can't actually exist in the user cases (in production env) because H2 is only for local test, WDYT?

avoid false negative report from security scanning tools

What do you mean a false report? A CVE can't be produced in our case?

I mean the CVE can't actually exist in the user cases (in production env) because H2 is only for local test, in another word, by "false negative report" I mean it's totally OK if we don't fix this, WDYT?

I think, from recommendation perspective, this CVE shouldn't rather than can't, because it can. H2 server could run in a server mode, which is more powerful and also could support OAP in cluster like MySQL did.
Referring from official doc, https://skywalking.apache.org/docs/main/latest/en/setup/backend/backend-storage/#h2

Activate H2 as storage, set storage provider to H2 In-Memory Databases. Default in distribution package. Please read Database URL Overview in H2 official document. You can set the target to H2 in Embedded, Server and Mixed modes.

We don't declare this should not be used in prod env.

Anyway, the above is just my feeling. I just want to remove  to avoid false negative report from security scanning tools from PR title and change log, because this is a real CVE.

This change seems breaking the e2e?

Here the patch becomes kind of unexpectedly large, because value becomes a reserved keyword in h2 and we didn't escape the column names, I slightly refactor the codes to add an escaper for JDBC implementations to escape the column/table names.

Noticed this. Do you check the solution of column alias mechanism in MySQL? I remember we have this in MySQL to prevert this conflict.

Noticed this. Do you check the solution of column alias mechanism in MySQL? I remember we have this in MySQL to prevert this conflict.

That would be preferable, I'll check


Restarting node service using shipit

I am having the following shipit.js file which I use to deploy my app, and restart it once everything is complete:
module.exports = function(shipit){
	require('shipit-deploy')(shipit);
    require('shipit-npm')(shipit);

  shipit.initConfig({
    'default':{
      workspace: '/tmp/app',
      repositoryUrl<EMAIL_ADDRESS>      branch: 'master',
      ignores: ['.git', 'node_modules'],
      rsync: ['--del'],
      keepReleases: 2,
      shallowClone: true,
      deployTo:'/home/deploy/app/httpdocs',
    },
    'server1':{
      restart: 'sudo systemctl restart app.service',
      servers: 'deployer@server1'
    },
    'server2': {
      restart: 'sudo service app restart',
      servers: 'deployer@server2'
    }
  });

  shipit.task('restart', function(){
      return shipit.remote(shipit.config.restart);
  });

};

You will notice that each server has a different command that restarts the service since one is using systemv and the other uses upstart to manage services
Running shipit server2 restart works fine, but running shipit server1 restart asks my for my password and then stays there doing nothing and I can only leave there by Ctrl+break. Has anyone encountered this thing? Am I doing something wrong?

@Loupax normally sudo will give you an interactive mode. But I advice you to give correct rights to your shipit user on that file to avoid using sudo.

Tried to do the same thing without using shipit just by doing ssh server sudo restart_my_server and received a sudo: sorry, you must have a tty to run sudo error that was not visible when using shipit.
Maybe there is something about the way errors get handled that causes the error not to appear?

Hey @Loupax, if this sorted out everything for you would you mind closing the issue?
Otherwise feel free to reply! (though that would suck if you still didn't have this working)

Haven't solved this issue with sudo, but I think we should close it anyway


⚠ Bofrost.de has degraded performance

In 7672e63, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1273 ms

Resolved: Bofrost.de performance has improved in 7c93cc5 after 13 minutes.


Clean up sanitizers

This PR attempts to get rid of most sanitizer warnings

All sanitizers complain about the huge allocations we do in our executor tests
UBSAN complains about empty memcpys on nullptrs
TSAN complains about some write-after-write data races in the stopping conditions. This is not an issue, since we always write the same value, but it can be mitigated by making the for-loop a reduction
TSAN complains about another write-after-write data race in bicgstab, where we do some unnecessarily duplicate work in parallel

The PR looks to be ready to review, only the sanitizer flags are seemingly not passed to the executed tests. I will investigate this

The result of this clean-up: Only the asynchronous iteration kernels (ParILU(T)/ParICT) are are complaining in TSAN, no further warnings except for the ones we deliberately produce in the sanitizer tests.


#12 - updating commander dependency; fixing warning issue

issue https://github.com/acro5piano/kysely-migration-cli/issues/12

Thanks.
I checked the changelog and confirmed it does not have breaking changes affecting this library.
https://github.com/tj/commander.js/blob/master/CHANGELOG.md

Released on v0.2.3
https://github.com/acro5piano/kysely-migration-cli/releases/tag/v0.2.3


Merge-- SSL: avoid calling SSL_shutdown() during handshake

Merge-- SSL: avoid calling SSL_shutdown() during handshake
&
update version number

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.yourchanges seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.

hi, this issue have been fixed in master branch.
See https://github.com/alibaba/tengine/pull/745.


First conversion commit to .yaml

@golbiga Let's chat this week. first iteration, already discussed with Matt T and Jaron.

@txhaflaire we need to remove the " " leading and trailing the predicates. If a customer copy/pastes those in the unified logs wont be triggered.


Schedule download?

Franiac
Would it be possible to have the program check and auto download new videos if it was left running?

please see my last comment on issue #76


[admin] fix missing characters

Docs Build status updates of commit 687a339:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch lindalu-fix-missing-chars has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Installation problem

Here is the output of the make comand
[ 11%] Building C object CMakeFiles/imgtxtenh.dir/imgio_magick.c.o
In file included from /home/dreyer/Downloads/imgtxtenh-master/imgio_magick.c:8:0:
/home/dreyer/Downloads/imgtxtenh-master/imgio_magick.h:14:31: fatal error: magick/MagickCore.h: No such file or directory
compilation terminated.
CMakeFiles/imgtxtenh.dir/build.make:158: recipe for target 'CMakeFiles/imgtxtenh.dir/imgio_magick.c.o' failed
make[2]: *** [CMakeFiles/imgtxtenh.dir/imgio_magick.c.o] Error 1
CMakeFiles/Makefile2:67: recipe for target 'CMakeFiles/imgtxtenh.dir/all' failed
make[1]: *** [CMakeFiles/imgtxtenh.dir/all] Error 2
Makefile:127: recipe for target 'all' failed
make: *** [all] Error 2

I  currently have magick 7.0.5-4 installed. Any ideas what might be wrong?

There is a big change from ImageMagick 6 to 7 and imgtxtenh is not prepared for this. Currently I don't have the time to fix this, so use the latest version from the ImageMagick 6 branch.


Alternative automatic API ref doc generation

Automatically generating API reference doc rather than using generate_module_rst.py file
Description
Enhancement. There are pros and cos to this new method. See https://docs.google.com/document/d/1AIV8-nGOAzqzrzcA8mxpcgdGKDKyQ9wpjVweCyFbl1g/edit for a comparison with the current method.
Minimal working example
To build the docs locally, don't run generate_module_rst.py. Instead, follow these instructions: https://github.com/JamesALeedham/GPflow/blob/33a89cb9b4ea23d57160492f27836d29691a4178/doc/README.md

Overall this looks very good - I've noticed that it doesn't show the type annotations in the signature anymore, but we can fix that by moving to proper Sphinx docstrings and explicitly having the :param argument_name: description everywhere.
More problematic is that as you point out it does not play nicely with the multiple dispatch. Would it be possible to make use of your approach for most of the modules, and have our custom multipledispatch-handling script for where it's needed?


Class Team from existing group

I don't appear to be able to create a Class Team from an existing group. Can I convert a staff or 'other' team to a class team as I can use an existing group with these 2 options?

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

ID: 43a2187e-570d-f235-3536-69df8eaa906a
Version Independent ID: da949f22-1617-c737-d5a4-5c51704443e8
Content: Enhance existing Office 365 groups with Microsoft Teams - Microsoft Teams
Content Source: Teams/enhance-office-365-groups.md
Service: msteams
GitHub Login: @LolaJacobsen
Microsoft Alias: lolaj

@alanbmhs thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Hope to support APPLE M1

Hope to support APPLE M1

Thanks for this feature request.
We have filed this internally as feature request for triage. At this time we don't have any updates if and/or when this would be available.


Add Postgres Tests to Performance Manager

Add tests that create a DatabaseManager object and ensure we can connect to the database.
Add pytest to Performance Manager CI
Add a postgres service to the test step in Performance Manager CI

Asana Task: https://app.asana.com/0/1203100082488441/1203144932470000/f

Just need specific postgres image, otherwise LGTM


When i use paypal smart button for payment,but the transaction_id save the payment order id info , and not real transaction_id

Describe the bug
Dear Master.
when i use paypal smart button for payment,but the transaction_id log the payment order id info

@xxl4
Thank you so much for your concern, we really appreciate your efforts toward bagisto.
I would love to know more about the issue. Can you please elaborate your query.
Thanks.

Thank you for reaching out. It looks like your request is not an issue report. For discussions, questions, or general support, we encourage you to use the Bagisto Forum. The forum is the best place for such queries, and you'll be able to connect with other community members and get the help you need.
Please reserve the GitHub issue tracker for reporting bugs and issues with the codebase. This helps us keep the issue tracker focused and ensures that problems are addressed promptly.
Thank you for your understanding!


Kavya - UI change for JoinHouseholdScreen & HouseholdDetailsScreen

UI for JoinHouseholdScreen & HouseholdDetailsScreen adhere to design system.

UI changes look good, LGTM!

Tested the app to ensure functionality still works, LGTM


SDR logo at the top of all pages at sdr.stanford.edu should be updated.

Tom pointed out that somehow we did not get the approved logo on the H2 site.
The correct logo file is here: https://drive.google.com/file/d/1KrqNa5MkocaDqipeBTj4XwBWMAlWX6gr/view
See the Design Doc for the header that is on every page (now updated with this link):
https://docs.google.com/document/d/1fcr2DYo7OrX-qTdeUOTWKSS1rlq_47sehESmbvgUsrk/edit#heading=h.o1072wkbq96t
and for the home page:
https://docs.google.com/document/d/1fcr2DYo7OrX-qTdeUOTWKSS1rlq_47sehESmbvgUsrk/edit#

fixed by #2152

PR was merged;  Amy said deploy could wait until Monday dep updates are deployed.

Now with capital R in Repository on home page!


Comment:Step-Basic-Db-connector

im trying to run this in my command line (windows) node ./examples/step/01/db-connector/1.js, and it's giving me Error: Cannot find module 'feathers', i'm new and lost sorry thanks!

Ensure you ran the dependencies while in the correct folder. See https://docs.feathersjs.com/guides/step-by-step/preliminaries.html#install-dependencies-used-by-the-examples . It should log a lot of lines to the console the first time its run.
Then ensure your current folder is the one containing ./examples before running the command.

thanks mate! working now!


Fix #396 default year range

Changed the default range to present 2014 accidents

Why did you close it?

It constantly pushes your fix in main.py too
Any idea why?

I don't know... It also looks as if I authored your commit.
Try creating a new updated branch and apply the change there.

I noticed that too. I tried another once or twice but it kept pushing your fix.
Thought maybe git needs time to update and I'll try again with a new one


CASMINST-5597 make initial IUF-base workflow template

Summary and Scope
CASMINST-5597 make initial IUF-base workflow template.
Summarize what has changed. Explain why this PR is necessary. What is impacted? Is this a new feature, critical bug fix, etc?
Is this change backwards incompatible, backwards compatible, or a backwards compatible bugfix?
Issues and Related PRs
List and characterize relationship to Jira/Github issues and other pull requests. Be sure to list dependencies.

Resolves [issue id](issue link)
Change will also be needed in <insert branch name here>
Future work required by [issue id](issue link)
Documentation changes required in [issue id](issue link)
Merge with/before/after <insert PR URL here>

Testing
List the environments in which these changes were tested.
Tested on:

<development system>
Local development environment
Virtual Shasta

Test description:
How were the changes tested and success verified? If schema changes were part of this change, how were those handled in your upgrade/downgrade testing?

Were the install/upgrade-based validation checks/tests run (goss tests/install-validation doc)?
Were continuous integration tests run? If not, why?
Was upgrade tested? If not, why?
Was downgrade tested? If not, why?
Were new tests (or test issues/Jiras) created for this change?

Risks and Mitigations
Are there known issues with these changes? Any other special considerations?
Pull Request Checklist

[ ] Version number(s) incremented, if applicable
[ ] Copyrights updated
[ ] License file intact
[ ] Target branch correct
[ ] CHANGELOG.md updated
[ ] Testing is appropriate and complete, if applicable
[ ] HPC Product Announcement prepared, if applicable

we need to update the chart.yaml to use this new image and then update csm manifest to use the new chart


docs/Fix GraphQL adv rate limiting plugin errors and inconsistencies

https://konghq.atlassian.net/browse/DOCS-998
Direct link to generated PR output:
https://deploy-preview-1994--kongdocs.netlify.app/hub/kong-inc/graphql-rate-limiting-advanced/

I'm assuming the "of" is a typo and should be "or" - contextually, "of" doesn't make sense. If that is right after all, then the descriptions need some rephrasing.

That was my first inclination but then I wasn't sure. Perhaps it should be something like "One of multiple ... "? @marcoam do you have some insight for us?

Hey @tlblessing / @lena-larionova - Looks a LOT better! Thank you for working on this :)
While we've made a ton of progress, I will still ask for additional changes. Specifically, the curl commands aren't working as they are using arrays, which don't seem to work now (they used to work before). When making calls like this:
  --data config.limit=10,100 \
  --data config.window_size=60,3600 \

It complains. Take a look at the following conversation:
https://kongstrong.slack.com/archives/C0DSXDXV1/p1588149697286100
So we have a couple of options:

Rather than using an array (which is a more complicated example) we simply work with one time window. What that means is that we put something like this:

  --data config.limit=10 \
  --data config.window_size=60 \

But as I do feel that having multiple windows is important to highlight, that would mean that we need to add a new section to just address multiple time windows and give examples.

We can use stick to the multiple windows and use a command that looks like this:

curl -i -X POST --url http://kong-ip:8001/services/{service}/plugins --data 'name=graphql-rate-limiting-advanced' --data 'config.window_size[]=60' --data 'config.window_size[]=3600' --data 'config.limit[]=10' --data 'config.limit[]=100' --data 'config.sync_rate=10'

or another call that a curl expert can come up with :)

We can use httpie, which is a lot cleaner but requires our user to have httpie. As we don't always use that across our site, I would recommend against this option.

On an somewhat unrelated note, note that the non-GraphQL curl command is also broken: https://docs.konghq.com/hub/kong-inc/rate-limiting-advanced/#notes. So when fixing this, I would suggest that we fix the aforementioned page.
Thanks!

@marcoam thank you for your comments. We can show single and multiple window examples. When @GayleNeumann, @lena-larionova, and I reviewed the backlog tickets for plugins yesterday, we encountered this ticket https://konghq.atlassian.net/browse/DOCS-441 concerning array syntax. So it looks like this array syntax [] update will need to be made across the board where applicable.

I already merged the advanced rate limiting PR that also added the sentinel pw config but will log another ticket for that. This ticket is increasing in scope but that is fine, we need to fix this. Were you going to provide cURL examples instead of the HTTPie as you mentioned in our convo? Thank you again for your assistance.

@marcoam thanks for the updated array syntax. I tested it locally and it works:
`➜  ~ curl -i -X POST --url http://localhost:8001/services/example_service/plugins
--data 'name=graphql-rate-limiting-advanced'
--data 'config.window_size[]=60'
--data 'config.window_size[]=3600'
--data 'config.limit[]=10'
--data 'config.limit[]=100'
--data 'config.sync_rate=10'
HTTP/1.1 201 Created
Date: Wed, 29 Apr 2020 17:17:22 GMT
Content-Type: application/json; charset=utf-8
Connection: keep-alive
Access-Control-Allow-Origin: http://localhost:8002
X-Kong-Admin-Request-ID: UyEEDdB7ykJeq825U37HAqQdwktth1OQ
Vary: Origin
Access-Control-Allow-Credentials: true
Server: kong/<IP_ADDRESS>-enterprise-edition
Content-Length: 792
X-Kong-Admin-Latency: 157
{"created_at":1588180642,"config":{"redis":{"host":null,"sentinel_master":null,"timeout":2000,"cluster_addresses":null,"sentinel_role":null,"database":0,"port":null,"sentinel_addresses":null,"password":null,"sentinel_password":null},"score_factor":1,"window_type":"sliding","hide_client_headers":false,"dictionary_name":"kong_rate_limiting_counters","window_size":[60,3600],"sync_rate":10,"cost_strategy":"default","strategy":"cluster","namespace":"u88PdLjYgOkEj4j1FmydM6t05aaeWW4s","limit":[10,100],"identifier":"consumer","max_cost":0},"id":"03809793-1242-47e3-a5b8-7a455176cc13","service":{"id":"ef3c597b-8259-4437-84d2-38f33d160cdc"},"name":"graphql-rate-limiting-advanced","protocols":["grpc","grpcs","http","https"],"enabled":true,"run_on":null,"consumer":null,"route":null,"tags":null}%`

Hey @tlblessing , that's great!
Ideally, we should have a standard throughout our docs to try and make sure that we use one notation. I talked to one of our engineers (https://kongstrong.slack.com/archives/C0DSXDXV1/p1588149697286100) and he even improved my call even more, which I think would be a good standard to keep. I'll go through this page and modify the calls to ones that work :)

Hi @tlblessing - I have gone ahead and updated the curl commands. I tested them, but please, do you mind testing them on your side? Want to make sure they work. A couple of random comments to mention of things I noticed:

It's hard to understand, from a reader's perspective, if you are guiding me through a specific example/use case (as we do in the section "Usage") or if it's just general guidance (as we do in "Limit query cost by using config.max_cost"). The reason why this is important is because the curl command would change based on this. If we are talking about a specific use case, we can refer to the service, route or plugin we're talking about. In other words, calls can be like:
curl -X POST http://kong:8001/services/example/plugins
and not
curl -X POST http://kong:8001/services/{service}/plugins

Does that make sense? So I will leave it up to you on what to do and how to organize it

While these curl commands aren't the most concise, I didn't want to introduce a new format when all of our other pages are using this type of curl calls.

Hope this helps!

I am okay if this is changed back to {service} depends on what we're trying to do: follow the use case that we're building on top of or just mention how to change the default strategy...

I think it is understood to replace the values as stated upfront in the topic

Hi @tlblessing - I have gone ahead and updated the curl commands. Here is the commit:
#4f8cdb6
I tested them, but please, do you mind testing them on your side? Want to make sure they work. A couple of random comments to mention of things I noticed:

It's hard to understand, from a reader's perspective, if you are guiding me through a specific example/use case (as we do in the section "Usage") or if it's just general guidance (as we do in "Limit query cost by using config.max_cost"). The reason why this is important is because the curl command would change based on this. If we are talking about a specific use case, we can refer to the service, route or plugin we're talking about. In other words, calls can be like:
curl -X POST http://kong:8001/services/example/plugins
and not
curl -X POST http://kong:8001/services/{service}/plugins

Does that make sense? So I will leave it up to you on what to do and how to organize it
@marcoam, @lena-larionova just gave Gayle and I a walkthrough on plugin templates and how it is generated. Docs above the usage section are autogenerated. So we should try to stick with that convention {service}. In the Usage area, it is more freeform (not generated) but I still think an obvious placeholder is preferable. I also noticed the template has kong rather than kong-ip like your new examples, so I should probably reset that back.


I can never sign out

Login works, logout doesn't as it hits the wrong api server as the signout button hits staging oauth api not production one.
It seems the Oauth config for panoptes client defaulting to staging if it's not set in here.

I think this is fixed in the staging version. Will keep an eye on it though.

out of interest - where is staging deployed?

@camallen https://preview.zooniverse.org/oldweather (sorry, just now saw this comment)


chore: adds search example using autocomplete input

@nickskalkin was wondering how to use a fragment container with the AutocompleteInput — the answer is: it just works. Use the matchConnection instead of the searchConnection (to be deprecated), however.
https://github.com/artsy/force/assets/112297/f1de9123-5c1e-4069-8907-dc799421e47b

Might as well have a pattern for debounce since it's surprisingly hard to do for React inputs.

wow, thanks a lot @dzucconi! I'll try to refactor components I wrote when I will have some spare time!


Extend generate_bom.py example to consider variants

Should either be able to specify the variant we want to generate the BOM for AND/OR generate a BOM for each variant.
Also might want to consider adding another column for different variant fits.

@kdumontnu Could you point to an example repo on AS Hub?

Just a reminder that this issue/repo is public. I don't know of a public repo that has variants. I can send you something privately.


Add a mock showtraceback() method to MockKernel

Closes #130

Closing this in favor of https://github.com/posit-dev/shinylive/pull/161


Checking const that's set to null check expression should be equivalent to testing null check expression directly

Given the following snippet,
/* @flow */

type Socket = {
  connected: boolean;
  emit: (data: any) => void;
};

function sendData(socket: ?Socket) {
  const canTransmit = socket && socket.connected;
  if (canTransmit) {
    socket.emit('hello world');
  }
};

Flow 0.18.1 complains that the call of method emit might result on a potential null or undefined value.  This is somewhat similar to #76, but in this case
(a) there's no way other function closing over the scope that canTransmit is in
(b) canTransmit is a const, so it couldn't be modified anyway (even if (a) weren't true)
Am I missing something or should Flow be able to detect that socket.emit() won't be called if socket isn't truthy? Definitely with the const, but even if it were just var or let, right?

Currently refinements are syntactic. While the canTransmit booleans truthiness also implies the socket binding is non-null, Flow is not currently able to carry that implication along with the canTransmit boolean, and activate it in the conditional. #34 has some existing discussion around this, but I feel like this feature is still a bit far off.
For now, the following does work:
function sendData(socket: ?Socket) {
  if (socket && socket.connected) {
    socket.emit('hello world');
  }
};

Here the conditional expression (specifically, the if (socket) part) will refine the ?Socket type to just Socket.
Hope that helps!

Thanks @samwgoldman. Yep, I have to resort to a syntactic refinement right now, which is fine. Just makes the code more verbose that it needs to be.
Reading #34, it's not 100% to me whether the scope of that issue is to merely provide the ability to add predicates to helper functions like _.isFunction, or to make the refinements that they can provide semantic as well. I suppose your safeFormatNumber example (https://github.com/facebook/flow/issues/34#issuecomment-126774771) would imply semantic refinements, right?
Anyway, thanks for the explanation.

Duplicate of https://github.com/facebook/flow/issues/7228


https://github.com/felipemaiapolo/tinyBenchmarks/blob/main/tutorials/py-irt is no longer available

https://github.com/felipemaiapolo/tinyBenchmarks/blob/main/tutorials/py-irt is no longer available

Hi,
In our tutorial readme file, we explain how to clone this directory.


Fixed seeking behavior of property track with discrete interpolation

Fixed #52518 in conjunction with #52543.
AnimationNodeTransition seeks the animation of transition destination, but the seeking of property track with discrete interpolation was broken. With track of discrete interpolation, it may be wrong to do nothing in the frame of seeking.

Thanks!

Cherry-picked for 3.4.

is this issue fixed? because i can still experience a one frame delay in animation tree node state machine in 3.4.4

State machine has different playback method, so probably that is separated issue.

oh then do i open a new issue and provide the details of reproduce? i haven't seen one that is specific to state machine so far.

I think there was an issue somewhere about the first frame after the StateMachine transition is broken, but I can't remember. BTW, the animation team has already confirmed that the StateMachine transitions are broken.


WILL WORK ON THIS I SWEAR

REMINDER FOR MYSELF: GET SOME MOTIVATION AND FIX ISSUES AND MAKE SOME STABLE RELEASE!!!

Дай Бог тебе сил для этого проекта, Колян :)

:+1:

Yes, go on, Nikolai! I'm using your GoogleScraper in my own project, but only for parsing Bing SERP. It would be gorgeous if I was able to use it more.

Thanks for the huge support! I did actually some work on it :) The next 6 days, I will spent 4 hours on each day on GoogleScraper. Goals: Make it more stable, get some documentation on readthedocs.org and also support different proxies! And the best of it all: Write tests for it such that the pain finally stops...

:+1:

:+1:

Or make somebody collaborator :)

Great I'm loving this project, I wish it was 100% percent functional


Update nintendont.md

fixed the IPL description + made the file structure look nicer
Description

I think the paths should be changed to those present on the wiki version of the page. There's no need to put single-disc games into subfolders. Really, the whole page should be updated to reflect the wiki's changes, but the game paths are most relevant here.

Can confirm it renders fine at the very least.

I think the paths should be changed to those present on the wiki version of the page. There's no need to put single-disc games into subfolders. Really, the whole page should be updated to reflect the wiki's changes, but the game paths are most relevant here.

That’s literally how Nintendont reads games. All games have to be in sub folders whether you want them to be there or not.

All games have to be in sub folders whether you want them to be there or not.
I literally have my setup so that no games are in subfolders and it works fine. I'm speaking from experience. It's undocumented, but it works.

All games have to be in sub folders whether you want them to be there or not.

I literally have my setup so that no games are in subfolders and it works fine. I'm speaking from experience. It's undocumented, but it works.

if you want gamecube games to read and play in usbloader gx this exact file structure is required even if nintendont by itself is less strict.


Reset attributes at end, emit newline before prompt.

Hi, I played with your program, and noticed it left my terminal in a different state.  This fixes it for me on OSX Terminal.app.  Haven't tested others, but I would expect it to be common, unless goterm has some reset logic which is failing in my case.
Also put a newline after the URL so that the URL doesn't run on into the prompt.
[Also, apologies if my pull request is wrong.  First pull request on github :-).]

Could you revert to the old code and test to see if the issue persists consistently? I ask this because I have also had the issue where it seems to leave the terminal in a different "state" i.e. does not clean up properly, though this seems to be quite rare.
I can see you have added \033[0m which I believe simply just resets the styles and might not be a permanent fix.
I will take a further look into goterm and its ability to reset/clean-up properly, though the library isn't that extensive. Thank you for the pull request, however.

I do not offhand see how it could ever go back to the normal attributes, unless perhaps your prompt includes an ESC[m sequence.  The code sets ESC[37m and ESC[97m to set droplet colors, but never resets attributes.  There is a constant RESET in goterm which is used to reset attributes after a color change - but only when using the goterm color-changing functions such as tm.Color(), whereas your code is sending manual sequences which goterm will forward but not interpret.
Did I misunderstand the end code?  Due to the call to Exit(), I assume that that's intended to be the last output for the program, so unless the mainline code runs again, ESC[0m will reset styles to default.  There is a tiny race condition between the core of main() and the signal-handling goroutine, but I'm not sure there is a reasonable way to resolve that without more-intrusive changes.  Like maybe putting a select in the for loop with case <-c doing the exit code and default doing the current contents of the for loop.

Ah, I see what you meant now by leaving your terminal in a different state.
Shortly after I closed the PR, I did make a change to the escape codes in the colour of the drops and added an escape sequence to reset the terminal in the end, which now I see was the main goal here.

Now it leaves underlining (attribute 4) turned on :-).

Oh, I really did forget to remove the underline after the print statement, though when I did test on my MacOS native Terminal, it seemed to clear all fine no issues even without the reset code.

Oh, I get you, to underline the link!  In any case, current code leaves me with a clear screen with default settings, with your github landing page linkable and underlined.  Success!
The only other comment I'd have is to consider doing what "top" does to switch to alternate screen buffer, with ESC[?1049h and ESC[?1049l .  Informally, the alternate screen buffer doesn't participate in scrollback.  Those are xterm extended codes, but pretty broadly supported.  More portable would be to use terminfo smcup/rmcup or termcap ti/te, but I don't know how far down the rabbit hole you care to go.
https://stackoverflow.com/questions/15044274/how-does-vi-restore-terminal-content-after-quitting-it/15044321#15044321