Load terraformsh config from parent directory

Hi @Th0masL,
I reopened a pull request with some of the changes you requested. I liked your proposal, but I'm just trying to find a way to formulate the code so the functionality is based on reusable functions, rather than one function that searches for files in two different ways. (Your suggestion is completely appropriate for most software projects; but since this is more of a pet project, I'm being annoying and trying to make the code better for its own sake :wink: ) )
Will make another attempt at it, but if I don't get far we can just go with your original suggestion.

One problem to consider in looking for parent terraformsh configs is overloading/precedence.
If there's a config in the current directory and a config in the parent directory, how do we load them?

If the current directory's file is taken as authoritative, we shouldn't load the parent file, because it will overload things like CD_DIR.
If the parent directory's file is taken as authoritative, do we load the current directory's file and then the parent's? That may result in a mix of options that conflict with each other, because the current directory and parent directory may have been intended to run two different Terraform modules with different Terraform versions and options.

It seems that the simplest/most reliable option is to only load one config. And it seems most logical to load the current dir's file as that is how the current behavior works, and generally you'd expect to load a config from the directory you're currently in with most tools.
Similarly, it seems like loading the file "closest" to your current dir makes the most sense, if it isn't in your current dir. So that leaves situations when there's a config in a parent dir and a config in a child dir: do we assume we always want to load a parent dir's config, even if it might not relate to the current dir? And if there's a child dir, do we want to instead seek to that directory and run Terraform there?

I think that it makes sense to only read terraformsh.conf files from the parent folders, as it would be to risky to load the ones in the child folder, since terraformsh cannot predict what to do with the child folders.
Regarding where to stop the loading of files in the parents folders, I think it makes sense to either:

decide to only read the first match of terraformsh.conf file (first try to find in the current folder, then try any parent folders until finding a match)
OR allow to read multiple terraformsh.conf files in a row, but the behavior of that might be tricky to predict as things the different terraformsh.conf files could conflict. If we do that, I think it's important to an Environment Variable that could be exported by the terraformsh.conf file that would tell terraformsh to stop reading any other parent files.

Just for example, in my current implementation, in my repo/branch, I'm exiting after the first match.
See https://github.com/Th0masL/terraformsh/blob/main/terraformsh#L496

I think either of those options could work, but I think this is one of those times where we have a lot of options but not really know which is the better choice because we haven't used them yet.
Can you tell me more about your use case for this function?
For myself, I use terraformsh with the idea that I only run it from a directory that I want to deploy one Terraform module from. I also load the ~/.terraformshrc file so that I have a general global option to turn on Terraform's compact mode, but that breaks with older versions of Terraform, so I have to add options in certain modules' configs to override those settings. Kind of annoying but I couldn't find a better way (particularly because I have one directory hierarchy full of modules with a half-dozen different Terraform versions)

In my setup, I'm using a parent/child folder structure to breakdown Terraform stacks by folder, but I want a single terraform.conf script file for enforcing the same behavior in all Terraform stacks.
Here is an overview of my folder structure:
$ tree
.
├── aws
│   ├── 1-vpc
│   │   ├── 0-backend-dynamic.tf # Generated dynamically by terraformsh.conf
│   │   ├── 1-variables-dynamic.tf # Generated dynamically by terraformsh.conf
│   │   ├── 1-variables-dynamic.tfvars # Generated dynamically by terraformsh.conf
│   │   ├── 2-locals.tf
│   │   ├── 3-errorcheck.tf
│   │   ├── 4-providers.tf
│   │   ├── 5-modules.tf
│   │   ├── 6-output.tf
│   │   ├── vpc-dev.json # Stack specific JSON files used as input
│   │   ├── vpc-prod.json # Stack specific JSON files used as input
│   │   └── vpc-shared.json # Stack specific JSON files used as input
│   ├── 2-ecs-cluster
│   │   ├── <... dynamically generated files ...>
│   │   ├── 2-locals.tf
│   │   ├── 3-errorcheck.tf
│   │   ├── 4-providers.tf
│   │   ├── 5-modules.tf
│   │   ├── 6-output.tf
│   │   └── <... stack specific JSON files ...>
│   ├── 3-ecs-service-frontend
│   │   ├── <... dynamically generated files ...>
│   │   ├── 2-locals.tf
│   │   ├── 3-errorcheck.tf
│   │   ├── 4-providers.tf
│   │   ├── 5-modules.tf
│   │   ├── 6-output.tf
│   │   └── <... stack specific JSON files ...>
│   ├── aws-dev.json # AWS specific JSON files used as input
│   ├── aws-prod.json # AWS specific JSON files used as input
│   └── aws-shared.json # AWS specific JSON files used as input
├── global-dev.json # Global JSON files used as input
├── global-prod.json # Global JSON files used as input
├── global-shared.json # Global JSON files used as input
├── terraformsh.conf  # Single terraformsh.conf file, stored in parent folder
└── terraformsh.conf.json # JSON file that helps configuring terraformsh.conf

@Th0masL sure, if you could share the content that would be great!
So what I've done in the past is in aws/1-vpc/terraformsh.conf I will have content like:
source ../../terraformsh.conf
source ../terraformsh.conf
CD_DIR=../../aws/modules/k8s-vpc

Since terraformsh.conf is just a shell script, I can use the shell command source or (better for POSIX but worse for readability) . to load in a parent terraformsh.conf file.
To run this in CI/CD, I just check out the repo, then cd aws/1-vpc && terraformsh, and it inherits parent configs.
I also use symlinks to commonly-used configs to keep the configs DRY. (In the past I have experimented with functionality that auto-generates temporary config directories by overlaying multiple directories on top of one another, but some weird problems crop up because of that, and the current system ends up being much simpler while still working)

Would still like to see your sample configs, but I'm leaning toward the terraformsh env var option that would tell it to look in parent directories for more configs to load. The value could be a number to indicate how far back to look. That way the user could either opt to do the source trick above, or auto-load parent configs.
Of course... if there is such an option, what happens when more than one config sets that option? Do we assume the second one is redundant? Or say that they both indicate how many parents to traverse; do we assume that the second instance of the variable increases the number of parents to traverse? Do we ignore it? Is it possible someone made a mistake by configuring that option in a parent file?

I've been pretty busy recently so I couldn't spend too much time trying to implement a more complex way to parse parent folders, ans you suggested.
I had a look at your suggestion, and that's true that the following approach would work fine for my use case:

So what I've done in the past is in aws/1-vpc/terraformsh.conf I will have content like:
source ../../terraformsh.conf
source ../terraformsh.conf
CD_DIR=../../aws/modules/k8s-vpc

So I'm going to use your suggested approach for the time being, and I might work on an improved semi-automated way to load files from parent folders in the future, but for the moment that's fine :)
Thanks !

You can close this PR for now. Thanks !


feature request: en and de in one request

the old imprint service returned on default both languages
<div lang="en">
   ...
</div>
</div lang="de">
...
</div>

can we provide this feature in the new one as well, maybe using something like ?locale=all

what's the usecase for providing both languages on the same page?

what's the usecase for providing both languages on the same page?

exactly this, having both languages on the same page

usually your website would be in either de or en, or it would allow to toggle website language. hence my question why you would ever need to display both on the same page.

Because currently we show both in existing websites. But I can l can work around the current implementation


Update toc.yml

Learn Build status updates of commit bfd5d27:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/azure-video-indexer/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
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.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@Juliako - can you use the private repo for this change? thanks.


Length of File Path

Reported by Ansley B Manke on 15 Oct 2001 21:50 UTC
LET VAR2 = VAR1is limited to less than 40 characters}
ds_name too short: lengthen to match des_name
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/324

Comment by<EMAIL_ADDRESS>on 15 Oct 2001 21:52 UTC
LET VAR2 = VAR1is limited to less than 40 characters}
Fix: increase length of ds_name  to match length of des_name


T-Systems Smart Contract

Context:
We need a smart contract that will manage T-system's rewards.
Definition of Done:
@laynelafrance  to provide specs

As Josh suggests, we could do this with a transaction that stores the amount of tokens sent directly in storage, rather than writing a new contract.


Document debug n updates

https://github.com/adventuregamestudio/ags-manual/wiki/Globalfunctions_General#debug
Debug(2, 0) command that displayed walkable areas is superceded by Debug(2, n), where "n" is a mask type: 0 - none, 1 - hotspots, 2 - walkbehinds, 3 - walkable areas, 4 - regions. This command also works as a toggle switch, calling it with the same mask index will turn it off.
Debug(5, n) command that displayed character's walk paths now also works as a toggle switch.
Update the behavior update of Debug for 3.6.0 in the above entry

added mentions in here: https://github.com/adventuregamestudio/ags-manual/wiki/Globalfunctions_General/_compare/657dd7f^...657dd7f


Improve error messages for invalid Azure credentials

To reproduce,

Go to Cluster Management > Cloud Credentials
Click Create
Click Azure
Enter valid credentials
Get error saying authentication test failed:

I also tried to create cloud credentials in the AKS cluster provisioning form and it gave a lot of errors about JSON syntax:

This is probably operator error on my end, but those syntax errors make it look like something is broken. Can someone confirm they can create cloud credentials?

Although it is not easy to find how to create a secret (or the client ID), I have found things labelled Client ID, Subscription ID, and I created a secret and used the Client Secret value. This always leads to an authentication error as described above.
Perhaps someone who has done this before should test it to be sure?

Apprently operator error is very likely. I have successfully created credentials with the required fields

What did the problem turn out to be? could it be turned into a better error message?

In the end, it was a problem with how I set up things on the Azure side. As I was just trying to test I didn't go into detail on setting up Azure. Jordon provided an Azure that was properly configured and everything worked as expected. Ideally, we would give more specific information about what is failing but I get the feeling that we cannot get anything more specific from Azure.
I think we can close this, as it does work, if you know how to set up Azure properly

We should provide a link to the docs. I think the docs might be slightly out of date was well, so we should check and work with docs team as necessary.

Work was done to improve the handling of errors here, so I don't think this is an issue.


Default search paths not editable

In our Settings tab, some of the "Defaulat Search Paths" are not editable by the user.

No longer relevant.


Updated screenshot

Description
This PR fixes #1580
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

Hey @sudhanshutech  I have generated a new pr resolving all the errors coming in the previous pr .I hope it matches the requirements thanks .

Hi @surajvast1  ,
Thanks for your PR! I encourage you to share this in the Website Meeting tomorrow at 6:30 PM IST. Share your screen, and walk us through it. It's cool to take and share feedback. Check the Doc here to include this PR.
You're welcome at the meeting
Check to community calendar for more details of meetings at Layers.

@surajvast1 update?

I am packed up with my exams . I'll complete it asap

@sudhanshutech Screenshot is updated

Its a simple merge i hope it  matches the requirements please merge this .


Allow magellan to support global resource strategy.

To allow magellan to support global resource strategy.
This PR contains

remove some unnecessary dependencies
remove promise library q
add resource strategy support

Codecov Report

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

@@            Coverage Diff            @@
##             master     #257   +/-   ##
=========================================
  Coverage          ?   97.37%
=========================================
  Files             ?       31
  Lines             ?      953
  Branches          ?        0
=========================================
  Hits              ?      928
  Misses            ?       25
  Partials          ?        0

Impacted Files
Coverage Δ

src/help.js
100% <ø> (ø)

src/test_queue.js
100% <100%> (ø)

test/mock/mockSetupTeardownListener.js
100% <100%> (ø)

src/get_tests.js
100% <100%> (ø)

src/profiles.js
96.92% <100%> (ø)

src/util/mkdir_sync.js
100% <100%> (ø)

src/hosted_profiles.js
100% <100%> (ø)

src/util/load_relative_module.js
100% <100%> (ø)

src/test.js
100% <100%> (ø)

src/cli_help.js
100% <100%> (ø)

... and 14 more

Continue to review full report at Codecov.

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


Update and fix portuguese messages

Short description of what this resolves:
These changes will resolve a lot of portuguese errors and remove distinct meanings from original messages in English.
Changes proposed in this pull request:

Fix errors in Portuguese language.

Fixes: #
How Has This Been Tested?
My changes didn't affect code files, but I tested package.nls.pt-br.json syntax on VS Code.
Screenshots (if appropriate):
It's not necessary.
Checklist:

[x] I have read the contribution guidelines.
[ ] My change requires a change to the documentation and GitHub Wiki.
[ ] I have updated the documentation and Wiki accordingly.

there are conflicts, can you solve the conflics so i can merge it


create methods to fix river input data

The snippet below comes from PR #1127
It was intended to fix the river arrays so that the resulting river will pass validation.
        target_bottom_subset = target_bottom.sel(
            {"layer": regridded_package_data["bottom_elevation"].coords["layer"]}
        )
        regridded_package_data["bottom_elevation"] = xr.where(
            regridded_package_data["bottom_elevation"] > target_bottom_subset,
            regridded_package_data["bottom_elevation"],
            target_bottom_subset,
        )
        regridded_package_data = mask_arrays(regridded_package_data)
        regridded_package_data["stage"] = xr.where(
            regridded_package_data["stage"]
            > regridded_package_data["bottom_elevation"],
            regridded_package_data["stage"],
            regridded_package_data["bottom_elevation"],
        )

However the reviewer @JoerivanEngelen  pointed out that these fixes should be in separate functions that the user calls so that these changes do not get carried out without the users consent.

https://github.com/Deltares/imod-python/pull/1172


Fetching data from RMap is slow. 11000 entries in RMap taking total 207mb. Is there  a way to make it fast. Every minute my cache is updated. so,I want to fetch all entries in RMap every minute.

RMap containing 11000 entries (RMap<String, Map<Integer, Data>>) .Total size 207 mb.
Time taken to retrieve all keys - 8-10 seconds.
My Rmap is getting updated every minute. So i need to get data from RMap every minute
Is there a way to fasten get call.
Redis version - 5.0.1
Redisson version - 3.10.4
Redisson configuration - single node

Try to use SnappyCodec and RMap.readAllMap method

I am already using RMap.readAllMap with JsonJacksonCodec (since I have third party classes which doen't implement serializable).
SnappyCodec would need all classes to implement Serializable I guess.

you can supply JsonJacksonCodec to SnappyCodec like this: new SnappyCodec(new JsonJacksonCodec())

Tried with SnappyCodec and RMap.readAllMap but still same performance.

Have you tried to use RLocalCachedMap?

I believe RLocalCachedMap will not fit my use case.
My redis cache is going to get updated every minute and eventually it wlll go around 2 - 2.5 gb of data in redis. And also every minute all entries are getting processed. Also, the architecture is such that my instace is supposed to be stateless (no cache).
Is there any way by which all entries in Rmap is fetched in parallel?

Is there any way by which all entries in Rmap is fetched in parallel?

I'm afraid no.
RMap.readAllMap returns result in one network call

RMap<String, RMap<Integer, Data>>
So ,in above nested RMap, RMap.readAllMap  will fetch all nested RMap  in single n/w call right?
if so that means its size of data which is making RMap.readAllMap slow?

So ,in above nested RMap, RMap.readAllMap will fetch all nested RMap in single n/w call right?

No, in this call Redisson will create RMap object for each map value, but won't fetch them.

RMap<String, RMap<Integer, Data>>
So, whne iterated over all entries of outer RMap and RMap.readAllMap on each inner RMap will be a seperate n/w call?

@sagarsitap596 How does your inner RMap updated?  If you are updating it in the same way then it is the best not to use the nested RMap and replace is with a simple Map instead. This will reduce the total number of calls to the server side. Additionally, you should be able to use getAll(Set) or getAllAsync(Set) to do reduce the total number of calls even further. Please be aware the bigger a single fetch is, the longer the delay other client/process will experience.

I just get inner map since I know the key to it while updating.
I tried with normal inner Map. Still it takes same amount of time to fetch whole data.

Are you using batched fetch by any chance with getAll(Set) or getAllAsync(Set)?

Nope.
I am fetching all the data present in RMap.   RMap.readAllMap

Physically fetching RMap<String, RMap<Integer, Data>> including nested RMap shoud be slower than fetching RMap<String, Map<Integer, Data>> If you are saying there is no difference between the two, I think something is not quite right here.

Surprisingly but yes both are taking same time.
I am using RMap<String, RMap<Integer, Data>> because my inner RMap is updated by multipe threads and I dont want data to be overridden.

Can you show a snipet of your code?

{
Map<String, Map<Integer, String>> result = new HashMap<>();
RMap<String, RMap<Integer, String>> rMap = redissonClient.getMap("key");
Map<String, RMap<Integer, String>> localMap = rMap.readAllMap();
for (Map.Entry<String, RMap<Integer, String>> entry : localMap.entrySet()) {
Map<Integer, String> map = new HashMap<>();
String id = entry.getKey();
map.putAll(entry.getValue().readAllMap());
result.put(id, map);
}
return result;
}

{
    Map<String, Map<Integer, String>> result = new HashMap<>();
    RMap<String, RMap<Integer, String>> rMap = redissonClient.getMap("key");
    Map<String, RMap<Integer, String>> localMap = rMap.readAllMap();
    for (Map.Entry<String, RMap<Integer, String>> entry : localMap.entrySet()) {
        Map<Integer, String> map = new HashMap<>();
        String id = entry.getKey();
        map.putAll(entry.getValue().readAllMap());
        result.put(id, map);
    }
    return result;
}

Have you tried to use RBatch to populate map.putAll(entry.getValue().readAllMap());?

Nope. I will try and let you know

@sagarsitap596 Any update?

Closed for no further communication.


Fix cpu format

/retest

[test]

@nurali-techie can you please take a look at the build failure?

@alexeykazakov build https://ci.centos.org/job/devtools-fabric8-oso-proxy/163/console was failing with error flag provided but not defined: -test.v


Logging example doesn't work

The logging example at item 3 in chapter Logs (https://docs.microsoft.com/en-us/azure/azure-monitor/app/opencensus-python#logs) doesn't work. I've copied it as-is to my workspace, I have pip-installed the imports, but the log messages don't show up in Application Insight. I noticed this behaviour in my real application. Caught exceptions are showing up in the Exceptions table, but using something like logger.warning("message") or logger.info("message2") don't show up in the traces table.

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

ID: fe78588d-6ed5-52b1-7b02-6e53affb000d
Version Independent ID: df4cdd51-678e-7a57-ef23-59c48b623fdd
Content: Monitor Python applications with Azure Monitor (preview) - Azure Monitor
Content Source: articles/azure-monitor/app/opencensus-python.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @reyang
Microsoft Alias: reyang

@lzchen @ramthi

I have to apologies for jumping the gun, there was a bit longer delay in the messages coming through in AI then I was used to. The example as-is does seem to work. However, using logger.info() messages do not show up. The example on the page uses logger.warning(). How do I change the behavior so it also submits logger.info() or logger.debug() messages?

Obviously you would use logger.setLevel("DEBUG"). Time to go to bed for me..

@JarroVGIT Happy to hear that everything worked out!


Access to form.busy

Hello @cklmercer
I recently found this new project you have created here on github through a friend of mine.
But as i look through the source code i cannot see any way to get access to form.busy like laravel spark allows us to.
Can you impliment that?

It already exists.
Assuming you have one of the following two components..
Vue.component('example', {
    data() {
        return {
            form: new AppForm({
                'field': ''
            })
        }
    }
}

Vue.component('example', {
    forms: {
        form: {
            field: ''
        }
    }
}

You can access busy using
vm.form.busy

Ah, so I wasn't creating new Form objects with forms created from the forms option.
This should be fixed in versions 1.0.7-beta+


TAMAYA-332: Move beans.xml file into META-INF directory

This moves the beans.xml file into the standard (./src/main/resources/META-INF) directory for CDI bean discovery.

While I agree its in the wrong place, technically its not needed.  By definition any JAR with a CDI extension defined (https://github.com/apache/incubator-tamaya-extensions/blob/master/modules/microprofile/src/main/resources/META-INF/services/javax.enterprise.inject.spi.Extension) is always a bean archive.

In the long run we might remove the file, otherwise I accepted the PR to fix its location. Thanks.


Download raw data for videos while others are converting

vividl <IP_ADDRESS>
It seems inefficient for my system to be spending so much time and energy converting videos, when it could also be downloading the other videos in the queue in the meantime.  I would like all the videos to be downloading while converting happens in the background.  If space runs out on the hard drive, then at least all the raw data is downloaded, and then you pop up a message giving me options what to do.  Then, I can leave the internet connection and converting can still happen.

Yeah, you download a channel of 260 videos, you don't want to have to wait for each one to convert before the next one is downloaded.  You want the whole thing right away, and let it convert over night.


I'm afraid that's not possible. This footer is some extra data StackEdit needs to manage merges between different versions, collaborators discussions and file properties if any, sorry.

I'm afraid that's not possible. This footer is some extra data StackEdit needs to manage merges between different versions, collaborators discussions and file properties if any, sorry.
Originally posted by @benweet in https://github.com/benweet/stackedit/issues/1237#issuecomment-369080122

in the issue some other more elegant solutions are proposed, would you be so kind and review them?
maybe the idea with inclusion in the yamel data would be a good solution?
maybe reopen the old issue?


Did you steal my idea?

I'm not sure if you stole my idea or not, it's possible you came up with this on your own, but it seems like you copied my extension which I made about a month ago.
https://github.com/Explosion-Scratch/tab_searcher
Screenshots of mine:

There's plenty of spotlight-like products out there: Raycast, Alfred, SlapDash...
Besides, I have been making concepts for it since October last year, as you can see here in a tweet.
In any case, it's honestly a bit odd to make such a claim (especially without even bothering to do any digging), you could have emailed me if anything, the email is even in the README.

There's plenty of spotlight-like products out there: Raycast, Alfred, SlapDash...

True, I probably shouldn't have jumped to that conclusion

Besides, I have been making concepts for it since October last year, as you can see here in a tweet.

Oh, ok.

In any case, it's honestly a bit odd to make such a claim (especially without even bothering to do any digging), you could have emailed me if anything, the email is even in the README.

I feel like it was applicable to make an issue, I think it's best for things relating FOSS software to stay public.


Use flattenToAppURL instead of addAppURL in link widget

@giuliaghisini There was any reason for use addAppURL instead?

I guess the rationale here is that we want always to save "flat" links into the field... or else then the resolveuids won't work?
/cc @cekk @tisto


Custom Job for BIQ

Hi,
I would like to pull the below metrics from the past one month for all applications preset in a single controller.
of slow/Stalled transactions
of calls/min
% error rate
I tried to run the dexter today, but it's collecting too many metrics. My pc disk space got filled within a few hours. Could you please help me with the custom job JSON. (I don't need any other data)
Thanks,
Sravan

You can control which metrics are pulled via https://github.com/Appdynamics/AppDynamics.DEXTER/wiki/Metric-Sets specified via https://github.com/Appdynamics/AppDynamics.DEXTER/wiki/Job-File#metricsselectioncriteria.


Bug fix/7246_SVN folder related fixes

Please check failing unit test

Hi Jinendra,
I got following error message while checking the failing Unit test. How to resolve framework issue on local as well as on server?
<EMAIL_ADDRESS>Regards,
Aditi Jagtap
From: Jinendra Ghodke<EMAIL_ADDRESS>Sent: Friday, April 26, 2019 11:03 PM
To: Ginger-Automation/Ginger<EMAIL_ADDRESS>Cc: Aditi Jagtap<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Ginger-Automation/Ginger] Bug fix/7246_SVN folder related fixes (#849)
Please check failing unit test
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Ginger-Automation/Ginger/pull/849#issuecomment-487138595, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKVY2QGWOEUPYP3YRW7NVMLPSM4GDANCNFSM4HIWMDTA.
This email and the information contained herein is proprietary and confidential and subject to the Amdocs Email Terms of Service, which you may review at https://www.amdocs.com/about/email-terms-of-service https://www.amdocs.com/about/email-terms-of-service


Add option to output to STDOUT

It would be nice if we could also just have the tool emit its output to stdout, making it potentially easier to compose it with other unix tools.
One way to implement this is to do this by default if neither --outDir nor --outFile is specified. In the Javascript API we probably want a third option after file and dir, perhaps stream ?
(low priority for me)

Sure, that makes sense for the CLI options. For the JS API, you mean { stream: fs.WriteStream; } ?
Personally I have no use for this feature right now but I don't see any harm in adding it.

Yeah, mainly thinking about this for the CLI case!


removed format check from index test

This addresses problem no.3 in #1192 - removing a data frame format check from the test on "indices are equal" which is off-topic in the test, and causes all combined uni/multivariate forecasters (with scitype:y = both) to fail.

@GuzalBulatova, would suggest you try merge into your PR (or a new branch based on your PR) and test if this resolves class no.3 of test failures.


tfrecords should also include depth and format

https://github.com/ahundt/tf-image-segmentation/blob/ahundt-keras/tf_image_segmentation/utils/tf_records.py
I believe this TFRecord format currently has implicit information including:

number of channels
channel order (bgr vs rgb)

Would it be okay to update this to include the relevant information?

this can solve OutOfRangeError (see above for traceback): RandomShuffleQueue '_2_shuffle_batch/random_shuffle_queue' is closed and has insufficient elements (requested 1, current size 0)

Trying to reproduce this error, it did not happen again but reproduced this error:
FIFOQueue '_1_processed_queue' is closed and has insufficient elements (requested 7, current size 0)
which I fixed by changing the line:
filename_queue = tf.train.string_input_producer([tfrecords_filename], num_epochs=num_epochs)
to
filename_queue = tf.train.string_input_producer([tfrecords_filename])
In the tensorflow document for tf.train.string_input_producer, it says:
num_epochs: .... If not specified, string_input_producer can cycle through the strings in string_tensor an unlimited number of times.
That fixed my issue since I did not necessarily have this error in the first round; but very randomly on the subsequent epochs!


Build error with octotiger and hpx commit e657426d705dd2292109ff18e271ccaa5339f7ef

I get the following error when building octotiger on queenbee:
https://gist.github.com/dmarce1/de23f7f02f3fb4c63815b05d0b4a07c8
Here is the script I used to build it:
https://gist.github.com/dmarce1/a729adb59de81201ec4566db97e3419a

External boost/cache moved to hpx/util/cache in #2096

The cache has been moved and the addressing_service.hpp does not refer to the old position anymore. Please check that you have a consistent checkout.

You are right, somehow I had an HPX installation where I didn't expect it.


MOD design review

@mmontesanonyc A few more design adjustments in this PR. Thanks!

Actually I'm going to close this in favor of another one in just a bit.


logging: syst: add support for catalog messages

MIPI Sys-T catalog messages are similar to dictionary logging where an ID is emitted instead of the whole format string to minimize log data. This adds the necessary bits to determine when to emit catalog messages.
This also changes how the collateral file for dictionary logging is created.
See individual commits for details.

Failure in bluetooth tests is due to https://github.com/zephyrproject-rtos/zephyr/issues/42608

This won't work in C++... so need to start over.


startChat not respecting name and email on Android

Hi, we have a app that is being built for iOS and Android, on iOS I've been able to set user name and email so I can identify him on Zendesk chat dashboard, but on android the user is always handled as a Visitor.
Been calling startChat like:
        onPress: () =>
          ZendeskChat.startChat({
            department: 'App',
            localizedDismissButtonTitle: 'Voltar',
            name: 'User Name',
            email<EMAIL_ADDRESS>          })

The department is correctly selected, but on Android the user is always a visitor.

It looks like you have the right code @diogoviannaaraujo -- I don't know what's going on?

If the builder properly has your data in it, then I'd check with Zendesk Support to see what the problem might be.

Ill be looking on that tomorrow, i had once or twice seen on iOS the bot responding with a visitor name on greeting and then changing to correct name, so maybe its a zendesk glitch.

Looks like a bug in Android SDK, preform is overriding visitorInfo settings https://develop.zendesk.com/hc/en-us/community/posts/360044257813/comments/360011630833

@jakmak24 that sounds like exactly the root cause.
@diogoviannaaraujo -- do you still need this ticket, since the issue is a difference in behavior in Zendesk's libraries?

Hey, no need to. Will try to get around it then.

@diogoviannaaraujo this was I had put in our app:
			preChatFormOptions: {
				name: !name ? "required" : "optional", // Notice this line -- you could use "none" instead!
				email: "optional",
				phone: "optional",
				department: "required",
			},

Additionally, you can disable the preChatForm in behaviorFlags if you detect you don't need the Form.
Please let me know if you need further help!

I know this is closed, but I'm experiencing same issue with v0.4.0 of the library. RN0.60.4.
Setting like this, first the preform bot shows up and ask name, email and phone. Only then in dashboard the info shows up.
How do I preset the details from login info in my app? I did like this but even explicitly presetting won't make it work...
       onPress: () =>
          ZendeskChat.startChat({
            department: 'appname',
            localizedDismissButtonTitle: 'Dismiss',
            name: 'user',
            email<EMAIL_ADDRESS>            phone: '12345',
          });

Using 0.4.1-beta.4 fixed this. This will be merged into v0.4.1 soon right? Thanks for the great libary!

Indeed, the the beta has passed QA at a couple companies, and we'll probably ship it in production in our next release.
I don't know when I'll get a chance to do the full official npm release without the beta tag though!


Ejected CLI Feedback

The aim of this seed is to have as clean an interaction and integration with the angular CLI as possible. Currently with native modules, I see no other way to handle those without ejecting the CLI, which is something I did not want to do. I'm opening this issue to get everyone's feedback as to the direction to take the seed to remain CLI friendly.
What your thoughts on ejecting the CLI?
Suggest an alternative to ejection?

Closed, alternative found.


added few books to competitve programming and Algorithms. Also, added…

can someone please tell me why it is showing 'Review required' and 'merging is blocked'.
Is it normal or i did something wrong, i am complete newbie.

@vicky002 verify and Approve this pull request


Forms are not styled

In many frameworks, you render forms from a view. For example, in Django you often write something like: {{ login_form.as_p }} and it injects the relevant fields into the template from your view.
I noticed that when using Bootstrap, you have to specify classes like "form-control" for each text input, to get the necessary basic style. Why wouldn't you apply some class to the form and then have the CSS style the form in a sensible way? It makes everything very repetitive. This is also true for buttons.
Doesn't this make everything crazily repetitive for people writing websites?
P.S. My current hacky workaround is to modify the form itself injecting the class, however this is a gross mixing of content and presentation. That is why I think Bootstrap would be smarter to have a class that styles the contents in a sensible way, instead of having a handful of classes that you apply to each and every mother-loving input.

This is how we've built our forms for years now, and it's unlikely to change. You'll have to add classes to the inputs, buttons, tables, modals, navs, etc as you need them.

Ah, 'we've always done it that way.' That is an avowal of a bad conscience.

@schmitgreg it's not. If anything, its your task as developer to know your tools and how to make them work together.
Django 1.11 will (finally!) ship with template-based form rendering, allowing you to setup bootstrap templates. Thats weeks away from landing.
In meantime you may use alternative form rendering approach, either using ready solution like crispyforms or floppyforms (both support Bootstrap 3 form markup out of the box), or by devising custom rendering strategy. Bulk of work can be achieved via custom templates mixed filters for introspecting otherwhise hidden attributes from form fields (like widget's type and backend specified props).

The whole philosophy of Bootstrap is that you opt into things. It sets a few basic normalisations/styles, and everything else requires you as the author to opt into them with relevant classes. This allows you to easily mix and match your own custom styles if you don't want to opt into specific ones for certain cases.

After I left that I started thinking about the consequences of having Bootstrap put in more active styles to all general form elements (not just clearing silly browser defaults, but really putting in active styles), and I see how this paradigm would end up requiring me to use things like "remove-bootstrap-annoying-default" classes for certain scenarios, which could be just as annoying. So I think I am convinced of this philosophy of opt-in.


functional simulation tests to enable external distribution classes

Continuing from this discussion:
https://github.com/econ-ark/HARK/pull/1104#issuecomment-1017620775
I believe that @llorracc is keen on replacing as much of the distribution modeling functionality in HARK with an external library as soon as possible.
A major obstacle to doing this is that the tests for model simulations currently mostly check for near equality with specific numerical values, as opposed to testing for a functional relationship between a model's state value and prior relevant state values.
Fixing this is just a matter of labor time.
This would make it much easier to work on #611 #949

Though it's not entirely clear from the documentation, I've tested it and this scipy.stats.rv_discrete class is more full-featured that our current DiscreteDistribution class -- including CDF, entropy, moment, and expected value computation -- with an initialization parameter, values, that mirrors the way we are currently constructing DiscreteDistributions.
https://docs.scipy.org/doc/scipy/reference/generated/scipy.stats.rv_discrete.html

Actually, I've spent a little time in the last couple of days looking into this and am glad to report that I think maybe the path forward is getting clearer, because sympy has recently put some effort into creating some tools that I hope we can build on for our purposes.  Encouragingly, they started with what Mathematica had done, but seem to have ended up building something somewhat simpler, which may be subject to future improvements.  There is also an excellent treatment of the topic on the quantecon site, https://python-advanced.quantecon.org/stationary_densities.html.
I also came across this recent paper which claims to have made some fundamental advances in the efficiency of simulating Markov processes.
One thing that is somewhat confusing about this whole area is where exactly the boundaries lie between Markov chains and optimal control problems.  The math-stats people seem mainly to be interested in the question "give me some fixed Markov process, and I'll tell you all sorts of things about its consequences."  Optimal control people (and, mostly, economists) are interested in the question, "tell me the environment someone is in, and I want to figure out the optimal behavior."  Once optimal behavior has been figured out, maybe that gives you some fixed coefficients that can then be represented in a Markov chain, but the Markov chain part of it is in a sense easier than the 'find the optimal behavior' part.  Still, the common element to all of them is that we need to find a robust way to state what the environment is and how it evolves in response to exogenous and endogenous variables; maybe the new sympy objects will be useful for that.

An example of the kind of test that needs to be changed:
https://github.com/econ-ark/HARK/blob/master/HARK/ConsumptionSaving/tests/test_IndShockConsumerType.py#L92
And example of the kind of test it should be changed into:
https://github.com/econ-ark/HARK/blob/master/HARK/ConsumptionSaving/tests/test_ConsPortfolioFrameModel.py#L153-L156

I have no objection to ADDITIONAL tests like https://github.com/econ-ark/HARK/blob/master/HARK/ConsumptionSaving/tests/test_ConsPortfolioFrameModel.py#L153-L156
And if the reason you think the first test should be eliminated is that it tests the outcome for a specific agent (at index 1), I'm on board.  Instead, what we should be testing is things like whether the average level of assets in simulations matches some raw number like 0.184 (or whatever the correct answer is).
Any time a change in the HARK code causes our simulations to produce significantly different quantitative answers, we need to be alerted to that because we know that our current code produces the right answers and any substantial change in those answers means the HARK change introduced a conceptual error somewhere (even if accounting identities remain satisfied).

Yes, the simulation tests I'm referring to are those that test a specific value for a specific agent.
We also have tests for the solution objects, which should not break with a change in how distributions are sampled. I.e. that the consumption function, for some amount of market resources, is 0.184. So these can go untouched.
I don't recall if we have any tests for the approximate equality of a simulated population mean to a fixed number. I would be wary of such tests, because the sensitivity of the equality approximation would need to be calibrated to the confidence interval of the result somehow to be effective if the only change was to distribution sampling.

I don't recall if we have any tests for the approximate equality of a simulated population mean to a fixed number. I would be wary of such tests, because the sensitivity of the equality approximation would need to be calibrated to the confidence interval of the result somehow to be effective if the only change was to distribution sampling.

Any such tests should be low precision ones.  If an aggregate variable changed from 3.14 to 3.15 or 3.13 we would not worry.  But if it changed to 6.8 or something, that would be a clear sign of a bug.

I'm taking responsibility for this issue, but am a little stuck.
One thing we could do is remove any tests for particular values that come out of a simulation's results.
In many cases, this would leave simulations as entirely untested except to show that they executed without error.
I wonder if there's anything else that comes to mind for what would be a good simulation test.
(The tests of a particular transition function are one idea, though writing these will be a cumbersome process.)


Created sv.json

Swedish translation.

Thank you very much for your contribution. Will merge this probably later today! 👍

@Grottmastaren Thanks again for your contribution! May I ask you to provide me with (if available) licensing texts for the official Swedish Fate publishers (if there is one)? We're providing this information for most of our translations, you can check the format here: https://github.com/anvil-vtt/FateX#translations
If there is no official swedish Fate version then you don't need to provide anything and only you will be named as the contributor :)


Stocks screening gets stuck at 660 or 39%

When screening all stocks in NSE(1673) after several reruns, the counter is stuck at 660 stocks or 39%.
To Reproduce

This issue happening while running compiled .run file or you have installed the dependencies and executing the python script?

Hi Pranjal,
I download the source code, installed the dependencies and executed the script.
I ran it again now and observed the same issue, stuck at 660(39%).

Option 0 works fine for one scrip at a time.
Can rule out the 2nd possibility if it works fine on your setup, provided we would be hitting the same yahoo fin instance.

Works fine at my end.
What is your geographical region? I don't know whether Yahoo finance library handles it by different instances by geography or not.
Also monitor the amount of free memory your device having during execution. Check whether it overflows.
Lastly, try using different network than the one you're currently using.

I am based in Bangalore, India. I already checked consumption and available resource, everything is fine on that front.
No worries though, I manually downloaded a list of nifty500 from NSE and created a watchlist to read from so don't need to screen all .
Thank you so much for the follow-up!!

Good to know that its giving you desired results.
However, if possible, try to switch your network once and run a full screening to ensure that this is not a software issue.

Sir, Very Slow Scanning also stuck at 14%

Please try screening after sometime as the Yahoo finance API may goes under maintenance during the aftermarket hours.

You can try again with latest version and keep me posted!

Dear Sir,
I am unable directly open trading view charts by clicking stock name on
windows 10 & 11
Plz send snapshots to install OS/Terminal supports unicode.
Regards,
Mohan
On Sun, Jan 16, 2022 at 10:15 PM Pranjal Joshi @.***>
wrote:

You can try again with latest
https://github.com/pranjal-joshi/Screeni-py/releases/latest version and
keep me posted!
—
Reply to this email directly, view it on GitHub
https://github.com/pranjal-joshi/Screeni-py/issues/58#issuecomment-1013909767,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI65DQG5NSUHTVDBVKXVTH3UWLYZJANCNFSM46RS3XEQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

@mhn0497 Please download Windows Terminal from Microsoft store for free and run screenipy.exe through Windows terminal to get the Unicode font and clickable TradingView links working.

Dear Sir,
Thanks for your valuable time to reply, Sir Any Powerful Screener  only
using ADX Indicator strategy to find stocks potential breakout probability.
Plz provide any other stock market related screeners if you have.
Regards,
Mohan
On Sun, Jan 23, 2022 at 9:46 AM Pranjal Joshi @.***>
wrote:

@mhn0497 https://github.com/mhn0497 Please download Windows Terminal
from Microsoft store for free and run screenipy.exe through Windows
terminal to get the Unicode font and clickable TradingView links working.
—
Reply to this email directly, view it on GitHub
https://github.com/pranjal-joshi/Screeni-py/issues/58#issuecomment-1019410878,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI65DQF3HCP2S5CVHLN25TDUXN6JDANCNFSM46RS3XEQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>


Bug report - floating point numbers (1.16.5)

Hi,
there's an issue with the overlay when playing on blocks far away from the spawn. I believe it's a floating point number accuracy issue - if you just layer each X next to another from 0, it might be hard to calculate for the game at such faraway places.
I hope you can fix this. I'm playing on 1.16.5 and discovered the issue there. Maybe it also appears on other versions, feel free to add where else it occurs.
I recorded the outcome in-game plus my journeymap coordinates on my map. Try with comparable coordinates. I managed to get this by setting the spawn radius to 1.000.000 blocks and let myself respawn few times.
https://user-images.githubusercontent.com/106652912/195215328-fd7df94d-c710-4a60-a575-a0620148a312.mp4

I've fixed this issue in the next release for Minecraft 1.20 that will be available soon. I'll also release the fix for 1.16.5, 1.18.2, 1.19.2 and 1.19.4 soon.


C++ JSON/GraphQL Hex Support

Right now C++ JSON and GraphQL use hex representation for vector<8-bit types>. We were unable to reproduce this behavior on the Rust side of things using serde_json, so we created a hex wrapper type instead. This creates a portability issue since the C++ side doesn't have a type which uses fracpack's vector<8-bit type> representation tied with JSON's array-of-unsigned-8-bit and array-of-signed-8-bit representations.

Option 1: Drop vector<8-bit type> JSON specializations and create a hex wrapper type (historically called bytes, but we don't have to use that name).
Option 2: create a wrapper type which forces array-of-number (both signed and unsigned cases)

Either option depends on #272 unless knowledge of the specific wrappers is added to Fracpack's internal machinery (many places).

Instead of a wrapper, there's also the option of using std::vector<std::byte> for hex.

That seems like the better option


[www/nginx] proxy buffer size

I kindly ask to add some parameters in nginx location, related to proxy buffers
Default of 4k is not enough when you have big headers, for example if CSP is enabled with a lot of allowed hash
proxy_buffer_size          128k;
proxy_buffers              4 256k;
proxy_busy_buffers_size    256k;

please try this patch:
opnsense-patch -c plugins 8b66adcc2f683436243be5bcfee63c9a3595eb35
I wait for feedback for the final merge. If it is ok, I it will be in 1.16.

Hi Fabian,
looks great! And it works fine :)
Thank you so much
Cheers
Attilio
On Sun, Nov 10, 2019 at 5:14 PM Fabian Franz BSc<EMAIL_ADDRESS>wrote:

please try this patch:
opnsense-patch -c plugins 8b66adc
https://github.com/opnsense/plugins/commit/8b66adcc2f683436243be5bcfee63c9a3595eb35
I wait for feedback for the final merge. If it is ok, I it will be in 1.16.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/opnsense/plugins/issues/1561?email_source=notifications&email_token=AHKLKZKUX7YH5K4VCNABOS3QTAXPFA5CNFSM4JGUTYAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDVAW2Y#issuecomment-552209259,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHKLKZMIMMU44DH6AU3KAVLQTAXPFANCNFSM4JGUTYAA
.

@siga1975 the validation was hard to implement but it seems to be very failsafe now (except if I missed some size checking which is not documented).

for the moment I just checked with the values I had previously inserted
manually, and did a diff of the 2 conf files, then restarted the
application and checked a bit on my site, all is working good
but I didn't do a full test trying for invalid values and so...
On Sun, Nov 10, 2019 at 5:45 PM Fabian Franz BSc<EMAIL_ADDRESS>wrote:

@siga1975 https://github.com/siga1975 the validation was hard to
implement but it seems to be very failsafe now (except if I missed some
size checking which is not documented).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opnsense/plugins/issues/1561?email_source=notifications&email_token=AHKLKZOJKID5TGC4E6OBGMDQTA3D3A5CNFSM4JGUTYAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDVBKPY#issuecomment-552211775,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHKLKZLKVN754WUNEJD7S4DQTA3D3ANCNFSM4JGUTYAA
.

Will be now in plugin release 1.16.


Change if-expression conditions to allow non-boolean values

Many languages allow for non boolean values in the conditions of if statements / expressions.
For instance, in python, one can do this:
my_list = []

if my_list:
    print("only happens when my_list is populated")

This works because an empty list in python is falsey. Similarly, the number 0 is falsey, along with empty strings, None values and others.
In Rust, I find myself doing things like:
let my_vec: Vec<i32> = some_vec_returning_function();

if !my_vec.is_empty() {
    println!("Got some results");
}

I think it would be nice to be able to do something like:
let my_vec: Vec<i32> = some_vec_returning_function();

if my_vec {
    println!("Got some results")
}

I'm wondering if this is something that others feel would be a nice addition to the language? I have an idea as to how it might work (using traits) and I'd like to write the RFC for this, but wanted to get a feel for whether it is wanted before starting on it.
Does anyone else see merit in this?

Not a fan, personally. I'd rather see something explicit like if my_vec.is_empty() than just if my_vec.

I am against it also. Sometimes I want it, but then I remember all disadvantages:

Ambiguous
Error-prone. You can misspell something and it will be evaluated without error.

ifs are for bools, not anything else.

Until someone decides that the truthiness of Option should be as follows:
impl<T: Truth> Truth for Option<T> {
    fn is_true(&self) -> bool {
        self.as_ref().map(|x| x.is_true()).unwrap_or(false)
    }
}

And someone else decides that Date should be false at midnight and we end up with this.
This is a case where I feel that being explicit is really important.

I feel this wouldn't be an improvement.  The code becomes less explicit, and having objects like collections be "truthy" often leads to sloppy programming where the meaning of "true" is confused and edge cases aren't properly handled.  The C-style "nonzero integers are true", on the other hand, is more reasonable and makes dealing with bitmasks easier; but I think Rust's decision to only support 'bool' in conditionals is possibly the best choice given that they had the opportunity to make 'bool' strongly typed.

As stated this is a bad idea.  if let already exists and provides the form of this that is reasonable.

Fair points everyone. Thanks for the feedback!


chore: fix filmstrip tracer test

fix #214
close all open file handle and close the browser session correctly.

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Build Cause: Pull request #218 opened

Start Time: 2021-02-08T23:06:06.743+0000

Duration: 15 min 26 sec

Commit: 4d729f5401421b4849dde6b1d70ba2d2abd3a45c

Test stats :test_tube:

Test
Results

Failed
0

Passed
59

Skipped
7

Total
66

Trends :test_tube:


[decode] blank penalty

The "blankpenalty" can mitigate deletion errors and is highly effective when the number of deletion errors in the recognition results is close to or even exceeds the number of substitution errors.
results on wenetspeech (u2pp trained):

稍等我顺带修改下readme中的结果

unit test fix by https://github.com/wenet-e2e/wenet/pull/2277


Grab bag of small GitHub API metric improvements

Four small things:

Send method and status tags to Datadog (e.g., method:GET and  status:200)
Add method, path, and status to log message (plus add tests)
Remove unnecessary dyno global tag.

Some sample metrics:
github-request:647|h|#path:/installation/repositories,method:GET,status:200
github-request:704|h|#path:/graphql,method:POST,status:200
github-request:528|h|#path:/repos/_owner/_repo/issues/_number,method:PATCH,status:200

See commit messages for specifics.

let's be aware of how much we are increasing our usage of tags though

💯 Great point. This is top of mind. Once it goes out, I'll monitor the cardinality and ensure it doesn't grow too large.


save model issue

There's an error when i tried to train the model on my dataset.
Traceback (most recent call last):
File "train.py", line 29, in
auto_device=True  # Auto choose CUDA or CPU
File "/home/hsz/.conda/envs/win_env/lib/python3.6/site-packages/pyabsa/functional.py", line 157, in train_atepc
model_path.append(train4atepc(t_config))
File "/home/hsz/.conda/envs/win_env/lib/python3.6/site-packages/pyabsa/tasks/atepc/training/atepc_trainer.py", line 382, in train4atepc
return trainer.train()
File "/home/hsz/.conda/envs/win_env/lib/python3.6/site-packages/pyabsa/tasks/atepc/training/atepc_trainer.py", line 204, in train
self._save_model(self.opt, self.model, save_path, mode=0)
File "/home/hsz/.conda/envs/win_env/lib/python3.6/site-packages/pyabsa/tasks/atepc/training/atepc_trainer.py", line 354, in _save_model
torch.save(model_to_save.cpu(), save_path + args_to_save.model_name + '.model')  # save the whole model
File "/home/hsz/.conda/envs/win_env/lib/python3.6/site-packages/torch/serialization.py", line 379, in save
_save(obj, opened_zipfile, pickle_module, pickle_protocol)
File "/home/hsz/.conda/envs/win_env/lib/python3.6/site-packages/torch/serialization.py", line 484, in _save
pickler.dump(obj)
TypeError: can't pickle _thread.RLock objects
Seems that one or some objects in self.model that cannot be dumped by pickle.
But if I change the saving mode to any value but 0, the _save_model function works well. So i guess the difference could be the type of model that i saved, which means 'pytorch.bin' can be generated but '.model' file cannot.
Could you help me out, thx

Hi,
I noticed this issue report in the early, however I test the code on Windows and CentOS, but cannot reproduce this problem. I suppose the reason is the version of pickle. I will fix this problem as soon as I can. Please watch the new release on PyPi.

Great! Appreciate

Great! Appreciate

As a suggestion, you can uncomment https://github.com/yangheng95/PyABSA/blob/97b8836eeb26a9d66a608969fd04aa308e849dcd/pyabsa/tasks/atepc/training/atepc_trainer.py#L352 , to save the state_dict only (comment L354 accordingly). If this works, you need to download BERT-BASE model from hugginface (if you didnt download before, which is hard for poor network.)

The model loading code is compatible with the model saved in state_dict mode. Please let me know if this works or not, many thanks.

The model loading code is compatible with the model saved in state_dict mode. Please let me know if this works or not, many thanks.

Got it. I'll try it later and send you the feedback. Thank you for the quick reply and solutions.

 Can you export the package list to help reproduce the problem?

 Can you export the package list to help reproduce the problem?

 Can you export the package list to help reproduce the problem?

I cant reproduce the problem, still. However, I found some answers saying it may be caused by locked logger while saving model. I revise some code about logger in V<IP_ADDRESS>，please help me review if the code works.

According to a test report, this problem occurs on some situations has been solved in <IP_ADDRESS> . Thanks for your help.

According to a test report, this problem occurs on some situations has been solved in <IP_ADDRESS> . Thanks for your help.

It works perfect. Brilliant!


⚠ Avatar API Endpoint has degraded performance

In 15fabdd, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 5104 ms

Resolved: Avatar API Endpoint performance has improved in e40e35e after 25 minutes.


addRow sends malformed requests

I have inserted console.log(data_xml) in index.js after line 323.
When I run
worksheet.addRow({
  'data:with>xml<lol': 'test',
})

I can see from the console that it creates the following element
<gsx:data:with>xml<lol>test</gsx:data:with>xml<lol>

Which then causes a bad request to trigger in Google's API which I can see in the error that addRow throws.

How do I use trim function in query to read data from spread sheet?
please comment here if you know answer.

I've just a published a new version of this module which is a complete rewrite using Google's V4 Sheets API (v3 is being deprecated on March 3, 2020).
Docs available here - https://theoephraim.github.io/node-google-spreadsheet
The new API is json based so you can put whatever xml stuff you want in there.
Please test it out and if you find that there is still a problem, reopen the issue.
Thanks! 😄


StructBlock - get_prep_value to return OrderedDict

get_prep_value should return an OrderedDict so we keep the order for the fields inside the StructBlock.
This is needed for the rewriting of Streamfield in React.

Hi @vincentaudebert,
This seems like a bad idea to me. get_prep_value exists primarily to provide a JSON-compatible representation of the streamfield data, as opposed to the 'native python' representation which might contain model instances, datetimes and so on - and OrderedDicts don't survive the round trip to JSON and back (JSON objects are defined as unordered, as per http://json.org/). Once one component starts to rely on key order being preserved, then all other components either need to preserve ordering everywhere (which isn't possible using standard JSON tools), or keep track of which values are ordered and which ones aren't (which will be a nightmare to maintain).
Is there any reason you can't obtain the ordering from the StructBlock definition? (Presumably you must have access to the StructBlock definition on the React side, or else you wouldn't know what field types to render.) The standard pattern for iterating over a StructBlock value is:
for (block_name, block_def) in my_structblock_definition.blocks:
    do_something_with(my_structblock_value[block_name], block_def)

@gasman The idea was to be able to get the right order then I have to run a loop again to switch everything to a list of objects with type/value.
Order in a list are preserved in JSON.
If it helps to understand here is my code:
        value = self.bound_field.value()
        json_value = self.block_def.get_prep_value(value)

        # this is to keep the order of children on complex blocks
        for obj in json_value:
            if isinstance(obj['value'], dict):
                to_arr = []
                for key in obj['value']:
                    to_arr.append({
                        'type': key,
                        'value': obj['value'][key]
                    })
                obj['value'] = to_arr

        return json.dumps(json_value, cls=DjangoJSONEncoder)

Having it on the StructBlock class avoids to add a new get_prep_value to all the children.
On a side note, the data structure is really not consistent. Sometimes blocks are given as dict (which is a problem for the order) and sometimes it's a list of dicts.
Quite confusing :)
I will try to apply directly your loop :)
I just thought it would be nice to have the possibility to get it directly from the object.


Encoding issue with GroupSession

Describe the bug
I believe that there is an encoding issue with the GroupSession messages.
To Reproduce
Steps to reproduce the behavior:
fn test(payload: &str) -> anyhow::Result<()> {
    let outbound = vodozemac::megolm::GroupSession::new();

    let json = serde_json::to_string(&outbound.pickle())?;
    let mut outbound: vodozemac::megolm::GroupSession =
        serde_json::from_str::<vodozemac::megolm::GroupSessionPickle>(&json)?.into();

    let session_key = outbound.session_key().to_base64();
    let inbound = olm_rs::inbound_group_session::OlmInboundGroupSession::new(&session_key)?;

    let mut i = 10;
    let msg = loop {
        let msg = outbound.encrypt(payload).to_bytes();

        i -= 1;
        if i == 0 {
            break base64::encode(&msg);
        }
    };

    let _msg = inbound.decrypt(msg)?;

    //assert_eq!(payload, msg.0);
    //println!("Message: {msg:?}");

    Ok(())
}

fn main() {
    for i in 0..256 {
        let payload = "0".repeat(i);
        println!("{i}: {}", if test(&payload).is_ok() { "Ok" } else { "Err" });
    }
}

Expected behavior
All messages decode properly.
Screenshots

Log output
0: Ok
1: Ok
2: Ok
3: Ok
4: Ok
5: Ok
6: Ok
7: Ok
8: Ok
9: Ok
10: Ok
11: Ok
12: Ok
13: Ok
14: Ok
15: Ok
16: Err
17: Err
18: Err
19: Err
20: Err
21: Err
22: Err
23: Err
24: Err
25: Err
26: Err
27: Err
28: Err
29: Err
30: Err
31: Err
32: Err
33: Err
34: Err
35: Err
36: Err
37: Err
38: Err
39: Err
40: Err
41: Err
42: Err
43: Err
44: Err
45: Err
46: Err
47: Err
48: Ok
49: Ok
50: Ok
51: Ok
52: Ok
53: Ok
54: Ok
55: Ok
56: Ok
57: Ok
58: Ok
59: Ok
60: Ok
61: Ok
62: Ok
63: Ok
64: Err
65: Err
66: Err
67: Err
68: Err
69: Err
70: Err
71: Err
72: Err
73: Err
74: Err
75: Err
76: Err
77: Err
78: Err
79: Err
80: Err
81: Err
82: Err
83: Err
84: Err
85: Err
86: Err
87: Err
88: Err
89: Err
90: Err
91: Err
92: Err
93: Err
94: Err
95: Err
96: Ok
97: Ok
98: Ok
99: Ok
100: Ok
101: Ok
102: Ok
103: Ok
104: Ok
105: Ok
106: Ok
107: Ok
108: Ok
109: Ok
110: Ok
111: Ok
112: Err
113: Err
114: Err
115: Err
116: Err
117: Err
118: Err
119: Err
120: Err
121: Err
122: Err
123: Err
124: Err
125: Err
126: Err
127: Err
128: Ok
129: Ok
130: Ok
131: Ok
132: Ok
133: Ok
134: Ok
135: Ok
136: Ok
137: Ok
138: Ok
139: Ok
140: Ok
141: Ok
142: Ok
143: Ok
144: Err
145: Err
146: Err
147: Err
148: Err
149: Err
150: Err
151: Err
152: Err
153: Err
154: Err
155: Err
156: Err
157: Err
158: Err
159: Err
160: Err
161: Err
162: Err
163: Err
164: Err
165: Err
166: Err
167: Err
168: Err
169: Err
170: Err
171: Err
172: Err
173: Err
174: Err
175: Err
176: Ok
177: Ok
178: Ok
179: Ok
180: Ok
181: Ok
182: Ok
183: Ok
184: Ok
185: Ok
186: Ok
187: Ok
188: Ok
189: Ok
190: Ok
191: Ok
192: Err
193: Err
194: Err
195: Err
196: Err
197: Err
198: Err
199: Err
200: Err
201: Err
202: Err
203: Err
204: Err
205: Err
206: Err
207: Err
208: Err
209: Err
210: Err
211: Err
212: Err
213: Err
214: Err
215: Err
216: Err
217: Err
218: Err
219: Err
220: Err
221: Err
222: Err
223: Err
224: Ok
225: Ok
226: Ok
227: Ok
228: Ok
229: Ok
230: Ok
231: Ok
232: Ok
233: Ok
234: Ok
235: Ok
236: Ok
237: Ok
238: Ok
239: Ok
240: Err
241: Err
242: Err
243: Err
244: Err
245: Err
246: Err
247: Err
248: Err
249: Err
250: Err
251: Err
252: Err
253: Err
254: Err
255: Err

Additional context
The reverse test (encoding with libolm and decoding with vodozemac) works all the time.

You seem to be using your own base64 encoding method, does this happen with the to_base64() method that MegolmMessage has as well?
You'll need to configure your base64 encoder to use unpadded base64 for libolm to be able to decode it.

@poljar yea, I dug a bit deeper into that after lunch :) … and came to the same result. Thanks for taking a look!
I also did try this out with a matrix of encoding/decoder x volozemac/olm. Volozemac actually decodes base64 just fine, in any case. While libolm fails in weird ways when the format is not right.
I shouldn't have expected that a pure Rust implementation fails like this :grin:
So if there is a bug, then I guess it is with libolm, and not with vodozemac :) Sorry for the noise.


Spring 3.4.0 compatibility broken

Search first

[x] I searched and no similar issues were found

Description
The class org.springframework.boot.autoconfigure.liquibase.LiquibaseProperties in spring 3.4.0 returns lists for the getContexts() and getLabelFilter(), but the class liquibase.integration.spring.SpringLiquibase does not support this yet (still using simple String). This breaks the spring boot compatibility and integration
Steps To Reproduce
Use Spring-Boot dependency org.springframework.boot:spring-boot-autoconfigure:3.4.0 and liquibase-core:4.30.0
Expected/Desired Behavior
Methods should be compatible
Liquibase Version
4.30.0
Database Vendor & Version
No response
Liquibase Integration
No response
Liquibase Extensions
No response
OS and/or Infrastructure Type/Provider
No response
Additional Context
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR (Thank you!)

This can be closed. It was an issue on my side.

what was the issue? did I understood correctly that need to use liquibase.setContexts(StringUtils.collectionToCommaDelimitedString(liquibaseProperties.getContexts())); ?


Add logging to Job objects

https://docs.google.com/document/d/1s1SOG2AjE7Gu63_rkMlJZDrI5j7xlsRUo_satdTk_3c/edit?usp=sharing

Lread:

Username
Timestamp
Objects Written
QC Params

Lwrite:

Username
Timestamp
Objects Written
QC Params

Lbug:

Username
Timestamp
Session Info (like what currently shows up in the bug report screen)

Please change "make_log_api_call" parameter to "flush" (for clarity)


 Is this still an issue? I see that the tests are passing now.

Yes this is still an issue. The tests are passing because I've pinned them to the older commit https://github.com/cloudfoundry/disaster-recovery-acceptance-tests/commit/de52f3ea5d2922240c1ef22aa3e61741ca0478f7.

Hey, I think this should be fixed by your cf having a go buildpack with 1.21 available. Looking at the logs of your failed runs, it seems that it is using go 1.20?

On the cf-deployment "develop" branch, we currently have go-buildpack v1.10.15:
https://github.com/cloudfoundry/go-buildpack/releases/tag/v1.10.15
This buildpack supports Go 1.20.12 and 1.21.5. It seems to chose 1.20 as default, however. If you now use go toolchains, you have to configure 1.21.

but we do?
https://github.com/cloudfoundry/disaster-recovery-acceptance-tests/blob/main/fixtures/credhub-test-app/go.mod#L3

https://github.com/cloudfoundry/go-buildpack/issues/344
This is something that the go buildpack needs to properly deal with.

This could be fixed by the go-buildpack. However, looking at the issue https://github.com/cloudfoundry/go-buildpack/issues/344 there has been no activity in months and so I would not expect a fix to be released soon. To unblock the cf-deployment, can you specify the required Go version in your test app's manifest? There's not much documentation, but this could work:
---
applications:
  - name: my-app-name
    env:
      GOVERSION: go1.21

Yeah but that would hardcode the version via the manifest and require updating that every time a new go version releases.. I'm going to look at updating the actual tests to set the version from the go.mod file as staging vars.

The test is still failing:
https://concourse.wg-ard.ci.cloudfoundry.org/teams/main/pipelines/cf-deployment/jobs/bbr-run-drats/builds/917
I think you have to set the GOVERSION env variable before you start the app, so that the staging succeeds. This should be the right place:
https://github.com/cloudfoundry/disaster-recovery-acceptance-tests/blob/152026a98e65eeffa3b467ab825a01cda4c07d63/testcases/credhub_instance_creds_testcase.go#L67
I've tested that manually with cf set-env appCIgHVi6l-R_sjQ4mgDcfcQ GOVERSION go1.21 and it works.

Sorry for that oversight. I just pushed another commit to fix it in the 2nd cf push as well.

Now the tests fail in the new GetGoVersionFromGoModFile function:
https://concourse.wg-ard.ci.cloudfoundry.org/teams/main/pipelines/cf-deployment/jobs/bbr-run-drats/builds/922
  runtime error: invalid memory address or nil pointer dereference

  Full Stack Trace
    github.com/cloudfoundry/disaster-recovery-acceptance-tests/testcases.(*CfCredhubSSITestCase).GetGoVersionFromGoModFile(0xc0003a45b0?)
    	/tmp/build/b7784222/src/github.com/cloudfoundry/disaster-recovery-acceptance-tests/testcases/credhub_instance_creds_testcase.go:112 +0x75
    github.com/cloudfoundry/disaster-recovery-acceptance-tests/testcases.(*CfCredhubSSITestCase).BeforeBackup(_, {{{0xc00021c358, 0x2}, {0xc00002e980, 0x37}, {0xc00021c35a, 0x5}, {0xc000032ae0, 0x1e}, {0xc000034b58, ...}, ...}, ...})
    	/tmp/build/b7784222/src/github.com/cloudfoundry/disaster-recovery-acceptance-tests/testcases/credhub_instance_creds_testcase.go:66 +0x1b2
    github.com/cloudfoundry/disaster-recovery-acceptance-tests/runner.RunDisasterRecoveryAcceptanceTests.func2()
    	/tmp/build/b7784222/src/github.com/cloudfoundry/disaster-recovery-acceptance-tests/runner/runner.go:57 +0x17e

Can you please check?

Here goes attempt nr 3 :/
Sorry seems I misunderstood the modfile.Parse() docs..

Ok, now the test is green: https://concourse.wg-ard.ci.cloudfoundry.org/teams/main/pipelines/cf-deployment/jobs/bbr-run-drats/builds/923


Product for 0,01$ creates checkout error

The title says it, you can test it yourself here
For me clicking "Buy now" leads to 500 Internal Server Error

Thank you @maxschulmeister. Missed the minimum requirement here from Stripe and only had validation for the maximum. Added a minimum validation now to match Stripe's requirement of $0.5
https://github.com/polarsource/polar/pull/3961


WIP: 37 add test data ew

close #37

EW Added metadata
Updated test for Medatada (orbit attitude and gcp)

TODO: add  fake data

Codecov Report

Merging #40 (f865584) into main (e74f5bd) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #40      +/-   ##
==========================================
+ Coverage   93.39%   93.43%   +0.03%
==========================================
  Files           7        7
  Lines         530      533       +3
  Branches       60       60
==========================================
+ Hits          495      498       +3
  Misses         19       19
  Partials       16       16

Impacted Files
Coverage Δ

tests/test_xarray.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


How I can get the pre-trained model?

Hello:
Please remember that due to the Internet blocking in China, anything published in Baidu, people outside China can't use.
Please consider to publish them in google.
Thanks,

Hello! I believe I did upload the model in Google Drive. Please check the tabular.

Hello:
Thanks, I got the model.
How many langauges this model can use to produce video captioning? English and Chinese?

Only English for now. Because both the datasets do not support multi-language. I am planing to use another dataset(VATEX) but have difficulty downloading Youtube videos

Hello:
You can try this url to download  the dataset, but it may not work if you are in China.
But I am not sure if it is what you need.
https://www.kaggle.com/datasets/neverix/vatex-lowres/download

Thank you for the link! It is what I am looking for. I didn't know it is available in Kaggle. I will try it later.

You are welcome.
If you can make Chinese subtitle, let me know!


Using placeOnTile makes it easy to clip through walls for large sprite

For sprites that are 16x16 or larger, using placeOnTile will sometimes cause your sprite to be placed so that it overlaps with adjacent walls. This causes you to clip though the floor
Here's a game that demonstrates the issue with a 16x16 sprite. Run into the "bad" tile and it should teleport you up top. Roughly 50% of the time you will clip through the floor.
https://makecode.com/_PgJC7T2tR1HM
And here's the same code with a 17x17 sprite which clips 100% of the time:
https://makecode.com/_1su0wLWwJWLk
We should be smarter when placing a sprite on a tile to ensure it isn't colliding with any walls (if possible)

This bug was reported through the forum here: https://forum.makecode.com/t/sprite-in-continuous-fall/1237/11

@riknoll I am removing next-release as there are no taker.


Add initial test cases

Added first tests using TestStore.
Ref: https://github.com/tryswift/trySwiftTokyoApp/issues/28

I have fixed the pointed out issues.
One point of concern is that I named it 'mock' based on the sample code from TCA, but thinking about it, 'dummy' might be a better choice. What do you think?

One point of concern is that I named it 'mock' based on the sample code from TCA, but thinking about it, 'dummy' might be a better choice. What do you think?

Either is OK in this test.

Thank you for your review!


New website build

cc @billonahill
See http://lucperkins.github.io/heron/ for my fork of the site

Thanks @lucperkins.
As a follow-up we should probably clean up the layout of this page (e.g., bullets and indents):
http://lucperkins.github.io/heron/docs/getting-started-troubleshooting/


Use modifiers in number-variable

Overview
See https://github.com/lblod/ember-rdfa-editor/pull/962
this refactors the code to use some new modifiers defined in the editor repo.
connected issues and PRs:
https://github.com/lblod/ember-rdfa-editor/pull/962
Setup
npm link the editor repo and npm link @lblod/ember-rdfa-editor this repo. the ember serve this repo.
How to test/reproduce
add a number variable
make sure nothing of functionality changed for arrow keys (left/right) and pressing enter. Arrow keys should let you go out of the number variable, enter should set the cursor behind the number variable node.
Challenges/uncertainties

@x-m-el I changed the base branch of this PR to the update-editor branch as the 5.3.0 release of the editor is required for this PR to work.


fix typo in readme

do not review yet

Jenkins: test C semantics

Jenkins: test C semantics

Jenkins: test C semantics

@grosu can you review this quickly? it's basically just a typo fix that I took advantage of in order to test the Jenkins build infrastructure.

lgtm


RuntimeError in llava image encoding

When running llava 1.6 mistral 7b, i get this error:
RuntimeError in llava image encoding: The expanded size of the tensor (0) must match the existing size (2438) at non-singleton dimension 0.  Target sizes: [0, 4096].  Tensor sizes: [2438, 4096]
torch.Size([2758, 4096])
0 -1

Note the sizes 2438 and 2758 changes
This error happens randomly and is not specific to data.
Removing image input removes this error too.

Same bug raised by using gen(choices=[...]) with image input, as mentioned in #222 .

this bug is not related to using gen choices or not, but happens universally as long as i have image as input

Same bug raised by using gen(choices=[...]) with image input, as mentioned in #222 .

that's because when generating the  "logprob_start_len" variable used to get the start position of the choices, sglang does not add the necessary position shift when inserting image tokens into the input sequence.
https://github.com/sgl-project/sglang/blob/main/python/sglang/backend/runtime_endpoint.py#L217
For now, I add a temp fix to make it work as follows:
       # Compute logprob
        data = {
            "text": [s.text_ + c for c in choices],
            "sampling_params": {"max_new_tokens": 0},
            "return_logprob": True,
            "logprob_start_len": max(prompt_len - 1, 0),
            # should be prompt_len-1 here i think, otherwise the normed_logp will be wrong
            "return_text_in_logprobs": True,
        }
        self._add_images(s, data)

        if s.images_:  # only support one image
            # TODO: This is a very naive way to shift the logprob_start_len
            # maybe in future we should directly modify `prompt_tokens` variable
            # to take the added image tokens into account
            data["logprob_start_len"] += 576 - 1

Have you yet found out what exactly caused it and how to solve it. I do not use regex, thus, the gen choices should not be called.


Add Select All to DuplicatesContactFixFragment

Hello @Quadriyanney, The PR title says work-in-progress, do you want me to hold on on the review?

Also, please exclude local (.idea, etc) & generated folders from git.

The PR is ready for review now. Removed local files that were pushed with the first commit.

Welldone @Quadriyanney ...


Script is now able to be used for upgrading or initializing a new node + important bugfix

What's new:
The script has been renamed to genesisd.sh and is now able to init and upgrade. For init you're required to provide a moniker, but is optional in upgrading. If you don't provide one then it will try to extract the current moniker from the config.toml file in the existing .genesisd folder.
Initialization (which will generate a new key):
sh genesisd.sh init $YOUR_NEW_NODE_NAME
Upgrading (for if you already have an existing .genesisd folder and configuration):
sh genesisd.sh upgrade
Upgrading (for if you already have an existing .genesisd folder and configuration, but want a different node name):
sh genesisd.sh upgrade $YOUR_NEW_NODE_NAME
Bugfix in the Makefile:
If one removed the go/bin folder and ran the make install command, the script would create a file called bin instead of creating a folder called bin and placing the genesisd file there. The bugfix in the Makefile is therefore important to merge.

There's this one inconsistency that is happening throughout the script, code etc in general. Sometimes ~/ is used and sometimes it's /root/, this works all well if you're logged in as 'root', but doesn't if you're another user.
Therefore I've added a check in the script to see whether the user is logged in as the 'root'-user.


Install ELK logging in the staging environment

This has been implemented.

I use docker-compose to deploy the Sock Shop system. I would like to ask you how to use ELK to view the running log of Sock Shop. I also want to view the running log of the entire system. Could you please tell me how you deployed it?


ORA-01000 error again after ver:"enable Queryer"

My program prompted ORA-01000 error,But I can't give the test code.

Last change should fix this.


fix(ci): update the Buf CI to push tags to the Buf Schema Registry

This PR fixes the push of a new tag to BSR.
Tested in my fork in here: https://buf.build/chamirachid/nmt/labels

Summary by CodeRabbit

New Features

Introduced a new GitHub Actions workflow for continuous integration of Buf-related changes.
Added a workflow for automating the release of protobuf definitions to the Buf Schema Registry.
New script for downloading the specified version of the buf binary.

Bug Fixes

Removed the outdated buf-ci workflow to streamline processes.

failing go build CI is because I am creating the PR from a fork and codecov doesn't have permissions to check it.
the failing linter is fixed in https://github.com/celestiaorg/nmt/issues/278
the markdown contains some dead links but there is an issue to fix them


AnyCodable Extras

Context
This PR aims to introduce AnyCodable extras for the MediaProduct class, which will allow any aCodablea object from the client app to be passed as extras.
Description

The AnyCodable library was recently added to this project as part of the OpenAPI work being done. This simplified this piece of work quite a bit, as this library is exactly the solution that was proposed for this solution.
A new type (typealias for now) AnyCodableDictionary = [String: AnyCodable?] was introduced in the Common module. This was done to prevent the undesired spread of the AnyCodable type to the Player module. By doing this, the Player doesn't require new explicit dependencies at least.
In addition all Encodable types were extended with a asAnyCodableDictionary() adapter method to make it easy to convert any Encodable type to AnyCodableDictionary.
A new type MediaProduct.Extras (typealias for AnyCodableDictionary  were introduced to further curb the spread of the new type for users of the Player module. With this type, users of the Player doesn't require any other imports other than the Player. This allows independent change and evolution of this type, without impacting users of the module too much.
MediaProduct.Extras was also updated with its own asMediaProductExtras() conversion on Encodable types, which just uses asAnyCodableDictionary() under to hood. This is again to prevent the undesired spread of the AnyCodable type.
PlayerEvent.Extras was also updated accordingly with the same idea in mind.
Lastly, AnyCodableDictionaryConversionTests tests were added to prove the conversion to AnyCodableDictionary for any Encodable type.

I need to merge another PR today in a bit. Would you like for me to make a new release with both things for today's cutoff?
If you don't want it for today, don't merge it until tomorrow 😀 !

@asendra would be awesome if you can include this in the new version 💪


use fullnode and gateway service

https://user-images.githubusercontent.com/8676844/175378457-b53a45cf-8a0c-43b6-a704-c884513c12de.mov

@pchrysochoidis I added the fullnode service to the .env.default file

@pchrysochoidis I added the fullnode service to the .env.default file

@Jibz1 yeah I saw it, I just said we should remove the defaultRPCENV
(Also something else, I think we don't need the :443 for the endpoints since it's the default port for https)


remove rp5.ru

#246
This is not a Hungarian site, nor is it translated to Hungarian. Related discussion #150

This is an exception rule

Oh, sorry, I totally missed the @ in rp5.ru#@#[id^="aswift_"]
However, limiting ##[id^="aswift_"] to the hu TLD is indeed good, because then we don't need the rp5.ru#@#[id^="aswift_"] rule, plus hu##[id^="aswift_"] won't cause any other issues on foreign sites in the future.

@scripthunter7

could you please take a look?

I agree with @ehrhardt06 - hu##[id^="aswift_"] is enough. But I don't know if this rule is useful - usually google's aswift elements used with ins tag.

usually google's aswift elements used with ins tag.

Currently, all types of tags are covered by the attr selector, should we limit it to only ins tag?

@scripthunter7 another tags usually used in antiadblock baits. Like ##.adsbygoogle and ##ins.adsbygoogle - the second is more safe, the most scripts check div with a list of bait classes.

@Alex-302 thanks, changed


Nativescript Support

hello
can i use this plugin in Nativescript ?

@mshanak no, this is not supported.


fix load_state_dict for npu

What does this PR do?

Add npu support in load_state_dict when device_map value is an integer, since torch.device(0) defaults to cuda
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Hey @BenjaminBossan, all checks pass and I think this PR is ready to be merged 🤗

@muellerzr is this good to be merged?


Docs should include example of regex primitives

Also explain how primitives can be wrapped in a composite entity if used alongside non-regex primitives.

Fixed in Wiki


funny artifacts on pies

@evshvarov could you please provide an example or how to reproduce

Just click left-to-right buttons on decoration and back and you'll see it

It is here
the video

Switch on and then off annotations and then values

Done in 3.1.79


Missing default value for cooldown in generated config results in crash on first launch

ItemFlexer 1.1.1, Fabric 0.10.8
Crash Report: https://paste.ee/p/VGob0

That looks like you messed up the config file, make sure it's in a proper format

That's strange, that was the first time I launched the mod, so I didn't have the chance to modify the config.

Ah, I see. There's no default value for cooldown when the config file generates; setting it to 0 fixes this.

Strange, that shouldn't be possible

Yeah looking at microconfig's code that value should always be set, unless their some quirk of reflection or file writers I can't find easily
Is this consistent behavior? (Always happens when the config file is missing?)

Fixed in latest version


master/examples/custom_object_detection.py

When trying to process image with possible stop sign in it this error occurs, I think there is issue with detectCustomObjectsFromImage functions object name "stop sign", I think error occurs, because object name consists of two words.
Traceback (most recent call last):
  File "C:\Program Files\Python35\lib\site-packages\imageai\Detection\__init__.py", line 430, in detectCustomObjectsFromImage
    if (custom_objects[check_name] == "invalid"):
KeyError: 'stop sign'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Users\justa\Desktop\ObjectDetection project\detect.py", line 16, in <module>
    detections = detector.detectCustomObjectsFromImage(custom_objects=custom_objects, input_image=os.path.join(execution_path , "image3.jpg"), output_image_path=os.path.join(execution_path , "image3custom.jpg"))
  File "C:\Program Files\Python35\lib\site-packages\imageai\Detection\__init__.py", line 480, in detectCustomObjectsFromImage
    "Ensure you specified correct input image, input type, output type and/or output image path ")
ValueError: Ensure you specified correct input image, input type, output type and/or output image path

Thank you very much for bringing my notice to this. It will be fixed in next version. The next version of ImageAI will be available latest 2 weeks from now.

This has been solved. Thank you once again.

ValueError: Ensure you specified correct input image, input type, output type and/or output image path
This is the error what i get., Please guide me further

@Kanagachidambaresan It's hard to decipher what the problem is if you post the Error alone. Maybe you can open a new issue, describe the problem in detail, and post the traceback (not just the error alone).
If it's long, just copy and paste it in a log.txt file, and then upload.

How can apply it to a video?

How can we apply it to a video?

Visit the tutorials linked below.
https://heartbeat.fritz.ai/detecting-objects-in-videos-and-camera-feeds-using-keras-opencv-and-imageai-c869fe1ebcdb
https://heartbeat.fritz.ai/analyze-and-visualize-detected-video-objects-using-keras-and-imageai-d84c99b0ae8e


Fix typo and forgotten config keys

I did the mistake to assume I was perfect and the small changes wouldn't break my feature.
This fixes the Privacy mode, I should've tested the changes before committing.

Error 500 from Node.js 🤔


mgmt core, bug fix in ManagementErrorDeserializer

_objectCodec in JsonParser is not guaranteed to be available.
Use a similar solution as https://github.com/Azure/autorest-clientruntime-for-java/blob/master/azure-client-runtime/src/main/java/com/microsoft/azure/serializer/CloudErrorDeserializer.java#L53

Fix https://github.com/Azure/azure-sdk-for-java/issues/21615


Doesn't work with tables

Is there any plans on making this plugin working with tables?

@guilhermecvm the sprint example on the demo uses table structure.

The sprint example repeats a  inside a table.
Does it work for table rows ?
It seams that it is kinda ugly : http://plnkr.co/edit/eZepNR1OomLBHaDa2Rfj?p=preview

@a5hik as @MaitreDede said, it does not work with table rows..

This is now the third drag an drop library for angular I have rejected for my project for not supporting tables. Such a shame it was good in every other respect. I think it is the block formatting causing the issue meaning the row content doesn't align with the headers, but changing it to table-row make a mess when dragging it around.

@guilhermecvm @M4v3R @markantill please check with the latest as this is fixed as part of #237

The support is still very wonky also with the latest release. It's still like @markantill describes it, the formatting is messed up because of display: table-row is overwritten or if you change that, it is when dragging a row.


Create hello-world example

This is a start at #50. The main goal of this PR is to figure out the primary structure for an example directory (README.md structure, how to install @zeit/ncc, how to communicate which commands to run, etc.) so that it becomes easier to add other examples moving forward. Feedback welcome (inline comments incoming)!

Codecov Report

Merging #153 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #153   +/-   ##
======================================
  Coverage    72.3%   72.3%
======================================
  Files          10      10
  Lines         473     473
======================================
  Hits          342     342
  Misses        131     131

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 f7bc7a0...9e59cbb. Read the comment docs.

@rauchg addressed your comments. Here are examples of running the npm scripts from the examples/hello-world directory:

Thanks @macklinu

@rauchg this should be a good for another review - apologies for not sending a comment after my latest commit to communicate that more clearly. Let me know if I can help out without anything else on this PR. Will keep contributing example PRs if we like the start of this format. 👍
(This PR is also good to be squashed and merged unless you'd like me to squash/rebase it manually.)

Awesome @macklinu ! Next step would be to show off an example of using arg and a CLI with subcommands, where each subcommand is a different bundle


Assign fixed team assignments per table

By assigning fixed team assignments per table, it makes it easier for attendees to login using their team-ids.
User: team-1
Password: openshift
Define it in the corresponding workshop-interface.

[ ] Print Cards for every table - https://docs.google.com/document/d/1knjt_qV-nY4Oo4cOWvsTFnnOo6Pd3aN5o2s7MJTsAWs/edit?tab=t.0#heading=h.66y4kqbj468a
[ ] Printing and wrap it into plastic => @goetzrieger